js-confuser 1.6.0 → 1.7.1

package/test/transforms/identifier/movedDeclarations.test.ts

@@ -12,7 +12,7 @@ test("Variant #1: Move variable 'y' to top", async () => {
     movedDeclarations: true,
   });
 
-  expect(output).toContain("var y;");
+  expect(output).toContain("var x=10,y;");
   expect(output).toContain("y=15");
 
   var TEST_VARIABLE;
@@ -34,7 +34,7 @@ test("Variant #2: Move variable 'y' and 'z' to top", async () => {
     movedDeclarations: true,
   });
 
-  expect(output).toContain("var y,z");
+  expect(output).toContain("var x=10,y,z;");
   expect(output).toContain("y=15");
   expect(output).toContain("z=5");
 
@@ -44,7 +44,7 @@ test("Variant #2: Move variable 'y' and 'z' to top", async () => {
   expect(TEST_VARIABLE).toStrictEqual(30);
 });
 
-test("Variant #2: Don't move 'y' (destructuring)", async () => {
+test("Variant #3: Don't move 'y' (destructuring)", async () => {
   var code = `
     var x = 10;
     var [y] = [15];
@@ -64,7 +64,7 @@ test("Variant #2: Don't move 'y' (destructuring)", async () => {
   expect(TEST_VARIABLE).toStrictEqual(25);
 });
 
-test("Variant #3: Don't move 'y' (nested lexical scope)", async () => {
+test("Variant #4: Move 'y' (nested lexical scope)", async () => {
   var code = `
     var x = 10;
     var y = 15;
@@ -81,7 +81,7 @@ test("Variant #3: Don't move 'y' (nested lexical scope)", async () => {
     movedDeclarations: true,
   });
 
-  expect(output).toContain("var y=15");
+  expect(output).toContain("var x=10,y;");
 
   var TEST_VARIABLE;
   eval(output);
@@ -89,7 +89,7 @@ test("Variant #3: Don't move 'y' (nested lexical scope)", async () => {
   expect(TEST_VARIABLE).toStrictEqual(20);
 });
 
-test("Variant #4: Move 'y' (for statement initializer)", async () => {
+test("Variant #5: Move 'y' (for statement initializer)", async () => {
   var code = `
     var x = 10;
     for ( var y = 0; y < 15; y++ ) {
@@ -111,7 +111,7 @@ test("Variant #4: Move 'y' (for statement initializer)", async () => {
   expect(TEST_VARIABLE).toStrictEqual(25);
 });
 
-test("Variant #5: Move 'y' (for-in left-hand initializer)", async () => {
+test("Variant #6: Move 'y' (for-in left-hand initializer)", async () => {
   var code = `
     var x = 10;
     for ( var y in [0,1,2,3,4,5,6,7,8,9,10,11,12,13,14,15] ) {
@@ -134,7 +134,7 @@ test("Variant #5: Move 'y' (for-in left-hand initializer)", async () => {
   expect(TEST_VARIABLE).toStrictEqual(25);
 });
 
-test("Variant #6: Don't move const or let variables", async () => {
+test("Variant #7: Don't move const or let variables", async () => {
   var code = `
     var fillerExpr;
 
@@ -158,7 +158,7 @@ test("Variant #6: Don't move const or let variables", async () => {
   expect(TEST_VARIABLE).toStrictEqual(25);
 });
 
-test("Variant #7: Work with 'use strict'", async () => {
+test("Variant #8: Work with 'use strict'", async () => {
   var code = `
   function myFunction(){
     'use strict';
@@ -184,3 +184,31 @@ test("Variant #7: Work with 'use strict'", async () => {
 
   expect(TEST_OUTPUT).toStrictEqual(true);
 });
+
+test("Variant #9: Defined variable without an initializer", async () => {
+  var code = `
+  var x;
+  x = 1;
+  var y;
+  y = 2;
+  TEST_OUTPUT = x + y;
+  `;
+
+  var output1 = await JsConfuser(code, {
+    target: "node",
+    movedDeclarations: true,
+    controlFlowFlattening: true,
+    duplicateLiteralsRemoval: true,
+  });
+
+  var output2 = await JsConfuser(output1, {
+    target: "node",
+    movedDeclarations: true,
+    controlFlowFlattening: true,
+    duplicateLiteralsRemoval: true,
+  });
+
+  var TEST_OUTPUT;
+  eval(output2);
+  expect(TEST_OUTPUT).toStrictEqual(3);
+});

package/test/transforms/identifier/renameVariables.test.ts

@@ -582,3 +582,40 @@ test("Variant #23: Re-use previously generated names", async () => {
 
   expect(TEST_OUTPUT).toStrictEqual("Correct Value");
 });
+
+test("Variant #24: Reference function name with parameter", async () => {
+  var output = await JsConfuser(
+    `
+  function myFunction(myFunction){
+    myFunction.property = "Correct Value";
+  }
+
+  myFunction(myFunction);
+  TEST_OUTPUT = myFunction.property;
+  `,
+    { target: "node", renameVariables: true }
+  );
+
+  var TEST_OUTPUT;
+  eval(output);
+
+  expect(TEST_OUTPUT).toStrictEqual("Correct Value");
+});
+
+test("Variant #25: Reference catch parameter", async () => {
+  var output = await JsConfuser(
+    `
+  try {
+    throw "Correct Value";
+  } catch ( e ) {
+    TEST_OUTPUT = e;
+  }
+  `,
+    { target: "node", renameVariables: true }
+  );
+
+  var TEST_OUTPUT;
+  eval(output);
+
+  expect(TEST_OUTPUT).toStrictEqual("Correct Value");
+});

package/test/transforms/lock/integrity.test.ts

@@ -135,3 +135,27 @@ it("should work on High Preset", async () => {
 
   expect(TEST_OUTPUT).toStrictEqual("Hello World");
 });
+
+it("should work with RGF enabled", async () => {
+  var output = await JsConfuser(
+    `
+  function getTestOutput(){
+    return "Hello World";
+  }
+
+  TEST_OUTPUT = getTestOutput();
+  `,
+    {
+      target: "node",
+      rgf: true,
+      lock: {
+        integrity: true,
+      },
+    }
+  );
+
+  var TEST_OUTPUT;
+  eval(output);
+
+  expect(TEST_OUTPUT).toStrictEqual("Hello World");
+});

package/test/transforms/lock/lock.test.ts

@@ -64,6 +64,7 @@ it("should work with endDate and call countermeasures function", async () => {
   expect(value).toStrictEqual(true);
 });
 
+// REMOVED FEATURE:
 // it("strings should be encoded when startDate and endDate are given", async () => {
 //   var startDate = await JsConfuser.obfuscate(` input("ENCODED_STRING") `, {
 //     target: "node",
@@ -82,54 +83,6 @@ it("should work with endDate and call countermeasures function", async () => {
 //   expect(value).toStrictEqual("ENCODED_STRING");
 // });
 
-it("should work with nativeFunctions and call countermeasures function", async () => {
-  var output = await JsConfuser.obfuscate(
-    ` function countermeasures(){ input(true) } `,
-    {
-      target: "node",
-      lock: {
-        nativeFunctions: ["fetch"],
-        countermeasures: "countermeasures",
-      },
-    }
-  );
-
-  // custom function, not "native"
-  var fetch = () => {};
-
-  var value = "never_called";
-  function input(valueIn) {
-    value = valueIn;
-  }
-
-  eval(output);
-  expect(value).toStrictEqual(true);
-});
-
-it("should work with nativeFunctions and not call countermeasures function when correct", async () => {
-  var output = await JsConfuser.obfuscate(
-    ` function countermeasures(){ input(true) } `,
-    {
-      target: "node",
-      lock: {
-        nativeFunctions: ["fetch"],
-        countermeasures: "countermeasures",
-      },
-    }
-  );
-
-  // bound functions return the "[native code]" string
-  var fetch = (() => {}).bind(this);
-
-  var value = "never_called";
-  function input(valueIn) {
-    value = valueIn;
-  }
-
-  eval(output);
-  expect(value).toStrictEqual("never_called");
-});
-
 it("countermeasures function should still work even with renameVariables enabled", async () => {
   var output = await JsConfuser.obfuscate(
     ` function countermeasures(){ input(true) } `,

package/test/transforms/minify.test.ts

@@ -517,3 +517,22 @@ test("Variant #26: Don't break nested redefined function declaration", async ()
 
   expect(TEST_OUTPUT).toStrictEqual(1);
 });
+
+// https://github.com/MichaelXF/js-confuser/issues/91
+test("Variant #27: Preserve function.length property", async () => {
+  var output = await JsConfuser(
+    `
+    function oneParameter(a){};
+    var twoParameters = function({a},{b,c},...d){};
+    function threeParameters(a,b,c,d = 1,{e},...f){};
+
+    TEST_OUTPUT = oneParameter.length + twoParameters.length + threeParameters.length;
+  `,
+    { target: "node", minify: true }
+  );
+
+  var TEST_OUTPUT;
+  eval(output);
+
+  expect(TEST_OUTPUT).toStrictEqual(6);
+});