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

Export top-level suite on global mocha object (mocha.suite). Closes #448 #458

Merged
merged 1 commit into from
Jun 18, 2012

Conversation

joliss
Copy link
Contributor

@joliss joliss commented Jun 11, 2012

This should probably be tested, since it's an API that we'll rely on in
Konacha, but I'm not sure what's the best way to do so.

@joliss
Copy link
Contributor Author

joliss commented Jun 16, 2012

@visionmedia do you think you can merge this even without a test? It would be covered by the Konacha test suite, so if anything breaks in the future, we'll find out pretty quickly.

@tj
Copy link
Contributor

tj commented Jun 16, 2012

what's the need to expose it? (just curious)

@joliss
Copy link
Contributor Author

joliss commented Jun 17, 2012

See #448. We'd like Konacha to work with every interface. For that it needs to have a way to set up its own beforeEach callback, regardless of the interface that's been activated.

…ochajs#448

This should probably be tested, since it's an API that we'll rely on in
Konacha, but I'm not sure what's the best way to do so.
@tj
Copy link
Contributor

tj commented Jun 18, 2012

sounds reasonable. I would like if we could do similar on the node-side of things since it's a bit inconsistent to do one but im cool with this for now

tj added a commit that referenced this pull request Jun 18, 2012
Export top-level suite on global mocha object (mocha.suite). Closes #448
@tj tj merged commit a1ca706 into mochajs:master Jun 18, 2012
@joliss
Copy link
Contributor Author

joliss commented Jun 18, 2012

I would like if we could do similar on the node-side of things

Oh, I hadn't realized this wouldn't work on Node. Having it on Node as well would definitely be reasonable.

Anyways, I'll go use this with Konacha now.

By the way, if you need to change the API later so it works with Node as well, don't worry about breaking compatibility for us (as long as it doesn't go away entirely) -- we'll just update our code as Mocha changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants