Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Programmatically specify reporter in .run() #190

Closed
wants to merge 1 commit into from

Conversation

diversario
Copy link

Unless I missed the existing way of doing this.

Like this:

.run({reporter: 'spec'});

If reporter is not found - it would default to dot-matrix.

@ghost
Copy link

ghost commented Feb 15, 2012

Good one. Please merge it.

@Myztiq
Copy link

Myztiq commented Feb 27, 2012

Seconded, I think this would be very helpful.

@JerrySievert
Copy link

i like the idea, but i'm getting failing tests with this addition.

@JerrySievert
Copy link

i would love to be able to merge this, but cannot with failing tests. please update. :)

thanks!

@ujjwal
Copy link

ujjwal commented Mar 19, 2013

Would be really nice to have this bit of functionality, are the tests still failing on the current build? I patched my local suite.js and the tests pass.

@jokesterfr
Copy link

+1 very usefull
Easier than any:

var spec = require(modules+'vows/lib/vows/reporters/spec');
/* Some stuff */
.run({reporter:spec}); 

@indexzero
Copy link

Cherry-picked. @JerrySievert the problem here was that it broke backwards compatibility for object-based reporters.

@indexzero indexzero closed this Nov 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants