Summary

Fixes passing parameters to Nose when running tests.

Review Request #995 — Created Aug. 25, 2009 and submitted

Information

Owner

helder

Repository

Review Board

Branch

http://github.com/obvio171/reviewboard/tree/nose_bug

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

(This revreq already got a "ShipIt" from David Trowbridge; awaiting integration upstream)

Fixes passing parameters to Nose when running tests.

./manage.py captures any parameter passed before an optional "--", so
running:

./manage.py test -s

to pass the -s to nose (to make it not capture stdout) doesn't work,
and manage.py dies with a "no such option" error. With this fix, this
works:

./manage.py test -- -s

Reviewed at: http://reviews.review-board.org/r/995

Testing Done

Ran ./manage.py test -- -s and nose ran without capturing stdout.

Description:

	+	(This revreq already got a "ShipIt" from David Trowbridge; awaiting integration upstream)
	+
		Fixes passing parameters to Nose when running tests.

		./manage.py captures any parameter passed before an optional "--", so
		running:

		./manage.py test -s

		to pass the -s to nose (to make it not capture stdout) doesn't work,
		and manage.py dies with a "no such option" error. With this fix, this
		works:

		./manage.py test -- -s

		Reviewed at: http://reviews.review-board.org/r/995

Ship it!

So, I don't know if Christian has given you push access to the clone under the 'reviewboard' account, but if not, it's probably easiest if you push this to your fork on github, then I can merge it into the main fork.

chipx86 Sept. 9, 2009, 7:06 p.m.

Come get us on IRC and we'll set you up.

Branch:

+	http://github.com/obvio171/reviewboard/tree/nose_bug

Ship it!

```
Merged into master. Thanks!
```