Skip to content

Commit 2a567a3

Browse files
committed
Remove the microsecond TODO after verifying intent with Trent. Rename the local fixture variables.
1 parent 356868a commit 2a567a3

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

test/jquery-ui-timepicker-addon_spec.js

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -543,17 +543,16 @@ describe('datetimepicker', function() {
543543
});
544544
});
545545

546-
// TODO: Should microseconds be three digits or six? Does this complement millis or replace? The regexp looks like it replaces.
547546
describe('microseconds', function() {
548547
it('formats c correctly', function() {
549-
var singleDigitMillis = {microsec: 3},
550-
doubleDigitMillis = {microsec: 17},
551-
tripleDigitMillis = {microsec: 123};
548+
var singleDigitMicros = {microsec: 3},
549+
doubleDigitMicros = {microsec: 17},
550+
tripleDigitMicros = {microsec: 123};
552551

553552
expect($.datepicker.formatTime('c', emptyTime)).toBe('000');
554-
expect($.datepicker.formatTime('c', singleDigitMillis)).toBe('003');
555-
expect($.datepicker.formatTime('c', doubleDigitMillis)).toBe('017');
556-
expect($.datepicker.formatTime('c', tripleDigitMillis)).toBe('123');
553+
expect($.datepicker.formatTime('c', singleDigitMicros)).toBe('003');
554+
expect($.datepicker.formatTime('c', doubleDigitMicros)).toBe('017');
555+
expect($.datepicker.formatTime('c', tripleDigitMicros)).toBe('123');
557556
});
558557
});
559558

0 commit comments

Comments
 (0)