Upgrade package @testing-library/user-event from 12.8.3 to 13.5.0 #2468

.github/pull_request_template.md

@@ -1,5 +1,4 @@
-
 <!--
 This section can be deleted after reading.
 
 We employ the following branching strategy to simplify the development process and to ensure that only stable code is pushed to the `master` branch:
@@ -10,7 +9,6 @@
 NOTE!!!
 
 ONLY SUBMIT PRS AGAINST OUR `DEVELOP` BRANCH. THE DEFAULT IS `MAIN`, SO YOU WILL HAVE TO MODIFY THIS BEFORE SUBMITTING YOUR PR FOR REVIEW. PRS MADE AGAINST `MAIN` WILL BE CLOSED.
-
 -->
 
 <!--
@@ -29,6 +27,12 @@
 
 <!--Add snapshots or videos wherever possible.-->
 
+<!--Yes or No. Note: Add unit tests or automation tests for your code.-->
+
+**Snapshots/Videos:**
+
+<!--Add snapshots or videos wherever possible.-->
+
 **If relevant, did you update the documentation?**
 
 <!--Add link to Talawa-Docs.-->

.github/workflows/pull-request.yml

@@ -1,4 +1,4 @@
 ##############################################################################
 ##############################################################################
 #
 # NOTE!
@@ -217,7 +217,6 @@
               npx tsc --noEmit "$file"
             fi
           done
-
       - name: Present and Upload coverage to Codecov as ${{env.CODECOV_UNIQUE_NAME}}
         uses: codecov/codecov-action@v4
         with:
@@ -247,7 +246,6 @@
 
       - name: resolve dependency
         run: npm install -g @graphql-inspector/cli
-
       - name: Clone API Repository
         run: |
           # Retrieve the complete branch name directly from the GitHub context

package.json

@@ -1,4 +1,4 @@
 {
   "name": "talawa-admin",
   "version": "3.0.0",
   "private": true,
@@ -113,7 +113,7 @@
     "@babel/preset-typescript": "^7.26.0",
     "@testing-library/jest-dom": "^6.5.0",
     "@testing-library/react": "^16.0.1",
-    "@testing-library/user-event": "^12.1.10",
+    "@testing-library/user-event": "^13.5.0",
     "@types/inquirer": "^9.0.7",
     "@types/jest": "^26.0.24",
     "@types/js-cookie": "^3.0.6",

src/components/OrgSettings/ActionItemCategories/OrgActionItemCategories.test.tsx

@@ -61,10 +61,10 @@ const renderActionItemCategories = (
 describe('Testing Organisation Action Item Categories', () => {
   it('should render the Action Item Categories Screen', async () => {
     renderActionItemCategories(link1, 'orgId');
-    await waitFor(() => {
+    await waitFor(async () => {
       expect(screen.getByTestId('searchByName')).toBeInTheDocument();
-      expect(screen.getByText('Category 1')).toBeInTheDocument();
-      expect(screen.getByText('Category 2')).toBeInTheDocument();
+      expect(await screen.findByText('Category 1')).toBeInTheDocument();
+      expect(await screen.findByText('Category 2')).toBeInTheDocument();
     });
   });
 

src/components/OrganizationScreen/OrganizationScreen.test.tsx

@@ -81,14 +81,16 @@ describe('Testing OrganizationScreen', () => {
     fireEvent.click(closeButton);
 
     // Check for contract class after closing
-    expect(screen.getByTestId('mainpageright')).toHaveClass('_expand_ccl5z_8');
+    expect(screen.getByTestId('mainpageright')).toHaveClass(
+      '_pageContainer_ccl5z_1 _expand_ccl5z_15',
+    );
 
     const openButton = screen.getByTestId('openMenu');
     fireEvent.click(openButton);
 
     // Check for expand class after opening
     expect(screen.getByTestId('mainpageright')).toHaveClass(
-      '_contract_ccl5z_61',
+      '_pageContainer_ccl5z_1 _contract_ccl5z_121',
     );
   });
 

src/components/Pagination/Pagination.test.tsx

@@ -1,11 +1,10 @@
 import React, { act } from 'react';
-import { render, screen } from '@testing-library/react';
+import { fireEvent, render, screen } from '@testing-library/react';
 import { Provider } from 'react-redux';
 import { BrowserRouter } from 'react-router-dom';
 import { createTheme, ThemeProvider } from '@mui/material/styles';
 import Pagination from './Pagination';
 import { store } from 'state/store';
-import userEvent from '@testing-library/user-event';
 
 describe('Testing Pagination component', () => {
   const props = {
@@ -26,8 +25,8 @@ describe('Testing Pagination component', () => {
       </BrowserRouter>,
     );
     await act(async () => {
-      userEvent.click(screen.getByTestId(/nextPage/i));
-      userEvent.click(screen.getByTestId(/previousPage/i));
+      fireEvent.click(screen.getByTestId(/nextPage/i));
+      fireEvent.click(screen.getByTestId(/previousPage/i));
     });
   });
 });
@@ -58,7 +57,7 @@ test('Component should be rendered properly', async () => {
   );
 
   await act(async () => {
-    userEvent.click(screen.getByTestId(/nextPage/i));
-    userEvent.click(screen.getByTestId(/previousPage/i));
+    fireEvent.click(screen.getByTestId(/nextPage/i));
+    fireEvent.click(screen.getByTestId(/previousPage/i));
   });
 });