Fixing tests

Vincit · Sep 6, 2024 · c74f4ab · c74f4ab
1 parent 933ae2e
commit c74f4ab
Show file tree

Hide file tree

Showing 6 changed files with 8 additions and 5 deletions.
diff --git a/frontend/src/components/BottomDrawer/BottomDrawer.tsx b/frontend/src/components/BottomDrawer/BottomDrawer.tsx
@@ -84,6 +84,7 @@ interface Props {
     disableSwipeToOpen: boolean;
     mounted?: boolean;
     zindex?: number;
+    testId?: string;
 }
 
 const BottomDrawer = (props: Props) => {
@@ -162,7 +163,7 @@ const BottomDrawer = (props: Props) => {
                 }}
             />
             <Drawer
-                data-testid="BottomDrawer"
+                data-testid={props.testId || 'BottomDrawer'}
                 anchor="bottom"
                 open={isOpen}
                 onClose={toggleDrawer(false)}

diff --git a/frontend/src/components/CurrentBooking/AlterBookingDrawer.tsx b/frontend/src/components/CurrentBooking/AlterBookingDrawer.tsx
@@ -277,6 +277,7 @@ const AlterBookingDrawer = (props: Props) => {
             toggle={toggle}
             disableSwipeToOpen={true}
             zindex={1200}
+            testId={'BookingDrawer'}
         >
             <Box
                 style={{

diff --git a/frontend/src/components/DurationTimePickerDrawer/DurationTimePickerDrawer.test.tsx b/frontend/src/components/DurationTimePickerDrawer/DurationTimePickerDrawer.test.tsx
@@ -90,7 +90,7 @@ describe('Duration selection from picker', () => {
         const hour1 = getByText(hours, '01');
         fireEvent.click(hour1);
 
-        const btn = screen.getByRole('button');
+        const btn = screen.getByTestId('set-duration-button');
 
         fireEvent.click(btn);
 

diff --git a/frontend/src/components/DurationTimePickerDrawer/DurationTimePickerDrawer.tsx b/frontend/src/components/DurationTimePickerDrawer/DurationTimePickerDrawer.tsx
@@ -100,6 +100,7 @@ const DurationTimePickerDrawer = (props: DurationTimePickerDrawerProps) => {
                     <Row>
                         <DrawerButtonPrimary
                             aria-label="confirm"
+                            data-testid={'set-duration-button'}
                             onClick={() => handleSetDuration()}
                         >
                             Confirm

diff --git a/frontend/src/components/StartingTimePickerDrawer/StartingTimePickerDrawer.test.tsx b/frontend/src/components/StartingTimePickerDrawer/StartingTimePickerDrawer.test.tsx
@@ -99,7 +99,7 @@ describe('Starting time selection from drawer', () => {
         const hour = getByText(hours, (h < 10 ? '0' : '') + h.toString());
         fireEvent.click(hour);
 
-        const btn = screen.getAllByRole('button')[1];
+        const btn = screen.getByTestId('ConfirmStartingTimeButton');
         fireEvent.click(btn);
 
         expect(testObj.setExpandTimePickerDrawer).toBeCalledTimes(1);

diff --git a/frontend/src/components/util/TimeLeft.test.tsx b/frontend/src/components/util/TimeLeft.test.tsx
@@ -24,7 +24,7 @@ describe('TimeLeft', () => {
 
         const timeLeftValue = await screen.findByTestId('TimeLeftTest');
         await waitFor(() =>
-            expect(timeLeftValue).toHaveTextContent('Time Left: 1 h 29 min')
+            expect(timeLeftValue).toHaveTextContent('1 h 29 min')
         );
     });
 
@@ -41,7 +41,7 @@ describe('TimeLeft', () => {
 
         const timeLeftValue = await screen.findByTestId('TimeLeftTest');
         await waitFor(() =>
-            expect(timeLeftValue).toHaveTextContent(/Time Left: 4[45] min/)
+            expect(timeLeftValue).toHaveTextContent(/4[45] min/)
         );
     });
 });