Merge pull request #38 from rwjblue/add-simple-logging

Add some rudimentary console output.

Author: rwjblue

bin/ember-module-migrator

@@ -2,7 +2,7 @@
 'use strict';
 
 // Local Variables:
-// mode: javascript
+// mode: js2
 // End:
 
 // Provide a title to the process in `ps`
@@ -14,16 +14,19 @@ var nopt = require("nopt");
 var options = {
   'project-root': [String],
   'engine': [String, 'classic']
-}
+};
 var parsed = nopt(options);
 
 var migrator = new Migrator({
   projectRoot: parsed['project-root'] || process.cwd(),
-  engine: parsed['engine']
+  engine: parsed['engine'],
+  verbose: true
 });
 
 migrator.processFiles()
   .then(function() {
+    /* eslint no-console: 0 */
+
     console.log('Finished successfully!');
   })
   .catch(function(error) {

lib/index.js

@@ -5,6 +5,7 @@ var move = RSVP.denodeify(fse.move);
 var CoreObject = require('core-object');
 var engines = require('./engines');
 var FileInfoCollection = require('./models/file-info-collection');
+var Logger = require('./models/logger');
 
 var Engine = CoreObject.extend({
   init: function() {
@@ -26,13 +27,21 @@ var Engine = CoreObject.extend({
 
     this._promise = null;
     this._fileInfoCollection = new FileInfoCollection();
+    this._logger = new Logger({
+      projectRoot: this.projectRoot
+    });
     this._fileInfos = [];
   },
 
   _queueMoveFile: function(source, dest) {
+    var logger = this._logger;
+
     this._promise = this._promise
       .then(function() {
         return move(source, dest);
+      })
+      .then(function() {
+        logger.movedFile(source, dest);
       });
   },
 
@@ -90,6 +99,8 @@ var Engine = CoreObject.extend({
       this._filesInDir('tests')
     );
 
+    var verbose = this.verbose;
+    var logger = this._logger;
     var projectRoot = this.projectRoot;
     var i;
 
@@ -116,7 +127,13 @@ var Engine = CoreObject.extend({
     this._queueRemoveEmptyDirs('app');
     this._queueRemoveEmptyDirs('tests');
 
-    return this._promise;
+    return this._promise
+      .then(function() {
+        /*eslint no-console: 0 */
+        if (verbose) {
+          console.log(logger.flush());
+        }
+      });
   },
 
   fileInfoFor: function(path) {

lib/models/logger.js

@@ -0,0 +1,37 @@
+var path = require('path');
+var chalk = require('chalk');
+var CoreObject = require('core-object');
+var Table = require('cli-table2');
+
+var Logger = CoreObject.extend({
+  init: function() {
+    this._movedFiles = {};
+    this._renamedRenderables = [];
+  },
+
+  movedFile: function(_source, _destination) {
+    var source = path.relative(this.projectRoot, _source);
+    var destination = path.relative(this.projectRoot, _destination);
+
+    this._movedFiles[source] = destination;
+  },
+
+  flush: function() {
+    var maxWidth = process.stdout.columns - 4;
+    var table = new Table({
+      head: ['Source', 'Destination'],
+      colWidths: [maxWidth / 2, maxWidth / 2]
+    });
+
+
+    for (var source in this._movedFiles) {
+      var destination = this._movedFiles[source];
+
+      table.push([chalk.cyan(source), chalk.yellow(destination)]);
+    }
+
+    return table.toString();
+  }
+});
+
+module.exports = Logger;

package.json

@@ -23,6 +23,8 @@
     "rimraf": "^2.5.2"
   },
   "dependencies": {
+    "chalk": "^1.1.3",
+    "cli-table2": "^0.2.0",
     "core-object": "^2.0.0",
     "exists-sync": "0.0.3",
     "fs-extra": "^0.28.0",

test/logger-test.js

@@ -0,0 +1,36 @@
+var path = require('path');
+var assert = require('power-assert');
+var Logger = require('../lib/models/logger');
+
+describe('logger', function() {
+  var logger;
+
+  beforeEach(function() {
+    logger = new Logger({
+      projectRoot: __dirname
+    });
+  });
+
+  it('tracks moved files', function() {
+    var source = 'app/helpers/titleize.js';
+    var dest = 'src/ui/components/titleize.js';
+
+    logger.movedFile(source, dest);
+
+    var output = logger.flush();
+
+    assert(output.indexOf(source) > -1);
+    assert(output.indexOf(dest) > -1);
+  });
+
+  it('uses paths relative to root', function() {
+    var source = path.resolve(__dirname, 'app/helpers/titleize.js');
+    var dest = path.resolve(__dirname, 'src/ui/components/titleize.js');
+
+    logger.movedFile(source, dest);
+
+    var output = logger.flush();
+
+    assert(output.indexOf(__dirname) === -1);
+  });
+});