Skip to content
This repository has been archived by the owner on Sep 30, 2021. It is now read-only.

fix(getKarmaConfig): remove broken restartOnChange #44

Merged
merged 2 commits into from
Nov 11, 2016

Conversation

levithomason
Copy link
Contributor

@levithomason levithomason commented Nov 11, 2016

This option is borked in Karma. It is causing tests to not run and the browser to disconnect.

karma-runner/karma#1640 (comment)

@@ -16,6 +17,19 @@ module.exports = function test(karmaConfig) {
else reject(code)
})

if (karmaConfig.logLevel && karmaConfig.logLevel === 'DEBUG') {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added some debug for future use. gen test --debug

@@ -62,9 +62,6 @@ module.exports = function getKarmaConfig(config, webpackConfig) {
[config.test_entry]: ['webpack'],
},
reporters: ['mocha', 'coverage'],
// When watching, Karma delays new runs until the current run is finished.
// Cancel the current run and start a new run immediately on file change.
restartOnFileChange: true,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The culprit.

@dvdzkwsk
Copy link
Contributor

👻

@levithomason
Copy link
Contributor Author

ty ty!

@levithomason levithomason merged commit d39db3b into master Nov 11, 2016
@levithomason levithomason deleted the fix/test-watch branch November 11, 2016 21:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants