fix(adapter): do not emit debug url unless in a browser

johnjbarton · johnjbarton · commit 27f5dc74e3b6 · 2020-02-10T09:58:23.000-08:00
Surprise: someone uses karma-jasmine with native=script. Fixes karma-runner#262
diff --git a/src/adapter.js b/src/adapter.js
@@ -247,8 +247,10 @@ function KarmaReporter (tc, jasmineEnv) {
         result.log.push(formatFailedStep(steps[i]))
       }
 
-      // Report the name of fhe failing spec so the reporter can emit a debug url.
-      result.debug_url = debugUrl(specResult.fullName)
+      if (typeof window !== 'undefined' && window.location && window.location.origin) {
+        // Report the name of fhe failing spec so the reporter can emit a debug url.
+        result.debug_url = debugUrl(specResult.fullName)
+      }
     }
 
     // When failSpecWithNoExpectations is true, Jasmine will report specs without expectations as failed

Original file line number	Diff line number	Diff line change
`@@ -247,8 +247,10 @@ function KarmaReporter (tc, jasmineEnv) {`
`247`	`247`	`result.log.push(formatFailedStep(steps[i]))`
`248`	`248`	`}`
`249`	`249`
`250`		`- // Report the name of fhe failing spec so the reporter can emit a debug url.`
`251`		`- result.debug_url = debugUrl(specResult.fullName)`
	`250`	`+ if (typeof window !== 'undefined' && window.location && window.location.origin) {`
	`251`	`+ // Report the name of fhe failing spec so the reporter can emit a debug url.`
	`252`	`+ result.debug_url = debugUrl(specResult.fullName)`
	`253`	`+ }`
`252`	`254`	`}`
`253`	`255`
`254`	`256`	`// When failSpecWithNoExpectations is true, Jasmine will report specs without expectations as failed`