Skip to content
This repository has been archived by the owner on Dec 28, 2023. It is now read-only.

Mocha opts option #99

Closed
wants to merge 3 commits into from
Closed

Mocha opts option #99

wants to merge 3 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Jun 19, 2016

  • add opts mocha option
  • opts option supports globals, delay, timeout

@ghost ghost mentioned this pull request Jun 19, 2016
@dignifiedquire
Copy link
Member

Thanks, could you add some docs about how this is used, what places are looked in for the config and what takes precedence over what?

@ghost
Copy link
Author

ghost commented Jun 24, 2016

Hi,

I have just updated the doc.

As I said here, only the flag '--require' is missing.

I want to implement it by creating a bundle with the required files then to inject it in the window when running the tests. In order to do so, do you think that Webpack is a nice solution?

About the tests with TravisCI, they can't succeed in because my pull-request changes the index.js file and its behviour.

@dignifiedquire
Copy link
Member

I want to implement it by creating a bundle with the required files then to inject it in the window when running the tests. In order to do so, do you think that Webpack is a nice solution?

I'm not sure this should be implemented to be honest, and if so it should rather try to leverage karma itself to inject files rather than some other process.

About the tests with TravisCI, they can't succeed in because my pull-request changes the index.js file and its behviour.

If so please update the existing tests accordingly so that they pass

@maksimr maksimr mentioned this pull request Nov 5, 2016
@maksimr
Copy link
Contributor

maksimr commented Nov 5, 2016

I close this PR in favor this because here no feedback from author

@maksimr maksimr closed this Nov 5, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants