test(package): fix coverage for lib/package.js

lob · Jun 5, 2017 · 006a2a0 · 006a2a0
1 parent fc31e9a
commit 006a2a0
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 6 deletions.
diff --git a/lib/package.js b/lib/package.js
@@ -49,6 +49,7 @@ exports.extractRepoUrl = function () {
  * @returns {Promise<String>} - new version
  */
 exports.calculateNewVersion = function (options) {
+  options = options || {};
   return exports.getUserPackage()
   .then(function (userPackage) {
     var split = userPackage.version.split('.');

diff --git a/test/package.test.js b/test/package.test.js
@@ -114,12 +114,12 @@ describe('package', function () {
       Package.getUserPackage.restore();
     });
 
-    it('bumps the patch version if patch is true', function () {
-      var options = { patch: true };
+    it('bumps the major version if major is true', function () {
+      var options = { major: true };
 
       return Package.calculateNewVersion(options)
       .then(function (version) {
-        Expect(version).to.eql('1.2.4');
+        Expect(version).to.eql('2.0.0');
       });
     });
 
@@ -132,15 +132,21 @@ describe('package', function () {
       });
     });
 
-    it('bumps the major version if major is true', function () {
-      var options = { major: true };
+    it('bumps the patch version if patch is true', function () {
+      var options = { patch: true };
 
       return Package.calculateNewVersion(options)
       .then(function (version) {
-        Expect(version).to.eql('2.0.0');
+        Expect(version).to.eql('1.2.4');
       });
     });
 
+    it('leaves the version untouched if none of three options is true', function () {
+      return Package.calculateNewVersion()
+      .then(function (version) {
+        Expect(version).to.eql('1.2.3');
+      });
+    });
   });
 
 });