From 89edd15c5d0ccb90426e107dd078d9d64ca84399 Mon Sep 17 00:00:00 2001 From: siwalikm Date: Sat, 17 Nov 2018 14:02:22 +0530 Subject: adding test cases --- lib/time.js | 7 +++++-- test/time.unit.js | 10 ++++++++++ 2 files changed, 15 insertions(+), 2 deletions(-) diff --git a/lib/time.js b/lib/time.js index 51f7a28f..580d33ee 100644 --- a/lib/time.js +++ b/lib/time.js @@ -11,7 +11,11 @@ var _Time = function(faker) { * @method faker.time.recent * @param {string} outputType - 'abbr' || 'wide' || 'unix' (default choice) */ - self.recent = function(outputType = "unix") { + self.recent = function(outputType) { + if (typeof outputType === "undefined") { + outputType = 'unix'; + } + var date = new Date(); switch (outputType) { case "abbr": @@ -21,7 +25,6 @@ var _Time = function(faker) { date = date.toTimeString(); break; case "unix": - default: date = date.getTime(); break; } diff --git a/test/time.unit.js b/test/time.unit.js index 88b81904..4ea07d2d 100644 --- a/test/time.unit.js +++ b/test/time.unit.js @@ -8,9 +8,19 @@ describe("time.js", function () { describe("recent()", function () { it("returns the recent timestamp in Unix time format", function () { var date = faker.time.recent(); + assert.ok(typeof date === 'number'); assert.ok(date == new Date().getTime()); }); + it("returns the recent timestamp in full time string format", function () { + var date = faker.time.recent('wide'); + assert.ok(date == new Date().toTimeString()); + }); + + it("returns the recent timestamp in abbreviated string format", function () { + var date = faker.time.recent('abbr'); + assert.ok(date == new Date().toLocaleTimeString()); + }); }); }); -- cgit v1.2.3