fix(cli): Make unknown flag check a warning instead of an error. (#4028)

Author: heathkit

lib/runner.ts

@@ -93,10 +93,11 @@ export class Runner extends EventEmitter {
       unknownFlags = unknownFlags.filter((f) => extraFlags.indexOf(f) === -1);
     }
     if (unknownFlags.length > 0 && !this.config_.disableChecks) {
-      throw new ConfigError(
-          logger,
-          'Found extra flags: ' + unknownFlags.join(', ') +
-              ', please use --disableChecks flag to disable the Protractor CLI flag checks. ');
+      // TODO: Make this throw a ConfigError in Protractor 6.
+      logger.warn(
+          'Ignoring unknown extra flags: ' + unknownFlags.join(', ') + '. This will be' +
+          ' an error in future versions, please use --disableChecks flag to disable the ' +
+          ' Protractor CLI flag checks. ');
     }
     return this.plugins_.onPrepare().then(() => {
       return helper.runFilenameOrFn_(this.config_.configDir, this.preparer_);

scripts/errorTest.js

@@ -19,12 +19,6 @@ var checkLogs = function(output, messages) {
  *Below are exit failure tests*
  ******************************/
 
-runProtractor = spawn('node',
-    ['bin/protractor', 'example/conf.js', '--foobar', 'foobar']);
-output = runProtractor.stdout.toString();
-messages = ['Error: Found extra flags: foobar'];
-checkLogs(output, messages);
-
 // assert authentication error for sauce labs
 runProtractor = spawn('node',
     ['bin/protractor', 'spec/errorTest/sauceLabsAuthentication.js']);