Remove capability to load cookies from file

lighthouse-cli/bin.js

@@ -128,13 +128,11 @@ async function begin() {
     // the conversion here, but long term either the CLI flag or the setting should have
     // a different name.
     // @ts-ignore
-    let extraCookiesStr = /** @type {string} */ (cliFlags.extraCookies);
-    // If not a JSON array, assume it's a path to a JSON file.
-    if (extraCookiesStr.substr(0, 1) !== '[') {
-      extraCookiesStr = fs.readFileSync(extraCookiesStr, 'utf-8');
-    }
-
+    const extraCookiesStr = /** @type {string} */ (cliFlags.extraCookies);
     cliFlags.extraCookies = JSON.parse(extraCookiesStr);
+    if (!Array.isArray(cliFlags.extraCookies)) {
+      throw new Error('extraCookies parameter must be a valid JSON array');
+    }
   }
 
   if (cliFlags.precomputedLanternDataPath) {

lighthouse-cli/test/cli/bin-test.js

@@ -179,13 +179,10 @@ describe('CLI bin', function() {
       expect(getRunLighthouseArgs()[1]).toHaveProperty('extraCookies', [{'name': 'foo', 'value': 'bar', 'url': 'http://localhost'}]);
     });
 
-    it('should read extra cookies from file', async () => {
-      const headersFile = require.resolve('../fixtures/extra-cookies/valid.json');
+    it('should throw when invalid array is used', async () => {
       // @ts-ignore - see TODO: in bin.js
-      cliFlags = {...cliFlags, extraCookies: headersFile};
-      await bin.begin();
-
-      expect(getRunLighthouseArgs()[1]).toHaveProperty('extraCookies', require(headersFile));
+      cliFlags = {...cliFlags, extraCookies: 'INVALID_JSON_ARRAY'};
+      await expect(bin.begin()).rejects.toBeTruthy();
     });
   });
 

lighthouse-cli/test/cli/index-test.js

@@ -59,15 +59,6 @@ describe('CLI Tests', function() {
       assert.equal(ret.status, 1);
     });
 
-    it('should exit with a error if the file does not contain valid JSON', () => {
-      const ret = spawnSync('node', [indexPath, 'https://www.google.com',
-        '--extra-cookies',
-        path.resolve(__dirname, '../fixtures/extra-cookies/invalid.txt')], {encoding: 'utf8'});
-
-      assert.ok(ret.stderr.includes('Unexpected token'));
-      assert.equal(ret.status, 1);
-    });
-
     it('should exit with a error if the passsed in string is not valid JSON', () => {
       const ret = spawnSync('node', [indexPath, 'https://www.google.com',
         '--extra-headers', '{notjson}'], {encoding: 'utf8'});