JS backend: fix TranslationUtils.isCacheNeeded

js/js.tests/test/org/jetbrains/k2js/test/semantics/EvaluationOrderTest.java

@@ -112,4 +112,8 @@ public class EvaluationOrderTest extends AbstractExpressionTest {
     public void testOrOrWithBreakContinueReturn() throws Exception {
         checkFooBoxIsOk();
     }
+
+    public void testWhenJsLiteralWithSideEffect() throws Exception {
+        checkFooBoxIsOk();
+    }
 }

js/js.translator/src/org/jetbrains/k2js/translate/utils/TranslationUtils.java

@@ -397,7 +397,7 @@ public final class TranslationUtils {
     }
 
     public static boolean isCacheNeeded(@NotNull JsExpression expression) {
-        return !(expression instanceof JsLiteral) &&
+        return !(expression instanceof JsLiteral.JsValueLiteral) &&
                (!(expression instanceof JsNameRef) || ((JsNameRef) expression).getQualifier() != null);
     }
 

js/js.translator/testData/expression/evaluationOrder/cases/whenJsLiteralWithSideEffect.kt

@@ -0,0 +1,38 @@
+package foo
+
+var global: String = ""
+
+fun bar(s: String): Int {
+    global += s
+    return 1
+}
+fun testWhen() {
+    global = ""
+    when(array(bar("A"),2,3)) {
+        array(1) -> println("1")
+        array(2) -> println("2")
+        else  -> println("else")
+    }
+    assertEquals("A", global)
+
+}
+
+fun testIntrinsic() {
+    global = ""
+    val x = array(bar("A")) == try { array(bar("B")) } finally {}
+    assertEquals("AB", global)
+}
+
+fun testElvis() {
+    global = ""
+    var x = array(bar("A")) ?: 10
+    assertEquals("A", global)
+}
+
+fun box(): String {
+    testWhen()
+    testIntrinsic()
+    testElvis()
+
+    return "OK"
+}