-
-
Notifications
You must be signed in to change notification settings - Fork 59
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
Mocha@3: it.only does not work #73
Comments
I've never had issues with this. Out of curiosity, have you tried it without the |
Hey yeah I tried describe.only as well, with the same effect. For now the solution with grep is totally fine for me. |
We will need a demo repo to track this down...
|
This has always worked for me. Here's an example |
I have, however, noticed that you can't use |
I'm also seeing this now, I just don't know yet at what point it surfaced. Update: my issue surfaced because mocha implicitly upgraded to 3.0.0. This isn't an issue with ember-cli-mocha. Backing it back down to 2.x resolved the issue. |
@jasonmit do you have any clue why this popped up with Mocha 3? |
this is most likely related to https://github.com/teddyzeenny/ember-mocha-adapter/blob/master/adapter.js#L169-L172 and Mocha 3 having done some changes in the |
Any news on this? Is this still blocking #167? |
@phyrog yes, haven't found a good solution to this issue yet 😞 |
IMHO, we should drop the auto-run wrapping that we do inside @Turbo87 - Is that correct? |
@rwjblue in theory yes, but it would obviously be annoying for Mocha users that don't use async/await yet. I wanted to go back and see if there is really no way to us to hook in anywhere but I haven't found the time yet 😞 |
Hey I'm trying to only run a single test with it.only like it's defined in here https://mochajs.org/#exclusive-tests, but both tests are running when i open up my test suite in the browser. Am I doing something wrong or is this feature not supported by ember-cli-mocha?
My test looks like:
Update:
A workarround to this issue is to use the grep argument and paste in the test name. Like
Thanks,
Thomas
The text was updated successfully, but these errors were encountered: