Skip to content

Commit 7b1814c

Browse files
move ui tests to __tests__ and revert functional changes
1 parent 526e868 commit 7b1814c

4 files changed

Lines changed: 9 additions & 236 deletions

File tree

frontend/src/components/ui/date-time-picker.test.tsx renamed to frontend/src/components/ui/__tests__/date-time-picker.test.tsx

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
import { render, screen, fireEvent } from '@testing-library/react';
1+
import { render, screen, fireEvent, waitFor } from '@testing-library/react';
22
import userEvent from '@testing-library/user-event';
3-
import { DateTimePicker } from '../ui/date-time-picker';
3+
import { DateTimePicker } from '../date-time-picker';
44
import '@testing-library/jest-dom';
55

66
describe('DateTimePicker', () => {
@@ -38,7 +38,9 @@ describe('DateTimePicker', () => {
3838

3939
// Close popover using Escape key
4040
fireEvent.keyDown(document, { key: 'Escape' });
41-
expect(screen.queryByText(/February 2026/)).not.toBeInTheDocument(); // Check for absence of specific content
41+
await waitFor(() => {
42+
expect(screen.queryByText(/February 2026/)).not.toBeInTheDocument(); // Check for absence of specific content
43+
});
4244
});
4345

4446
it('allows selecting a date from the calendar', async () => {
@@ -58,7 +60,9 @@ describe('DateTimePicker', () => {
5860
await user.click(dateToSelect);
5961

6062
// Expect the popover to close after selecting a date
61-
expect(screen.queryByText(/February 2026/)).not.toBeInTheDocument();
63+
await waitFor(() => {
64+
expect(screen.queryByText(/February 2026/)).not.toBeInTheDocument();
65+
});
6266

6367
// Check if onDateTimeChange was called with the correct date (year, month, and day)
6468
expect(mockOnDateTimeChange).toHaveBeenCalledTimes(1);

frontend/src/components/ui/date-time-picker.tsx

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,13 +58,11 @@ export const DateTimePicker = React.forwardRef<
5858

5959
isInternalUpdate.current = true;
6060
onDateTimeChange(newDate, false);
61-
setIsOpen(false);
6261
} else {
6362
setInternalDate(undefined);
6463
setHasTime(false);
6564
isInternalUpdate.current = true;
6665
onDateTimeChange(undefined, false);
67-
setIsOpen(false);
6866
}
6967
};
7068

frontend/src/components/ui/multi-select-filter.test.tsx

Lines changed: 0 additions & 229 deletions
This file was deleted.

frontend/tsconfig.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,6 @@
2727
"@/*": ["./src/*"]
2828
}
2929
},
30-
"include": ["src"],
30+
"include": ["src", "../../extra/multi-select-filter.test.tsx"],
3131
"references": [{ "path": "./tsconfig.node.json" }]
3232
}

0 commit comments

Comments
 (0)