From f5bb0982211ded427f27ebf8511d7507fea8bd9b Mon Sep 17 00:00:00 2001 From: Kevin Jin Date: Fri, 2 Oct 2015 21:39:02 -0400 Subject: [PATCH] Consistency with existing code Add test for unrecognized directives to test.js. Add braces around if statement. --- strftime.js | 3 ++- test.js | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/strftime.js b/strftime.js index a3420f2..8cc8f35 100644 --- a/strftime.js +++ b/strftime.js @@ -500,8 +500,9 @@ break; default: - if (isInScope) + if (isInScope) { resultString += '%'; + } resultString += format[i]; break; } diff --git a/test.js b/test.js index 66db148..17efa56 100755 --- a/test.js +++ b/test.js @@ -121,7 +121,8 @@ assert.format('%y', '11'); assert.format('%Z', null, 'GMT'); assert.format('%z', null, '+0000'); assert.format('%:z', null, '+00:00'); -assert.format('%%', '%'); // any other char +assert.format('%%', '%'); // literal percent sign +assert.format('%g', '%g'); // unrecognized directive assert.format('%F %T', null, '1970-01-01 00:00:00', new Date(0)); assert.format('%U', null, '12', new Date('2017-03-25 00:00:00 +0000')); assert.format('%U', null, '13', new Date('2017-03-26 00:00:00 +0000'));