clone settings so they can't be mutated, add test

packages/server/lib/config.js

@@ -209,8 +209,8 @@ module.exports = {
       return this.set({
         projectName: this.getNameFromRoot(projectRoot),
         projectRoot,
-        config: settings,
-        envFile,
+        config: _.cloneDeep(settings),
+        envFile: _.cloneDeep(envFile),
         options,
       })
     })

packages/server/test/e2e/7_record_spec.js

@@ -16,9 +16,6 @@ const {
   postInstanceTestsResponse,
 } = require('../support/helpers/serverStub')
 const { expectRunsToHaveCorrectTimings } = require('../support/helpers/resultsUtils')
-const postRunResponseWithWarnings = jsonSchemas.getExample('postRunResponse')('2.2.0')
-const postRunResponse = _.assign({}, postRunResponseWithWarnings, { warnings: [] })
-const postRunInstanceResponse = jsonSchemas.getExample('postRunInstanceResponse')('2.1.0')
 
 const e2ePath = Fixtures.projectPath('e2e')
 const outputPath = path.join(e2ePath, 'output.json')
@@ -691,14 +688,13 @@ describe('e2e record', () => {
     })
 
     describe('create run 500', () => {
-      const routes = [{
-        method: 'post',
-        url: '/runs',
-        req: '[email protected]',
-        res (req, res) {
-          return res.sendStatus(500)
+      const routes = createRoutes({
+        postRun: {
+          res (req, res) {
+            return res.sendStatus(500)
+          },
         },
-      }]
+      })
 
       setupStubbedServer(routes)
 

packages/server/test/unit/config_spec.js

@@ -56,8 +56,7 @@ describe('lib/config', () => {
 
       this.setup = (cypressJson = {}, cypressEnvJson = {}) => {
         sinon.stub(settings, 'read').withArgs(this.projectRoot).resolves(cypressJson)
-
-        return sinon.stub(settings, 'readEnv').withArgs(this.projectRoot).resolves(cypressEnvJson)
+        sinon.stub(settings, 'readEnv').withArgs(this.projectRoot).resolves(cypressEnvJson)
       }
     })
 
@@ -81,6 +80,19 @@ describe('lib/config', () => {
       })
     })
 
+    it('clones settings and env settings, so they are not mutated', function () {
+      const settings = { foo: 'bar' }
+      const envSettings = { baz: 'qux' }
+
+      this.setup(settings, envSettings)
+
+      return config.get(this.projectRoot)
+      .then(() => {
+        expect(settings).to.deep.equal({ foo: 'bar' })
+        expect(envSettings).to.deep.equal({ baz: 'qux' })
+      })
+    })
+
     context('port', () => {
       beforeEach(function () {
         return this.setup({}, { foo: 'bar' })