refactor getHumanDate() to accept a timestamp; fixes test

This commit is contained in:
Max Leiter 2018-07-21 14:27:53 -07:00
parent 5b2de64af8
commit 9bee3eca5a
3 changed files with 4 additions and 4 deletions

View file

@ -32,7 +32,7 @@ module.exports = {
read(options, callback); read(options, callback);
}, },
getHumanDate() { getHumanDate(ts) {
return moment().format("YYYY-MM-DD HH:mm:ss"); return moment(ts).format("YYYY-MM-DD HH:mm:ss");
}, },
}; };

View file

@ -41,7 +41,7 @@ class TextFileMessageStorage {
return; return;
} }
let line = `[${log.getHumanDate()}] `; let line = `[${log.getHumanDate(msg.time)}] `;
// message types from src/models/msg.js // message types from src/models/msg.js
switch (msg.type) { switch (msg.type) {

View file

@ -56,6 +56,6 @@ describe("Helper", function() {
}); });
describe("#getHumanDate()", function() { describe("#getHumanDate()", function() {
expect(log.getHumanDate()).to.equal(moment().format("YYYY-MM-DD HH:mm:ss")); expect(log.getHumanDate(moment(1455090480000).utc())).to.equal("2016-02-10 07:48:00");
}); });
}); });