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

Expose skip in ember-qunit #164

Closed
wants to merge 1 commit into from

Conversation

olivierlesnicki
Copy link

No description provided.

@rwjblue
Copy link
Member

rwjblue commented Apr 22, 2015

import { skip } from 'qunit'; will give you skip

@olivierlesnicki
Copy link
Author

By default ember-cli generator uses import { test } from ember-qunit. I'll this raise it against ember-cli to make sure the same qunit is imported on all blueprints be it qunit or ember-qunit.

@pangratz
Copy link
Member

pangratz commented May 6, 2015

@olivierlesnicki be aware though that import { test } from 'ember-qunit' and import { test } from 'qunit' don't import the same test, as the ember-qunit one adds an assertion for failed promises.

@olivierlesnicki
Copy link
Author

@pangratz Ah! This is not obvious for a new user. ember-cli blueprints sometimes generate test that import test from qunit (e.g. controller-test), sometimes test that import test from ember-qunit (e.g. acceptance-test)

@olivierlesnicki
Copy link
Author

@pangratz @rwjblue - the behavior of test from q-unit could actually be the reason for an other issue I've raised: https://github.com/rwjblue/ember-qunit/issues/163

@rwjblue rwjblue force-pushed the master branch 2 times, most recently from 3173a7f to 85e6e7d Compare October 1, 2015 13:49
@trentmwillis
Copy link
Member

Hate to revive an old thread, but I'd like to see skip added as an export in ember-qunit as well. I don't think it makes much sense to force two different import statements on devs (one for moduleFor, test and another for skip). Additionally, I've seen several developers try to debug why skip wasn't working after attempting to import it from ember-qunit. I'll gladly open a new PR if this makes sense.

cc @nickiaconis

@rwjblue
Copy link
Member

rwjblue commented Nov 12, 2015

@trentmwillis - Adding a skip and module export to ember-qunit that does throw new Error('Please import skip from qunit'); should provide enough guidance to folks that accidentally use the wrong one...

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.

4 participants