feat(ES5 support): remove ES5 mutator

package.json

@@ -4,9 +4,6 @@
   "devDependencies": {
     "@types/chai": "4.1.7",
     "@types/chai-as-promised": "7.1.0",
-    "@types/escodegen": "0.0.6",
-    "@types/esprima": "^2.1.31",
-    "@types/estree": "0.0.38",
     "@types/execa": "^0.9.0",
     "@types/glob": "^7.1.0",
     "@types/istanbul": "^0.4.29",

packages/stryker-api/src/config/Config.ts

@@ -22,7 +22,7 @@ export default class Config implements StrykerOptions {
   public coverageAnalysis: 'perTest' | 'all' | 'off' = 'off';
   public testRunner: string = 'command';
   public testFramework: string;
-  public mutator: string | MutatorDescriptor = 'es5';
+  public mutator: string | MutatorDescriptor = 'javascript';
   public transpilers: string[] = [];
   public maxConcurrentTestRunners: number = Infinity;
   public symlinkNodeModules: boolean = true;

packages/stryker-vue-mutator/src/helpers/MutatorHelpers.ts

@@ -8,7 +8,7 @@ export function mutatorsFactory(pluginResolver: PluginResolver, injector: Inject
   const mutators: { [name: string]: Mutator; } = {};
   const mutatorPlugins = pluginResolver.resolveAll(PluginKind.Mutator);
   mutatorPlugins.forEach(plugin => {
-    if (plugin.name !== 'es5' && plugin.name !== 'vue') {
+    if (plugin.name !== 'vue') {
       mutators[plugin.name] = createPlugin(injector, plugin);
     }
   });

packages/stryker/README.md

@@ -152,13 +152,13 @@ In addition to requiring your test runner to be able to report the code coverage
  (`Mocha` is not yet supported).
 
 ### `mutator` [`object` | `string`]
-Default: `es5`  
-Command line: `--mutator es5`  
-Config file:  `mutator: 'es5'` or `mutator: { name: 'es5', excludedMutations: ['BooleanSubstitution', 'StringLiteral'] }`  
+Default: `javascript`  
+Command line: `--mutator javascript`  
+Config file:  `mutator: 'javascript'` or `mutator: { name: 'javascript', excludedMutations: ['BooleanSubstitution', 'StringLiteral'] }`  
 
 With `mutator` you configure which mutator plugin you want to use, and optionally, which mutation types to exclude from the test run.  
-The mutator plugin name defaults to `es5` if not specified. The list of excluded mutation types defaults to an empty array, meaning all mutation types will be included in the test.  
-The full list of mutation types varies slightly between mutators (for example, the `es5` mutator will not use the same mutation types as the `typescript` mutator). Mutation type names are case-sensitive, and can be found either in the source code or in a generated Stryker report.  
+The mutator plugin name defaults to `javascript` if not specified. Note: this requires you to have the `@stryker-mutator/javascript-mutator` plugin installed. The list of excluded mutation types defaults to an empty array, meaning all mutation types will be included in the test.  
+The full list of mutation types varies slightly between mutators (for example, the `javascript` mutator will not use the same mutation types as the `typescript` mutator). Mutation type names are case-sensitive, and can be found either in the source code or in a generated Stryker report.  
 
 When using the command line, only the mutator name as a string may be provided.  
 When using the config file, you can provide either a string representing the mutator name, or a `MutatorDescriptor` object, like so:  

packages/stryker/package.json

@@ -55,8 +55,6 @@
     "@stryker-mutator/util": "^0.1.0",
     "chalk": "~2.4.1",
     "commander": "~2.19.0",
-    "escodegen": "~1.8.0",
-    "esprima": "~2.7.0",
     "get-port": "~4.1.0",
     "glob": "~7.1.2",
     "inquirer": "~6.2.0",

packages/stryker/src/di/buildMainInjector.ts

@@ -45,8 +45,7 @@ export function buildMainInjector(cliOptions: Partial<StrykerOptions>): Injector
 
 function pluginDescriptorsFactory(config: Config): ReadonlyArray<string> {
   config.plugins.push(
-    require.resolve('../reporters'),
-    require.resolve('../mutators')
+    require.resolve('../reporters')
   );
   return config.plugins;
 }