Fix invalid ISO format dates in the test suite.

Fixing the strictness of ISO date parsing exposed this invalid ISO format dates which included a time zone offset without including a time. By appending a time of "00:00:00" to the date strings they became ISO-valid again without changing the meaning of the test.
moment · Oct 19, 2016 · 1b6e497 · 1b6e497
1 parent 43594d5
commit 1b6e497
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 7 deletions.
diff --git a/src/test/moment/is_same.js b/src/test/moment/is_same.js
@@ -140,9 +140,9 @@ test('is same millisecond', function (assert) {
 });
 
 test('is same with utc offset moments', function (assert) {
-    assert.ok(moment.parseZone('2013-02-01T-05:00').isSame(moment('2013-02-01'), 'year'), 'zoned vs local moment');
+    assert.ok(moment.parseZone('2013-02-01T00:00:00-05:00').isSame(moment('2013-02-01'), 'year'), 'zoned vs local moment');
     assert.ok(moment('2013-02-01').isSame(moment('2013-02-01').utcOffset('-05:00'), 'year'), 'local vs zoned moment');
-    assert.ok(moment.parseZone('2013-02-01T-05:00').isSame(moment.parseZone('2013-02-01T-06:30'), 'year'),
+    assert.ok(moment.parseZone('2013-02-01T00:00:00-05:00').isSame(moment.parseZone('2013-02-01T00:00:00-06:30'), 'year'),
             'zoned vs (differently) zoned moment');
 });
 

diff --git a/src/test/moment/is_same_or_after.js b/src/test/moment/is_same_or_after.js
@@ -161,9 +161,9 @@ test('is same or after millisecond', function (assert) {
 });
 
 test('is same or after with utc offset moments', function (assert) {
-    assert.ok(moment.parseZone('2013-02-01T-05:00').isSameOrAfter(moment('2013-02-01'), 'year'), 'zoned vs local moment');
+    assert.ok(moment.parseZone('2013-02-01T00:00:00-05:00').isSameOrAfter(moment('2013-02-01'), 'year'), 'zoned vs local moment');
     assert.ok(moment('2013-02-01').isSameOrAfter(moment('2013-02-01').utcOffset('-05:00'), 'year'), 'local vs zoned moment');
-    assert.ok(moment.parseZone('2013-02-01T-05:00').isSameOrAfter(moment.parseZone('2013-02-01T-06:30'), 'year'),
+    assert.ok(moment.parseZone('2013-02-01T00:00:00-05:00').isSameOrAfter(moment.parseZone('2013-02-01T00:00:00-06:30'), 'year'),
             'zoned vs (differently) zoned moment');
 });
 

diff --git a/src/test/moment/is_same_or_before.js b/src/test/moment/is_same_or_before.js
@@ -161,10 +161,12 @@ test('is same or before millisecond', function (assert) {
 });
 
 test('is same with utc offset moments', function (assert) {
-    assert.ok(moment.parseZone('2013-02-01T-05:00').isSameOrBefore(moment('2013-02-01'), 'year'), 'zoned vs local moment');
+    assert.ok(moment.parseZone('2013-02-01T00:00:00-05:00').isSameOrBefore(moment('2013-02-01'), 'year'), 'zoned vs local moment');
     assert.ok(moment('2013-02-01').isSameOrBefore(moment('2013-02-01').utcOffset('-05:00'), 'year'), 'local vs zoned moment');
-    assert.ok(moment.parseZone('2013-02-01T-05:00').isSameOrBefore(moment.parseZone('2013-02-01T-06:30'), 'year'),
-            'zoned vs (differently) zoned moment');
+    assert.ok(
+      moment.parseZone('2013-02-01T00:00:00-05:00').isSameOrBefore(moment.parseZone('2013-02-01T00:00:00-06:30'), 'year'),
+      'zoned vs (differently) zoned moment'
+    );
 });
 
 test('is same with invalid moments', function (assert) {