Merge pull request #76 from simonihmig/add-reporting-test

Add bin test for error reporting

Author: rwjblue

__tests__/bin-test.js

@@ -7,9 +7,6 @@ const tmp = require('tmp');
 
 const originalCwd = process.cwd();
 
-const inputFile = path.join(originalCwd, '__testfixtures__/final-boss.input.js');
-const outputFile = path.join(originalCwd, '__testfixtures__/final-boss.output.js');
-
 let tmpPath;
 
 function run() {
@@ -95,6 +92,9 @@ describe('bin acceptance', function() {
 
     describe('with valid file', function() {
       let tmpFile;
+      const inputFile = path.join(originalCwd, '__testfixtures__/final-boss.input.js');
+      const outputFile = path.join(originalCwd, '__testfixtures__/final-boss.output.js');
+
 
       beforeEach(function() {
         fs.ensureDirSync(path.join(tmpPath, 'app'));
@@ -120,5 +120,37 @@ describe('bin acceptance', function() {
         });
       });
     });
+
+    describe('with invalid file', function() {
+      const inputFile = path.join(originalCwd, '__testfixtures__/leave-unmatched-destructuring.input.js');
+
+      beforeEach(function() {
+        fs.ensureDirSync(path.join(tmpPath, 'app'));
+
+        let tmpFile = path.join(tmpPath, 'app/leave-unmatched-destructuring.input.js');
+
+        fs.copySync(
+          inputFile,
+          tmpFile
+        );
+      });
+
+      it('reports errors', function() {
+        return run().then(result => {
+          let exitCode = result.exitCode;
+          let stdout = result.stdout;
+
+          expect(exitCode).toEqual(0);
+
+          expect(stdout).toMatch('Done! Some files could not be upgraded automatically. See MODULE_REPORT.md.\n');
+
+          let expectedOutput = fs.readFileSync(path.join(__dirname, 'expected', 'MODULE_REPORT.md'), 'utf8')
+            .replace(/\//, path.sep);
+          let actualOutput = fs.readFileSync(path.join(tmpPath, 'MODULE_REPORT.md'), 'utf8');
+
+          expect(actualOutput).toEqual(expectedOutput);
+        });
+      });
+    });
   });
 });

__tests__/expected/MODULE_REPORT.md

@@ -0,0 +1,14 @@
+## Module Report
+### Unknown Global
+
+**Global**: `Ember.String.pluralize`
+
+**Location**: `app/leave-unmatched-destructuring.input.js` at line 5
+
+```js
+const {
+  String: {
+    pluralize,
+    camelize
+  }
+```