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

Latest version of mocha-v7.0.0 seems to be breaking mocha-multi #68

Closed
HarshaNalluru opened this issue Jan 8, 2020 · 2 comments
Closed

Comments

@HarshaNalluru
Copy link

HarshaNalluru commented Jan 8, 2020

mocha recently had a release, version 7.0.0.
The current mocha-multi library can't seem to take in the changes from this latest version.

In a branch of my fork for the mocha-multi repo, updated the mocha version to 7.0.0 to verify - HarshaNalluru@3b6d5a5

The tests seem to fail.

With npm run test..

> mocha-multi@1.1.3 test <path>/mocha-multi
> run-s lint verify:*


> mocha-multi@1.1.3 lint <path>/mocha-multi
> eslint . --max-warnings 0


> mocha-multi@1.1.3 verify:node <path>/mocha-multi
> node ./verify.js

ERROR: reporter definitions should be set in the `multi` shell variable
eg. `multi='dot=- xunit=file.xml' mocha`
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! mocha-multi@1.1.3 verify:node: `node ./verify.js`
npm ERR! Exit status 1
npm ERR! 
npm ERR! Failed at the mocha-multi@1.1.3 verify:node script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

@glenjamin
Copy link
Owner

glenjamin commented Jan 13, 2020

If anyone has any insight into this issue, a PR would be appeciated.

@MarcCelani-at
Copy link
Contributor

All the tests seem to pass on mocha 9, in case you still care.

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

No branches or pull requests

3 participants