Keep user options intact in transformFile (#6890)

* Preserve user options in transformFile

* Improve tests for transformFile user opts handling

packages/babel-core/src/transform-file-sync.js

@@ -8,13 +8,14 @@ export default function transformFileSync(
   filename: string,
   opts: ?InputOptions,
 ): FileResult | null {
+  let options;
   if (opts == null) {
-    opts = { filename };
+    options = { filename };
   } else if (opts && typeof opts === "object") {
-    opts = Object.assign(opts, { filename });
+    options = Object.assign({}, opts, { filename });
   }
 
-  const config = loadConfig(opts);
+  const config = loadConfig(options);
   if (config === null) return null;
 
   return runSync(config, fs.readFileSync(filename, "utf8"));

packages/babel-core/src/transform-file.js

@@ -10,21 +10,22 @@ type TransformFile = {
 };
 
 export default ((function transformFile(filename, opts, callback) {
+  let options;
   if (typeof opts === "function") {
     callback = opts;
     opts = undefined;
   }
 
   if (opts == null) {
-    opts = { filename };
+    options = { filename };
   } else if (opts && typeof opts === "object") {
-    opts = Object.assign(opts, { filename });
+    options = Object.assign({}, opts, { filename });
   }
 
   process.nextTick(() => {
     let cfg;
     try {
-      cfg = loadConfig(opts);
+      cfg = loadConfig(options);
       if (cfg === null) return callback(null, null);
     } catch (err) {
       return callback(err);

packages/babel-core/test/api.js

@@ -118,26 +118,33 @@ describe("api", function() {
   });
 
   it("transformFile", function(done) {
-    babel.transformFile(
+    const options = {
-      __dirname + "/fixtures/api/file.js",
+      babelrc: false,
-      {
+    };
-        babelrc: false,
+    Object.freeze(options);
-      },
+    babel.transformFile(__dirname + "/fixtures/api/file.js", options, function(
-      function(err, res) {
+      err,
-        if (err) return done(err);
+      res,
-        assert.equal(res.code, "foo();");
+    ) {
-        done();
+      if (err) return done(err);
-      },
+      assert.equal(res.code, "foo();");
-    );
+      // keep user options untouched
+      assert.deepEqual(options, { babelrc: false });
+      done();
+    });
   });
 
   it("transformFileSync", function() {
+    const options = {
+      babelrc: false,
+    };
+    Object.freeze(options);
     assert.equal(
-      babel.transformFileSync(__dirname + "/fixtures/api/file.js", {
+      babel.transformFileSync(__dirname + "/fixtures/api/file.js", options)
-        babelrc: false,
+        .code,
-      }).code,
       "foo();",
     );
+    assert.deepEqual(options, { babelrc: false });
   });
 
   it("options throw on falsy true", function() {