Merge pull request #79 from brodybits/0.9.1-patch-updates

0.9.1 patch updates

Author: Chris Brody

.eslintignore

@@ -0,0 +1,2 @@
+# Stryker mutator output
+reports

.gitignore

@@ -27,3 +27,8 @@ Temporary Items
 
 node_modules
 npm-debug.log
+
+# Stryker mutator output
+.stryker-tmp
+reports
+stryker.log

README.md

@@ -121,7 +121,7 @@ createLibraryModule({
   prefix: String, /* The prefix for the library (Default: ``) */
   moduleName: String, /* The module library package name to be used in package.json. Default: react-native-(name in param-case) */
   modulePrefix: String, /* The module prefix for the library, ignored if moduleName is specified (Default: react-native) */
-  platforms: Array, /* Platforms the library will be created for. (Default: ['ios', 'android']) */
+  platforms: Array | String, /* Platforms the library will be created for. (Default: ['android', 'ios']) */
   packageIdentifier: String, /* (Android only!) The package name for the Android module (Default: com.reactlibrary) */
   githubAccount: String, /* The github account where the library is hosted (Default: `github_account`) */
   authorName: String, /* The author's name (Default: `Your Name`) */

lib/cli-program.js

@@ -24,6 +24,9 @@ program
 
 program.parse(process.argv);
 
-if (!program.args.length) {
-  program.help();
-}
+// TBD non-functional error handling code
+// FUTURE TODO: handle missing argument error in the right place
+// (see issue #48)
+// if (!program.args.length) {
+//   program.help();
+// }

lib/lib.js

@@ -139,9 +139,6 @@ const generateWithOptions = ({
 
         return true;
       }).map((template) => {
-        if (!template.name) {
-          return Promise.resolve();
-        }
         const templateArgs = {
           name: className,
           moduleName,

lib/utils/npmAddScriptSync.js

@@ -7,9 +7,6 @@ module.exports = (packageJsonPath, script, fs) => {
   try {
     var packageJson = jsonfile.readFileSync(packageJsonPath, { fs });
     if (!packageJson.scripts) packageJson.scripts = {};
-    if (!script.force && packageJson.scripts[script.key]) {
-      throw new Error(`That script entry for key: ${script.key} already exists.`);
-    }
     packageJson.scripts[script.key] = script.value;
     jsonfile.writeFileSync(packageJsonPath, packageJson, { fs, spaces: 2 });
   } catch (e) {

package.json

@@ -7,6 +7,7 @@
   "scripts": {
     "jest": "jest",
     "lint": "eslint .",
+    "run-stryker": "stryker run",
     "test": "npm run lint && npm run jest"
   },
   "repository": {
@@ -50,6 +51,10 @@
     "uuid": "^3.3.3"
   },
   "devDependencies": {
+    "@stryker-mutator/core": "^2.0.2",
+    "@stryker-mutator/html-reporter": "^2.0.2",
+    "@stryker-mutator/javascript-mutator": "^2.0.2",
+    "@stryker-mutator/jest-runner": "^2.0.2",
     "eslint": "^6.3.0",
     "eslint-config-standard": "^14.1.0",
     "eslint-plugin-import": "^2.18.2",

stryker.conf.js

@@ -0,0 +1,10 @@
+module.exports = config => {
+  config.set({
+    mutator: 'javascript',
+    packageManager: 'yarn',
+    reporters: ['html', 'clear-text', 'progress'],
+    testRunner: 'jest',
+    transpilers: [],
+    coverageAnalysis: 'off'
+  });
+};

tests/lib-cli-command/object/__snapshots__/lib-cli-command-object-text.test.js.snap renamed to tests/with-injection/cli/command/object/__snapshots__/lib-cli-command-object-text.test.js.snap

@@ -1,4 +1,4 @@
-const command = require('../../../lib/cli-command.js');
+const command = require('../../../../../lib/cli-command.js');
 
 test('cli-command object', () => {
   expect(command).toMatchSnapshot();