Merge pull request #70 from ddoria921/fix-displaying-invalid-date

Convert timestamp to JS Date object

Author: lukemelia

lib/legacy-table.js

@@ -39,7 +39,7 @@ module.exports = CoreObject.extend({
 
       if(key.name === 'timestamp') {
         // ember-cli-deploy-revision-data provides a JS Date object, so fall back to that if not milliseconds
-        let dt = typeof value === 'number' ? DateTime.fromMillis(value) : DateTime.fromJSDate(value);
+        let dt = typeof value === 'number' ? DateTime.fromMillis(value) : DateTime.fromJSDate(new Date(value));
         value = dt.toFormat("yyyy/MM/dd HH:mm:ss");
       }
 

lib/scm-table.js

@@ -72,7 +72,7 @@ module.exports = CoreObject.extend({
       if (this._isWide()) {
         let { timestamp } = data;
         // ember-cli-deploy-revision-data provides a JS Date object, so fall back to that if not milliseconds
-        let dt = typeof timestamp === 'number' ? DateTime.fromMillis(timestamp) : DateTime.fromJSDate(timestamp);
+        let dt = typeof timestamp === 'number' ? DateTime.fromMillis(timestamp) : DateTime.fromJSDate(new Date(timestamp));
         let value = dt.toFormat('yyyy/MM/dd HH:mm:ss');
         row.push(value);
       }

tests/index-test.js

@@ -164,8 +164,8 @@ describe('displayRevisions plugin', function() {
         },
         revisions: [
           { revision: "rev:abcdef", timestamp: 1438232435000, deployer: "My Hamster"},
-          { revision: "rev:defghi", timestamp: 1032123125000, deployer: "My Hamster", active: true},
-          { revision: "rev:jklmno", timestamp: 1032123128000, deployer: "My Hamster"},
+          { revision: "rev:defghi", timestamp: '2002-09-15T20:52:05.000Z', deployer: "My Hamster", active: true},
+          { revision: "rev:jklmno", timestamp: new Date(1032123128000), deployer: "My Hamster"},
           { revision: "rev:qrstuv", timestamp: 1032123155000, deployer: "My Hamster"},
           { revision: "rev:xyz123", timestamp: 1032123123000, deployer: "My Hamster"}
         ]
@@ -198,7 +198,7 @@ describe('displayRevisions plugin', function() {
       assert.equal(messages.length, 0);
     });
 
-    it('transforms timestamps to human-readable dates (yyyy/MM/dd HH:mm:ss)', function() {
+    it('transforms millisecond timestamps to human-readable dates (yyyy/MM/dd HH:mm:ss)', function() {
       plugin.displayRevisions(context);
       let expectedFormat = ('yyyy/MM/dd HH:mm:ss');
       let expectedDate   = DateTime.fromMillis(1438232435000).toFormat(expectedFormat);
@@ -213,6 +213,36 @@ describe('displayRevisions plugin', function() {
       assert.equal(messages.length, 1);
     });
 
+    it('transforms ISO string timestamps to human-readable dates (yyyy/MM/dd HH:mm:ss)', function() {
+      plugin.displayRevisions(context);
+      let expectedFormat = ('yyyy/MM/dd HH:mm:ss');
+      let expectedDate   = DateTime.fromISO('2002-09-15T20:52:05.000Z').toFormat(expectedFormat);
+
+      let messages = mockUi.messages.reduce(function(previous, current) {
+        if (current.indexOf(expectedDate) !== -1) {
+          previous.push(current);
+        }
+
+        return previous;
+      }, []);
+      assert.equal(messages.length, 1);
+    });
+
+    it('transforms JS Date object timestamps to human-readable dates (yyyy/MM/dd HH:mm:ss)', function() {
+      plugin.displayRevisions(context);
+      let expectedFormat = ('yyyy/MM/dd HH:mm:ss');
+      let expectedDate   = DateTime.fromJSDate(new Date(1032123128000)).toFormat(expectedFormat);
+
+      let messages = mockUi.messages.reduce(function(previous, current) {
+        if (current.indexOf(expectedDate) !== -1) {
+          previous.push(current);
+        }
+
+        return previous;
+      }, []);
+      assert.equal(messages.length, 1);
+    });
+
     it('highlights the active revision', function() {
       plugin.displayRevisions(context);
       let messages = mockUi.messages.reduce(function(previous, current) {