Refactored src/components/GroupChatDetails from jest to vitest (#3453)

* Refactored src/components/GroupChatDetails from jest to vitest * removed all instances and files related to jest * removed all instances and files related to jest * cleanup to eradicate jest fully * chore!: port to exclude instead ignore * refactor! path --------- Co-authored-by: Vamshi Maskuri <[email protected]>
PalisadoesFoundation · Jan 27, 2025 · 0445cb8 · 0445cb8
1 parent 73cf8cd
commit 0445cb8
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 145 deletions.
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -8,7 +8,6 @@
   "extends": [
     "plugin:react/recommended",
     "eslint:recommended",
-    "plugin:jest/recommended",
     "plugin:prettier/recommended",
     "plugin:@typescript-eslint/recommended",
     "eslint-config-prettier",
@@ -30,7 +29,6 @@
   "plugins": [
     "react",
     "@typescript-eslint",
-    "jest",
     "import",
     "eslint-plugin-tsdoc",
     "prettier"
@@ -128,8 +126,6 @@
     "react/jsx-equals-spacing": ["warn", "never"],
     "react/no-this-in-sfc": "error",
 
-    "jest/expect-expect": 0,
-
     "react/no-unstable-nested-components": ["error", { "allowAsProps": true }],
     "react/function-component-definition": [
       0,

diff --git a/.github/workflows/pull-request.yml b/.github/workflows/pull-request.yml
@@ -243,51 +243,20 @@ jobs:
       - name: Get changed TypeScript files
         id: changed-files
         uses: tj-actions/changed-files@v45
-
-      - name: Run Jest Tests
-        if: steps.changed-files.outputs.any_changed == 'true'
-        env:
-          NODE_V8_COVERAGE: './coverage/jest'
-        run: |
-          npm run test -- --watchAll=false --coverage
-
-      - name: Upload Jest Coverage to Codecov
-        if: steps.changed-files.outputs.any_changed == 'true'
-        uses: codecov/codecov-action@v4
-        with:
-          token: ${{ secrets.CODECOV_TOKEN }}
-          files: ./coverage/jest/lcov.info
-          flags: jest
-          fail_ci_if_error: true
 
       - name: Run Vitest Tests
         if: steps.changed-files.outputs.any_changed == 'true'
         env:
           NODE_V8_COVERAGE: './coverage/vitest'
         run: |
           npm run test:vitest:coverage
-
-      - name: Upload Vitest Coverage to Codecov
-        if: steps.changed-files.outputs.any_changed == 'true'
-        uses: codecov/codecov-action@v4
-        with:
-          token: ${{ secrets.CODECOV_TOKEN }}
-          files: ./coverage/vitest/lcov.info
-          flags: vitest
-          fail_ci_if_error: true
-
-      - name: Merge Jest and Vitest Coverage Reports
-        run: |
-          mkdir -p ./coverage
-          npx lcov-result-merger './coverage/*/lcov.info' './coverage/lcov.info'
   
-      - name: Upload Combined Coverage to Codecov
+      - name: Upload Coverage to Codecov
         uses: codecov/codecov-action@v4
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
-          gcov_ignore: 'docs/'
-          files: ./coverage/lcov.info
-          flags: combined
+          exclude: 'docs/'
+          files: ./coverage/vitest/lcov.info
           fail_ci_if_error: true
 
       - name: TypeScript compilation for changed files
@@ -301,7 +270,7 @@ jobs:
       - name: Test acceptable level of code coverage
         uses: VeryGoodOpenSource/very_good_coverage@v3
         with:
-          path: "./coverage/lcov.info"
+          path: "./coverage/vitest/lcov.info"
           min_coverage: 0
 
   # Graphql-Inspector:

diff --git a/.github/workflows/scripts/countline.py b/.github/workflows/scripts/countline.py
@@ -41,7 +41,7 @@ def _valid_filename(filepath):
 
     """
     # Initialize key variables
-    invalid_filenames = [".test.", ".spec."]
+    invalid_filenames = [".spec."]
     result = True
 
     # Test

diff --git a/jest-preview.config.ts b/jest-preview.config.ts
diff --git a/jest.config.js b/jest.config.js
diff --git a/...roupChatDetails/GroupChatDetails.test.tsx → ...roupChatDetails/GroupChatDetails.spec.tsx b/...roupChatDetails/GroupChatDetails.test.tsx → ...roupChatDetails/GroupChatDetails.spec.tsx
@@ -2,8 +2,6 @@ import React from 'react';
 import {
   render,
   screen,
-  // fireEvent,
-  // waitFor,
   act,
   fireEvent,
   waitFor,
@@ -18,6 +16,7 @@ import { USERS_CONNECTION_LIST } from 'GraphQl/Queries/Queries';
 import { I18nextProvider, initReactI18next } from 'react-i18next';
 import i18n from 'i18next';
 import { useLocalStorage } from 'utils/useLocalstorage';
+import { vi } from 'vitest';
 
 const { setItem } = useLocalStorage();
 
@@ -432,10 +431,10 @@ describe('GroupChatDetails', () => {
       <I18nextProvider i18n={i18n}>
         <MockedProvider mocks={mocks} addTypename={false}>
           <GroupChatDetails
-            toggleGroupChatDetailsModal={jest.fn()}
+            toggleGroupChatDetailsModal={vi.fn()}
             groupChatDetailsModalisOpen={true}
             chat={chat}
-            chatRefetch={jest.fn()}
+            chatRefetch={vi.fn()}
           />
         </MockedProvider>
       </I18nextProvider>,
@@ -456,10 +455,10 @@ describe('GroupChatDetails', () => {
       <I18nextProvider i18n={i18n}>
         <MockedProvider mocks={mocks} addTypename={false}>
           <GroupChatDetails
-            toggleGroupChatDetailsModal={jest.fn()}
+            toggleGroupChatDetailsModal={vi.fn()}
             groupChatDetailsModalisOpen={true}
             chat={mockChat}
-            chatRefetch={jest.fn()}
+            chatRefetch={vi.fn()}
           />
         </MockedProvider>
       </I18nextProvider>,
@@ -518,10 +517,10 @@ describe('GroupChatDetails', () => {
       <I18nextProvider i18n={i18n}>
         <MockedProvider mocks={mocks} addTypename={false}>
           <GroupChatDetails
-            toggleGroupChatDetailsModal={jest.fn()}
+            toggleGroupChatDetailsModal={vi.fn()}
             groupChatDetailsModalisOpen={true}
             chat={mockChat}
-            chatRefetch={jest.fn()}
+            chatRefetch={vi.fn()}
           />
         </MockedProvider>
       </I18nextProvider>,
@@ -558,19 +557,17 @@ describe('GroupChatDetails', () => {
     await act(async () => {
       fireEvent.click(await screen.findByTestId('addUserBtn'));
     });
-
-    await wait(10000);
   });
   // test case for updating group chat image
   it('update group chat image', async () => {
     render(
       <I18nextProvider i18n={i18n}>
         <MockedProvider mocks={mocks} addTypename={false}>
           <GroupChatDetails
-            toggleGroupChatDetailsModal={jest.fn()}
+            toggleGroupChatDetailsModal={vi.fn()}
             groupChatDetailsModalisOpen={true}
             chat={mockChat}
-            chatRefetch={jest.fn()}
+            chatRefetch={vi.fn()}
           />
         </MockedProvider>
       </I18nextProvider>,
@@ -597,7 +594,5 @@ describe('GroupChatDetails', () => {
     });
 
     fireEvent.change(fileInput);
-
-    await wait(10000);
   });
 });