code cleanup

Author: JamesMessinger

src/cli/parse-args.ts

@@ -28,7 +28,7 @@ export function parseArgs(argv: string[]): ParsedArgs {
         { name: "quiet", alias: "q", type: Boolean },
         { name: "version", alias: "v", type: Boolean },
         { name: "help", alias: "h", type: Boolean },
-        { name: "dry-run", type: Boolean, defaultOption: false }
+        { name: "dry-run", type: Boolean }
       ],
       { argv }
     );

test/specs/action/success.spec.js

@@ -268,7 +268,6 @@ describe("GitHub Action - success tests", () => {
   it("should not publish a package if dryRun is true", () => {
     files.create([
       { path: "workspace/package.json", contents: { name: "my-lib", version: "1.1.0" }},
-      { path: "home/.npmrc", contents: "This is my NPM config.\nThere are many like it,\nbut this one is mine." },
     ]);
 
     npm.mock({

test/specs/cli/args.spec.js

@@ -140,11 +140,4 @@ describe("CLI - argument tests", () => {
     });
   });
 
-
-  describe("npm-publish --dry-run", () => {
-    it("should call npm publish with the --dry-run flag", () => {
-      let cli = exec.cli("--dry-run");
-      expect(cli.args).to.include("--dry-run");
-    });
-  });
 });

test/specs/lib/success.spec.js

@@ -52,7 +52,7 @@ describe("NPM package - success tests", () => {
       package: "my-lib",
       version: "2.0.0",
       oldVersion: "1.0.0",
-      dryRun: false
+      dryRun: false,
     });
 
     files.assert.contents("home/.npmrc",
@@ -85,7 +85,7 @@ describe("NPM package - success tests", () => {
       package: "my-lib",
       version: "1.0.0",
       oldVersion: "1.0.0",
-      dryRun: false
+      dryRun: false,
     });
 
     files.assert.contents("home/.npmrc",
@@ -132,7 +132,7 @@ describe("NPM package - success tests", () => {
       package: "my-lib",
       version: "1.0.0-beta.1",
       oldVersion: "1.0.0",
-      dryRun: false
+      dryRun: false,
     });
 
     files.assert.contents("home/.npmrc",
@@ -290,7 +290,7 @@ describe("NPM package - success tests", () => {
       package: "my-lib",
       version: "1.0.0-beta",
       oldVersion: "1.0.0",
-      dryRun: false
+      dryRun: false,
     });
 
     files.assert.contents("home/.npmrc",