Fix #45

MichaelXF · Dec 24, 2022 · 39156df · 39156df
1 parent 47e908e
commit 39156df
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 2 deletions.
diff --git a/src/transforms/opaquePredicates.ts b/src/transforms/opaquePredicates.ts
@@ -199,8 +199,8 @@ export default class OpaquePredicates extends Transform {
       }
 
       var cloned = clone(expr);
-      if (object.type == "SwitchCase") {
-        var matching = Identifier(choice(["undefined", "null"]));
+      if (object.type == "SwitchCase" && object.test) {
+        var matching: Node = Identifier(choice(["undefined", "null"]));
 
         var test = object.test;
 

diff --git a/test/transforms/opaquePredicates.test.ts b/test/transforms/opaquePredicates.test.ts
@@ -16,3 +16,28 @@ it("should append logical expressions", async () => {
 
   expect(output).not.toContain("(test)");
 });
+
+// https://github.com/MichaelXF/js-confuser/issues/45
+it("should work on default Switch cases", async ()=>{
+    var code = `
+
+    switch (0) {
+        default:
+             input(true);
+    }
+  `;
+
+  var output = await JsConfuser(code, {
+    target: "browser",
+    opaquePredicates: true,
+  });
+
+  var value;
+  function input(valueIn){
+    value = valueIn;
+  }
+
+  eval(output);
+
+  expect(value).toStrictEqual(true);
+})