Skip to content

Commit 3296640

Browse files
committed
Merge pull request #48 from wmde/test-for-calendar-issue
Add test for time.Time.equals() with different calendar models.
2 parents 9001e5e + 2a27f44 commit 3296640

File tree

1 file changed

+16
-0
lines changed

1 file changed

+16
-0
lines changed

tests/lib/time/time.Time.tests.js

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -146,4 +146,20 @@ define( [
146146
time.settings.daybeforemonth = dbmStateBefore; // reset state of evil global setting
147147
} );
148148

149+
QUnit.test( 'Equality of Time objects with different calendar model', function( assert ) {
150+
$.each( validTimeDefinitions, function( name, definition ) {
151+
var time1 = new Time( definition ),
152+
time2 = new Time( definition, {
153+
calendarname: definition.calendarname === Time.CALENDAR.GREGORIAN
154+
? Time.CALENDAR.JULIAN
155+
: Time.CALENDAR.GREGORIAN
156+
} );
157+
158+
assert.ok(
159+
!time1.equals( time2 ) && !time2.equals( time1 ),
160+
'Time created from string "' + name + '" but different calendar model is not equal'
161+
);
162+
} );
163+
} );
164+
149165
} );

0 commit comments

Comments
 (0)