js-confuser 1.6.0 → 1.7.0

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/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);
+});