You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I really like the new Mocha config files. I've fully switched over to them from mocha.opts. I also use the --config flag to split my e2e test config from my unit test config.
Both config file resolution and the --config flag do not work at the moment. Well, to be more precise, some config is picked up and others are not.
So I took a look at the code. It seems like this might be the only option that is not super simple to implement. It seems like we could pass the resolved config file to MochaWrapper here https://github.com/mocha-parallel/mocha-parallel-tests/blob/master/src/bin/cli.ts#L53. However, this would mean that the custom config logic needed for parallel stuff is bypassed.
I'm willing to implement this but I'm not sure how to approach this since I'm not very well versed in this code or using Mocha programmatically. For my tests right now I'll try to find a workaround.
The text was updated successfully, but these errors were encountered:
I really like the new Mocha config files. I've fully switched over to them from
mocha.opts
. I also use the--config
flag to split my e2e test config from my unit test config.Both config file resolution and the
--config
flag do not work at the moment. Well, to be more precise, some config is picked up and others are not.So I took a look at the code. It seems like this might be the only option that is not super simple to implement. It seems like we could pass the resolved config file to
MochaWrapper
here https://github.com/mocha-parallel/mocha-parallel-tests/blob/master/src/bin/cli.ts#L53. However, this would mean that the custom config logic needed for parallel stuff is bypassed.Do we have to add each config item to
process.argv
instead, like what's happening with the oldmocha.opts
file? https://github.com/mocha-parallel/mocha-parallel-tests/blob/master/src/bin/cli.ts#L45 That approach feels iffy for the new config files though.I'm willing to implement this but I'm not sure how to approach this since I'm not very well versed in this code or using Mocha programmatically. For my tests right now I'll try to find a workaround.
The text was updated successfully, but these errors were encountered: