fix: error only when karma is navigating

This commit fixes a bug whereby Karma mistakenly errors out when it is *not* navigating. This is because the variable was flipped from `reloadingContext` to `karmaNavigating` in karma-runner#3565, but the condition was not negated.
kyliau · Nov 16, 2020 · 117e40c · 117e40c
1 parent 05dc288
commit 117e40c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/client/karma.js b/client/karma.js
@@ -131,7 +131,7 @@ function Karma (socket, iframe, opener, navigator, location, document) {
   }
 
   this.onbeforeunload = function () {
-    if (!karmaNavigating) {
+    if (karmaNavigating) {
       // TODO(vojta): show what test (with explanation about jasmine.UPDATE_INTERVAL)
       self.error('Some of your tests did a full page reload!')
     }

diff --git a/static/karma.js b/static/karma.js
@@ -141,7 +141,7 @@ function Karma (socket, iframe, opener, navigator, location, document) {
   }
 
   this.onbeforeunload = function () {
-    if (!karmaNavigating) {
+    if (karmaNavigating) {
       // TODO(vojta): show what test (with explanation about jasmine.UPDATE_INTERVAL)
       self.error('Some of your tests did a full page reload!')
     }