Fix: clone config before validating (fixes #12592)

lib/cli-engine/config-array-factory.js

@@ -623,7 +623,9 @@ class ConfigArrayFactory {
      * @private
      */
     _normalizeConfigData(configData, ctx) {
+
         validateConfigSchema(configData, ctx.name || ctx.filePath);
+
         return this._normalizeObjectConfigData(configData, ctx);
     }
 
@@ -697,6 +699,8 @@ class ConfigArrayFactory {
             ctx.matchBasePath
         );
 
+        const clonedRulesConfig = rules && JSON.parse(JSON.stringify((rules)));
+
         // Flatten `extends`.
         for (const extendName of extendList.filter(Boolean)) {
             yield* this._loadExtends(extendName, ctx);
@@ -731,7 +735,7 @@ class ConfigArrayFactory {
             processor,
             reportUnusedDisableDirectives,
             root,
-            rules,
+            rules: clonedRulesConfig,
             settings
         };
 

tests/fixtures/config-file/cloned-config/circularRefEslintConfig.js

@@ -0,0 +1,15 @@
+let errorConfig = ["error", {}];
+
+class CircularRef {
+  constructor() {
+    this.self = this;
+  }
+}
+
+module.exports = {
+  settings: {
+    sharedData: new CircularRef()
+  },
+
+  rules: { camelcase: errorConfig, "default-case": errorConfig }
+};

tests/fixtures/config-file/cloned-config/eslintConfig.js

@@ -0,0 +1,4 @@
+let errorConfig = ["error", {}];
+module.exports = {
+    rules: { camelcase: errorConfig, "default-case": errorConfig }
+};

tests/fixtures/config-file/cloned-config/eslintConfigFail.js

@@ -0,0 +1,13 @@
+let errorConfig = ["error", {}];
+module.exports = {
+    rules: {
+        camelcase: errorConfig,
+        "default-case": errorConfig ,
+        "camelcase" : [
+            'error',
+            {
+                "ignoreDestructuring": new Date().getUTCFullYear // returns a function
+            }
+        ]
+    }
+};

tests/fixtures/config-file/cloned-config/index.js

@@ -0,0 +1 @@
+var someValue  = "bar";

tests/fixtures/config-file/cloned-config/inlineText.js

@@ -0,0 +1,3 @@
+/*eslint default-case: ["error", {}]*/
+/*eslint camelcase: ["error", {}]*/
+var someValue  = "bar";

tests/lib/cli.js

@@ -187,6 +187,7 @@ describe("cli", () => {
         });
     });
 
+
     describe("when given a config with environment set to Node.js", () => {
         it("should execute without any errors", async () => {
             const configPath = getFixturePath("configurations", "env-node.json");
@@ -1160,4 +1161,45 @@ describe("cli", () => {
         });
     });
 
+    describe("testing the cloned config", () => {
+        describe("config file and input file", () => {
+            it("should not modify original configuration object", async () => {
+                const configPath = getFixturePath("config-file", "cloned-config", "eslintConfig.js");
+                const filePath = getFixturePath("config-file", "cloned-config", "index.js");
+                const args = `--config ${configPath} ${filePath}`;
+
+                const exit = await cli.execute(args);
+
+                assert.strictEqual(exit, 0);
+            });
+        });
+
+        describe("config file and input file", () => {
+            it("should exit with 1 as camelcase has wrong property type", async () => {
+                const configPath = getFixturePath("config-file", "cloned-config", "eslintConfigFail.js");
+                const filePath = getFixturePath("config-file", "cloned-config", "index.js");
+                const args = `--config ${configPath} ${filePath}`;
+
+                try {
+                    await cli.execute(args);
+                } catch (error) {
+                    assert.strictEqual(/Configuration for rule "camelcase" is invalid:/u.test(error), true);
+                }
+
+            });
+        });
+
+        describe("inline config and input file", () => {
+            it("should not modify original configuration object", async () => {
+                const filePath = getFixturePath("config-file", "cloned-config", "inlineText.js");
+                const args = `${filePath}`;
+
+                const exit = await cli.execute(args);
+
+                assert.strictEqual(exit, 0);
+            });
+        });
+
+    });
+
 });