Merge pull request #9 from nkbt/exit-code

fix(spawn): Propagate spawned process exit code

src/index.js

@@ -7,7 +7,9 @@ const envSetterRegex = /(\w+)=('(.+)'|"(.+)"|(.+))/;
 function crossEnv(args) {
   const [command, commandArgs, env] = getCommandArgsAndEnvVars(args);
   if (command) {
-    return spawn(command, commandArgs, {stdio: 'inherit', env});
+    const proc = spawn(command, commandArgs, {stdio: 'inherit', env});
+    proc.on('exit', process.exit);
+    return proc;
   }
 }
 

src/index.test.js

@@ -7,9 +7,10 @@ import assign from 'lodash.assign';
 chai.use(sinonChai);
 
 const {expect} = chai;
+const spawned = {on: sinon.spy()};
 const proxied = {
   'cross-spawn-async': {
-    spawn: sinon.spy(() => 'spawn-returned')
+    spawn: sinon.spy(() => spawned)
   }
 };
 
@@ -19,6 +20,7 @@ describe(`cross-env`, () => {
 
   beforeEach(() => {
     proxied['cross-spawn-async'].spawn.reset();
+    spawned.on.reset();
   });
 
   it(`should set environment variables and run the remaining command`, () => {
@@ -69,13 +71,16 @@ describe(`cross-env`, () => {
     env.APPDATA = process.env.APPDATA;
     assign(env, expected);
 
-    expect(ret, 'returns what spawn returns').to.equal('spawn-returned');
+    expect(ret, 'returns what spawn returns').to.equal(spawned);
     expect(proxied['cross-spawn-async'].spawn).to.have.been.calledOnce;
     expect(proxied['cross-spawn-async'].spawn).to.have.been.calledWith(
       'echo', ['hello world'], {
         stdio: 'inherit',
         env: assign({}, process.env, env)
       }
     );
+
+    expect(spawned.on).to.have.been.calledOnce;
+    expect(spawned.on).to.have.been.calledWith('exit');
   }
 });