refactor: switch to latest ShellJS plugin API

Author: nfischer

index.js

@@ -1,5 +1,5 @@
 // This exposes the plugin utilities
-var plugin = require('shelljs/src/common');
+var plugin = require('shelljs/plugin');
 
 // Require whatever modules you need for your project
 var child = require('child_process');

package.json

@@ -31,7 +31,7 @@
     "eslint-config-airbnb-base": "^4.0.2",
     "eslint-plugin-import": "^1.11.1",
     "mocha": "^2.5.3",
-    "shelljs": "0.7.2",
+    "shelljs": "github:shelljs/shelljs",
     "shelljs-changelog": "^0.2.2",
     "shelljs-release": "^0.2.0",
     "should": "^10.0.0"

test/test.js

@@ -1,4 +1,4 @@
-/* globals describe, it */
+/* globals describe, it, open */
 
 /*
  * Using ShellJS Plugins:
@@ -85,8 +85,7 @@ describe('plugin-open', function () {
     ret.stdout.should.equal('');
     var errorMsg = 'open: option not recognized: f';
     ret.stderr.should.equal(errorMsg);
-    // TODO(nate): refactor ShellJS to support this
-    // shell.error().should.equal(errorMsg);
+    shell.error().should.equal(errorMsg);
   });
 
   it('cannot open files that are missing', function () {
@@ -95,8 +94,12 @@ describe('plugin-open', function () {
     ret.stdout.should.equal('');
     var errorMsg = 'open: Unable to locate file: missingFile.txt';
     ret.stderr.should.equal(errorMsg);
-    // TODO(nate): refactor ShellJS to support this
-    // shell.error().should.equal(errorMsg);
+    shell.error().should.equal(errorMsg);
+    ret = open('missingFile.txt');
+    ret.code.should.equal(1);
+    ret.stdout.should.equal('');
+    ret.stderr.should.equal(errorMsg);
+    shell.error().should.equal(errorMsg);
   });
 
   it('opens URLs', function () {