node-notifier - npm Package Compare versions

Comparing version 3.0.1 to 3.0.2

lib/utils.js

@@ -1,3 +0,2 @@
-var child_process = require('child_process')
-  , exec = child_process.exec
+var cp = require('child_process')
   , osVersionError = 'Incorrect OS. node-notify requires Mac OS 10.8 or higher'
@@ -51,2 +50,6 @@ , shellwords = require('shellwords');
 
+module.exports.getOSXVersion = function (cb) {
+  return cp.exec("sw_vers -productVersion", cb);
+}
+
 module.exports.isMacOSX = function (cb) {
@@ -56,8 +59,7 @@ if (process.platform != 'darwin') {
   }
-
-  return exec("sw_vers -productVersion", function (error, stdout, stderr) {
+  return module.exports.getOSXVersion(function (error, stdout, stderr) {
     if (error) {
       return cb(true, error, stderr);
     }
-    if (stdout.split(".")[1] >= "8") {
+    if (Number(stdout.split(".")[1]) >= 8) {
       return cb(false);
@@ -64,0 +66,0 @@ }

package.json

 {
   "name": "node-notifier",
-  "version": "3.0.1",
+  "version": "3.0.2",
   "description": "A Node.js module for sending notifications on mac, windows and linux",
@@ -5,0 +5,0 @@ "main": "index.js",

README.md

@@ -186,2 +186,5 @@ # node-notifier [![NPM version][npm-image]][npm-url] [![Build Status][travis-image]][travis-url] [![Dependency Status][depstat-image]][depstat-url]
 
+### `v3.0.1`
+1. Fixes version check for Mac OS X Yosemite
+
 ### `v3.0.0`
@@ -188,0 +191,0 @@ 1. Updates terminal-notifier to version 1.6.0; adding support for appIcon and contentImage

test/terminal-notifier.js

@@ -10,2 +10,4 @@ var NotificationCenter = require('../').NotificationCenter
 var originalUtils = utils.command;
+var originalIsMacOsX = utils.isMacOSX;
+var originalType = os.type;
 
@@ -24,4 +26,9 @@ (function () {
       };
-    })
+    });
 
+    after(function () {
+      os.type = originalType;
+      utils.isMacOSX = originalIsMacOsX;
+    });
+
     describe('#notify()', function(){
@@ -28,0 +35,0 @@

No alert changes

Improved metrics

Total package byte prevSize

492998

increased by0.33%

Number of package files

24

increased by4.35%

Lines of code

552

increased by11.97%

Number of lines in readme file

204

increased by1.49%

Worsened metrics

Number of medium supply chain risk alerts

2

increased by100%

No dependency changes