Skip to content

Commit 49da266

Browse files
committed
add second support to isTimeDisabled, added test
1 parent 0d2765f commit 49da266

File tree

2 files changed

+26
-1
lines changed

2 files changed

+26
-1
lines changed

src/date_utils.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -599,7 +599,8 @@ export function isTimeInList(time, times) {
599599
return times.some(
600600
(listTime) =>
601601
getHours(listTime) === getHours(time) &&
602-
getMinutes(listTime) === getMinutes(time),
602+
getMinutes(listTime) === getMinutes(time) &&
603+
getSeconds(listTime) === getSeconds(time),
603604
);
604605
}
605606

test/include_times_test.test.js

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,4 +54,28 @@ describe("TimeComponent", () => {
5454
});
5555
expect(enabledTimeItemsHasNoAriaDisabled).toBe(true);
5656
});
57+
58+
it.only("should factor in seconds", () => {
59+
const includeHoursMinutesSeconds = [
60+
utils.addHours(utils.addSeconds(today, 30), 1), //01:00:30
61+
utils.addSeconds(today, 30), //00:00:30
62+
];
63+
const { container: timeComponent } = render(
64+
<TimeComponent
65+
format="HH:mm:ss"
66+
includeTimes={includeHoursMinutesSeconds}
67+
/>,
68+
);
69+
70+
const disabledTimeItems = timeComponent.querySelectorAll(
71+
".react-datepicker__time-list-item--disabled",
72+
);
73+
74+
expect(
75+
Array.from(disabledTimeItems).map((node) => node.textContent),
76+
).toContain("01:00:00");
77+
expect(
78+
Array.from(disabledTimeItems).map((node) => node.textContent),
79+
).toContain("00:00:00");
80+
});
5781
});

0 commit comments

Comments
 (0)