Merge pull request #146 from JetBrains/ktl-649-fix-empty-junit-run

nikpachoo · web-flow · commit d2eadaeead14 · 2022-01-26T18:30:07.000+03:00
KTL-649 Playground -  Run JUNIT with an empty state leads to JS error
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "kotlin-playground",
-  "version": "1.27.1",
+  "version": "1.27.2",
   "description": "Self-contained component to embed in websites for running Kotlin code",
   "keywords": [
     "kotlin",
diff --git a/src/webdemo-api.js b/src/webdemo-api.js
@@ -99,7 +99,7 @@ export default class WebDemoApi {
             if (data.text) output = processJVMOutput(data.text, theme);
             break;
           case TargetPlatform.JUNIT:
-            data.testResults ? output = processJUnitResults(data.testResults, onTestPassed, onTestFailed) : output = processJVMOutput(data.text, theme);
+            data.testResults ? output = processJUnitResults(data.testResults, onTestPassed, onTestFailed) : output = processJVMOutput(data.text || '', theme);
             break;
         }
       }

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "kotlin-playground",`
`3`		`- "version": "1.27.1",`
	`3`	`+ "version": "1.27.2",`
`4`	`4`	`"description": "Self-contained component to embed in websites for running Kotlin code",`
`5`	`5`	`"keywords": [`
`6`	`6`	`"kotlin",`
Original file line number	Diff line number	Diff line change
`@@ -99,7 +99,7 @@ export default class WebDemoApi {`
`99`	`99`	`if (data.text) output = processJVMOutput(data.text, theme);`
`100`	`100`	`break;`
`101`	`101`	`case TargetPlatform.JUNIT:`
`102`		`- data.testResults ? output = processJUnitResults(data.testResults, onTestPassed, onTestFailed) : output = processJVMOutput(data.text, theme);`
	`102`	`+ data.testResults ? output = processJUnitResults(data.testResults, onTestPassed, onTestFailed) : output = processJVMOutput(data.text \|\| '', theme);`
`103`	`103`	`break;`
`104`	`104`	`}`
`105`	`105`	`}`