Merge pull request #68 from kellyselden/exits-gracefully

add test for exiting gracefully

Author: Kelly Selden

__tests__/bin-test.js

@@ -10,22 +10,12 @@ const outputFile = path.join(process.cwd(), '__testfixtures__/final-boss.output.
 
 describe('bin acceptance', function() {
   let tmpPath;
-  let tmpFile;
   let tmpPackageJson;
 
   beforeEach(function() {
     tmpPath = tmp.dirSync().name;
 
     tmpPackageJson = path.join(tmpPath, 'package.json');
-
-    fs.ensureDirSync(path.join(tmpPath, 'app'));
-
-    tmpFile = path.join(tmpPath, 'app/final-boss.js');
-
-    fs.copySync(
-      path.join(process.cwd(), '__testfixtures__/final-boss.input.js'),
-      tmpFile
-    );
   });
 
   it('handles non-ember projects', function() {
@@ -64,15 +54,63 @@ describe('bin acceptance', function() {
       });
     });
 
-    it('works', function() {
-      cp.spawnSync('node', [
-        path.join(process.cwd(), 'bin/ember-modules-codemod')
-      ], {
-        cwd: tmpPath,
-        stdio: 'inherit'
+    it('exits gracefully when no files found', function() {
+      let stderr = '';
+      let exitCode;
+
+      return new Promise(resolve => {
+        let ps = cp.spawn('node', [
+          path.join(process.cwd(), 'bin/ember-modules-codemod')
+        ], {
+          cwd: tmpPath
+        });
+
+        ps.stderr.on('data', data => {
+          stderr += data.toString();
+        });
+
+        ps.on('exit', (code, signal) => {
+          exitCode = code;
+
+          resolve();
+        });
+      }).then(() => {
+        expect(exitCode).toEqual(0);
+
+        // jscodeshift can process in any order
+        expect(stderr).toMatch('Skipping path app which does not exist.');
+        expect(stderr).toMatch('Skipping path addon which does not exist.');
+        expect(stderr).toMatch('Skipping path addon-test-support which does not exist.');
+        expect(stderr).toMatch('Skipping path tests which does not exist.');
+        expect(stderr).toMatch('Skipping path test-support which does not exist.');
+        expect(stderr).toMatch('Skipping path lib which does not exist.');
       });
+    });
+
+    describe('with valid file', function() {
+      let tmpFile;
 
-      expect(fs.readFileSync(tmpFile, 'utf8')).toEqual(fs.readFileSync(outputFile, 'utf8'));
+      beforeEach(function() {
+        fs.ensureDirSync(path.join(tmpPath, 'app'));
+
+        tmpFile = path.join(tmpPath, 'app/final-boss.js');
+
+        fs.copySync(
+          path.join(process.cwd(), '__testfixtures__/final-boss.input.js'),
+          tmpFile
+        );
+      });
+
+      it('works', function() {
+        cp.spawnSync('node', [
+          path.join(process.cwd(), 'bin/ember-modules-codemod')
+        ], {
+          cwd: tmpPath,
+          stdio: 'inherit'
+        });
+
+        expect(fs.readFileSync(tmpFile, 'utf8')).toEqual(fs.readFileSync(outputFile, 'utf8'));
+      });
     });
   });
 });