Merge pull request #439 from zamotkin/bug/437_no-value-node

Fixed issue #437

src/node-transformers/converting-transformers/properties-extractors/AbstractPropertiesExtractor.ts

@@ -83,7 +83,8 @@ export abstract class AbstractPropertiesExtractor implements IPropertiesExtracto
      * @returns {propertyValueNode is Pattern}
      */
     protected static isProhibitedPattern (node: ESTree.Node): node is ESTree.Pattern {
-        return NodeGuards.isObjectPatternNode(node)
+        return !node
+            || NodeGuards.isObjectPatternNode(node)
             || NodeGuards.isArrayPatternNode(node)
             || NodeGuards.isAssignmentPatternNode(node)
             || NodeGuards.isRestElementNode(node);

test/functional-tests/issues/fixtures/issue437.js

@@ -0,0 +1,10 @@
+var obj1={
+    firstname:'John',
+    lastname:'Smith'
+};
+var obj2={
+    age:40,
+    country:'USA'
+}
+var user={...obj1, ...obj2}
+console.log(user);

test/functional-tests/issues/issue437.spec.ts

@@ -0,0 +1,29 @@
+import { assert } from 'chai';
+import { NO_ADDITIONAL_NODES_PRESET } from '../../../src/options/presets/NoCustomNodes';
+import { readFileAsString } from '../../helpers/readFileAsString';
+import { JavaScriptObfuscator } from '../../../src/JavaScriptObfuscatorFacade';
+
+//
+// https://github.com/javascript-obfuscator/javascript-obfuscator/issues/437
+//
+describe('Issue #437', () => {
+
+    describe('Fixture code should not break on obfuscating', () => {
+        let doObfuscate: Function;
+
+        before(() => {
+            const code: string = readFileAsString(__dirname + '/fixtures/issue437.js');
+            doObfuscate = () => JavaScriptObfuscator.obfuscate(
+                code,
+                {
+                    ...NO_ADDITIONAL_NODES_PRESET,
+                    transformObjectKeys: true
+                }
+            ).getObfuscatedCode();
+        });
+
+        it('does not break on obfuscating', () => {
+            assert.doesNotThrow(doObfuscate);
+        });
+    });
+});