feat: add update notifier to cli

hyperlink · Feb 10, 2017 · b6ff85a · b6ff85a
1 parent ad3d5d2
commit b6ff85a
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -2,6 +2,11 @@
 
 'use strict'
 
+const updateNotifier = require('update-notifier')
+const pkg = require('./package')
+
+updateNotifier({pkg}).notify()
+
 if (!process.env.DIGITAL_OCEAN_TOKEN || !process.env.SERVER_NAME) {
   console.error('Missing required environment variables: DIGITAL_OCEAN_TOKEN or SERVER_NAME')
 } else {

diff --git a/package.json b/package.json
@@ -25,7 +25,8 @@
     "bluebird": "^3.4.6",
     "digitalocean": "^0.9.1",
     "lodash": "^4.15.0",
-    "public-ip": "^2.1.0"
+    "public-ip": "^2.1.0",
+    "update-notifier": "^1.0.3"
   },
   "devDependencies": {
     "mocha": "^3.2.0",

diff --git a/test/test.ddns.js b/test/test.ddns.js
@@ -157,4 +157,15 @@ describe('Dynamic DNS Client', function () {
     require('../')
     sinon.assert.calledWithExactly(console.error, 'Missing required environment variables: DIGITAL_OCEAN_TOKEN or SERVER_NAME')
   })
+
+  it('should call updateNotifier notify', function () {
+    const notify = sandbox.stub()
+    const updateNotifier = sandbox.stub().returns({ notify })
+    proxyquire('../', {
+      'update-notifier': updateNotifier
+    })
+    sinon.assert.calledWith(updateNotifier, sinon.match.object)
+    sinon.assert.calledOnce(notify)
+    notify.calledAfter(updateNotifier)
+  })
 })