-
-
Notifications
You must be signed in to change notification settings - Fork 758
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor/test jest to vitest 2486 #2687
Changes from all commits
9efd2ff
c63ee2a
0632235
ba5255a
e301f94
872414a
e4b44e9
843d265
dd07aa4
c76df8d
eb4e574
f469f24
d3cbd1e
f5c4099
7eea8f4
8e33348
7993f66
9b0dba0
ae29808
28e9806
37b4e1a
cf57c47
ec3a2d1
d1437b3
9066062
3c9fa91
126c368
ebb69a8
0109414
15de308
f3f63f8
f5b3608
3944368
0d77178
54a5f3f
aca6777
fe4c92b
4ed0795
ff79bd3
99d1a13
29db3b6
05aaf09
5bff66e
19ad9a7
2d22bde
03a5a7d
9e7ebf7
ffb686e
0254bb0
45efc65
3b146ab
fcd3ea8
78b2b7c
3653220
2e7e904
d486670
836b534
5e50f49
d62bdb3
f6ebecf
1b5e335
d440ad4
7306f63
0970cb6
b6be492
f5337c4
cacf399
a1d5361
c9067f9
c8b8e20
ab9814a
02d67df
1dd56c5
7f71db5
5448735
fc70a6d
01f4785
e368487
43316fc
3d4a87e
f97ce56
392b042
2f4d7c8
6d6dadb
6e66777
7db1322
c0bd3c5
d36ec71
a3c3555
0f78540
c34c2c7
aa3d19e
62c0743
47bdf2b
cf8cda4
97ea075
cb34ca5
5ed3808
6f65709
b3794bb
989b154
968c284
3a08864
a421b28
75d2e55
6e51fc0
988564b
48bf6f5
e08238e
efd84c9
0cb4988
b9c7774
7bfb5dc
7e80b17
fc346d2
9869bde
fbce25b
190e5e3
07870fd
3d6db16
475377c
79832e2
5f45c16
478a142
315641f
979b156
ceb5cce
1c641cd
7841e5d
008b414
9a08f83
bb4029d
e143243
01ef5c6
c79aadc
26a6dbe
77ce887
959399d
e009361
895e094
0e77c80
84de887
c65afe8
f6922a6
f09bb77
7bc65dc
ef590d3
d0b0588
715041c
887af3a
4718e18
6172d5d
4815208
e922a69
428865e
278bfca
ce0606c
1b28603
c98b109
84f5af8
6467ea1
025bc1d
7496bb3
a2b0167
036b92e
ca7c569
d603a0a
ec3fb61
0e05992
6393648
d9d7269
d9a8b09
2935df1
6f8408e
499dcf7
52690f7
3a112ec
ffdaea2
6fadd0c
af2f509
e5f8a8b
ed2a945
e9775ce
2ca214d
7501e34
7707708
17c53a8
f971174
c49a58c
e670a8b
2fffd90
317dd68
0b978bf
fc0dc0e
0721017
6a0a64c
3694271
c858121
3f6b41a
cd9dd1a
9dd5d7f
e8b49a4
470be76
9a379f1
d16c545
c71af97
c923cff
626fe45
8bfaf51
2e6ea0b
f3a63b8
13eae8b
44f8584
b397013
60705d2
4ca1551
c3a6289
4c28ffa
86849df
e9531ff
ea145f2
ff25183
7d2f235
6ae40a3
92a9694
e563843
1332fec
305071b
383b4cf
b28fcde
c80b44f
9b5bd3e
c2630ae
90ae8b8
f1e8c32
afa614a
0630cff
cb5c255
316c3cb
a5c9d97
11ece4a
a318390
77b4561
a16118e
debe672
880f6f3
f4aafd7
4aaa8cf
c849b44
9723078
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json | ||
language: "en-US" | ||
early_access: false | ||
reviews: | ||
profile: "chill" | ||
request_changes_workflow: true | ||
high_level_summary: true | ||
poem: true | ||
review_status: true | ||
collapse_walkthrough: false | ||
auto_review: | ||
enabled: true | ||
drafts: false | ||
base_branches: | ||
- develop | ||
- main | ||
chat: | ||
auto_reply: true |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,25 @@ | ||
node_modules | ||
npm-debug.log | ||
Dockerfile | ||
.git | ||
.gitignore | ||
.env | ||
.env.* | ||
dist | ||
coverage | ||
.nyc_output | ||
*.md | ||
.github | ||
tests | ||
__tests__ | ||
*.test.* | ||
*.spec.* | ||
# Development files | ||
*.log | ||
*.lock | ||
.DS_Store | ||
.idea | ||
.vscode | ||
# Build artifacts | ||
build | ||
out |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,31 @@ | ||
# 👋 Welcome, we're glad you're setting up an installation of Talawa-admin. Copy this | ||
# file to .env or set the variables in your local environment manually. | ||
|
||
|
||
# Custom port number for the talawa-admin development server to run on. Default is 4321. | ||
|
||
PORT=4321 | ||
|
||
# Run Talawa-api locally in your system, and put its url into the same. | ||
|
||
REACT_APP_TALAWA_URL= | ||
|
||
# Do you want to setup and use "I'm not a robot" Checkbox (Google Recaptcha)? | ||
# If no, leave blank, else write yes | ||
# Example: REACT_APP_USE_RECAPTCHA=yes | ||
|
||
REACT_APP_USE_RECAPTCHA= | ||
|
||
# If you are using Google Recaptcha, i.e., REACT_APP_USE_RECAPTCHA=yes, read the following steps | ||
# Get the google recaptcha site key from google recaptcha admin or https://www.google.com/recaptcha/admin/create | ||
# from here for reCAPTCHA v2 and "I'm not a robot" Checkbox, and paste the key here. | ||
# Note: In domains, fill localhost | ||
|
||
REACT_APP_RECAPTCHA_SITE_KEY= | ||
|
||
# has to be inserted in the env file to use plugins and other websocket based features. | ||
REACT_APP_BACKEND_WEBSOCKET_URL=ws://localhost:4000/graphql/ | ||
|
||
# If you want to logs Compiletime and Runtime error , warning and info write YES or if u want to | ||
# keep the console clean leave it blank | ||
ALLOW_LOGS= |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
# Contains the PDF file of the Tag as JSON string, thus does not need to be linted | ||
src/components/CheckIn/tagTemplate.ts | ||
package.json | ||
package-lock.json | ||
tsconfig.json |
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
@@ -1,16 +1,18 @@ | ||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||
"env": { | ||||||||||||||||||||||||||||||||||
"browser": true, | ||||||||||||||||||||||||||||||||||
"node": true, | ||||||||||||||||||||||||||||||||||
"es6": true | ||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
// Specify the rulesets from other ESLint Plugins tobe used | ||||||||||||||||||||||||||||||||||
"extends": [ | ||||||||||||||||||||||||||||||||||
"plugin:react/recommended", | ||||||||||||||||||||||||||||||||||
"eslint:recommended", | ||||||||||||||||||||||||||||||||||
"plugin:jest/recommended", | ||||||||||||||||||||||||||||||||||
"plugin:prettier/recommended", | ||||||||||||||||||||||||||||||||||
"plugin:@typescript-eslint/recommended" | ||||||||||||||||||||||||||||||||||
"plugin:@typescript-eslint/recommended", | ||||||||||||||||||||||||||||||||||
"eslint-config-prettier", | ||||||||||||||||||||||||||||||||||
"prettier" | ||||||||||||||||||||||||||||||||||
], | ||||||||||||||||||||||||||||||||||
"globals": { | ||||||||||||||||||||||||||||||||||
"Atomics": "readonly", | ||||||||||||||||||||||||||||||||||
|
@@ -25,46 +27,121 @@ | |||||||||||||||||||||||||||||||||
"sourceType": "module" | ||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
// Specify the ESLint plugins tobe used | ||||||||||||||||||||||||||||||||||
"plugins": ["react", "@typescript-eslint", "react-hooks", "jest"], | ||||||||||||||||||||||||||||||||||
"plugins": [ | ||||||||||||||||||||||||||||||||||
"react", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint", | ||||||||||||||||||||||||||||||||||
"jest", | ||||||||||||||||||||||||||||||||||
"import", | ||||||||||||||||||||||||||||||||||
"eslint-plugin-tsdoc", | ||||||||||||||||||||||||||||||||||
"prettier" | ||||||||||||||||||||||||||||||||||
], | ||||||||||||||||||||||||||||||||||
Comment on lines
+30
to
+37
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Update ESLint plugins for Vitest Replace Jest plugin with Vitest plugin in the plugins section. "plugins": [
"react",
"@typescript-eslint",
- "jest",
+ "vitest",
"import",
"eslint-plugin-tsdoc",
"prettier"
], 📝 Committable suggestion
Suggested change
|
||||||||||||||||||||||||||||||||||
"rules": { | ||||||||||||||||||||||||||||||||||
"react/destructuring-assignment": ["warn", "always"], | ||||||||||||||||||||||||||||||||||
"react/no-multi-comp": ["error", { "ignoreStateless": false }], | ||||||||||||||||||||||||||||||||||
"react/jsx-filename-extension": ["error", { "extensions": [".tsx"] }], | ||||||||||||||||||||||||||||||||||
"react/destructuring-assignment": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/explicit-module-boundary-types": "error", | ||||||||||||||||||||||||||||||||||
"react/no-multi-comp": [ | ||||||||||||||||||||||||||||||||||
"error", | ||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||
"ignoreStateless": false | ||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||
], | ||||||||||||||||||||||||||||||||||
"react/jsx-filename-extension": [ | ||||||||||||||||||||||||||||||||||
"error", | ||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||
"extensions": [".tsx"] | ||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||
], | ||||||||||||||||||||||||||||||||||
"import/no-duplicates": "error", | ||||||||||||||||||||||||||||||||||
"tsdoc/syntax": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/ban-ts-comment": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/no-unused-vars": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/no-explicit-any": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/no-inferrable-types": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/no-non-null-asserted-optional-chain": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/no-non-null-assertion": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/no-var-requires": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/no-unsafe-function-type": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/no-wrapper-object-types": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/no-empty-object-type": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/no-duplicate-enum-values": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/array-type": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/consistent-type-assertions": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/consistent-type-imports": "error", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/explicit-function-return-type": [ | ||||||||||||||||||||||||||||||||||
2, | ||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||
"allowExpressions": true, | ||||||||||||||||||||||||||||||||||
"allowTypedFunctionExpressions": true | ||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||
], | ||||||||||||||||||||||||||||||||||
"camelcase": "off", | ||||||||||||||||||||||||||||||||||
"@typescript-eslint/naming-convention": [ | ||||||||||||||||||||||||||||||||||
"error", | ||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||
"selector": "interface", | ||||||||||||||||||||||||||||||||||
"format": ["PascalCase"], | ||||||||||||||||||||||||||||||||||
"prefix": ["Interface", "TestInterface"] | ||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||
"selector": ["typeAlias", "typeLike", "enum"], | ||||||||||||||||||||||||||||||||||
"format": ["PascalCase"] | ||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||
"selector": "typeParameter", | ||||||||||||||||||||||||||||||||||
"format": ["PascalCase"], | ||||||||||||||||||||||||||||||||||
"prefix": ["T"] | ||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||
"selector": "variable", | ||||||||||||||||||||||||||||||||||
"format": ["camelCase", "UPPER_CASE", "PascalCase"], | ||||||||||||||||||||||||||||||||||
"leadingUnderscore": "allow" | ||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||
"selector": "parameter", | ||||||||||||||||||||||||||||||||||
"format": ["camelCase"], | ||||||||||||||||||||||||||||||||||
"leadingUnderscore": "allow" | ||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||
"selector": "function", | ||||||||||||||||||||||||||||||||||
"format": ["camelCase", "PascalCase"] | ||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||
"selector": "memberLike", | ||||||||||||||||||||||||||||||||||
"modifiers": ["private"], | ||||||||||||||||||||||||||||||||||
"format": ["camelCase"], | ||||||||||||||||||||||||||||||||||
"leadingUnderscore": "require" | ||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||
"selector": "variable", | ||||||||||||||||||||||||||||||||||
"modifiers": ["exported"], | ||||||||||||||||||||||||||||||||||
"format": null | ||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||
], | ||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
// Ensures that components are always written in PascalCase | ||||||||||||||||||||||||||||||||||
"react/jsx-pascal-case": [ | ||||||||||||||||||||||||||||||||||
"error", | ||||||||||||||||||||||||||||||||||
{ "allowAllCaps": false, "allowNamespace": false } | ||||||||||||||||||||||||||||||||||
], | ||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
// Enforces whitespace around equal sign operators | ||||||||||||||||||||||||||||||||||
"react/jsx-equals-spacing": ["warn", "never"], | ||||||||||||||||||||||||||||||||||
"react/no-this-in-sfc": "error", | ||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
// Ensures that components are always indented by 2 spaces | ||||||||||||||||||||||||||||||||||
"react/jsx-indent": ["warn", 2], | ||||||||||||||||||||||||||||||||||
"react/jsx-tag-spacing": [ | ||||||||||||||||||||||||||||||||||
"warn", | ||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||
"afterOpening": "never", | ||||||||||||||||||||||||||||||||||
"beforeClosing": "never", | ||||||||||||||||||||||||||||||||||
"beforeSelfClosing": "always" | ||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||
], | ||||||||||||||||||||||||||||||||||
"jest/expect-expect": 0, | ||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove Jest-specific rules Remove Jest-specific rules since we're transitioning to Vitest. - "jest/expect-expect": 0, |
||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
// Enforce Strictly functional components | ||||||||||||||||||||||||||||||||||
"react/no-unstable-nested-components": ["error", { "allowAsProps": true }], | ||||||||||||||||||||||||||||||||||
"react/function-component-definition": [ | ||||||||||||||||||||||||||||||||||
"error", | ||||||||||||||||||||||||||||||||||
0, | ||||||||||||||||||||||||||||||||||
{ "namedComponents": "function-declaration" } | ||||||||||||||||||||||||||||||||||
] | ||||||||||||||||||||||||||||||||||
], | ||||||||||||||||||||||||||||||||||
"prettier/prettier": "error" | ||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
// Let ESLint use the react version in the package.json | ||||||||||||||||||||||||||||||||||
"settings": { | ||||||||||||||||||||||||||||||||||
"react": { | ||||||||||||||||||||||||||||||||||
"version": "detect" | ||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||
"ignorePatterns": ["**/*.css", "**/*.scss", "**/*.less", "**/*.json"] | ||||||||||||||||||||||||||||||||||
} |
Original file line number | Diff line number | Diff line change | ||||||
---|---|---|---|---|---|---|---|---|
@@ -1,9 +1,10 @@ | ||||||||
--- | ||||||||
name: Bug report | ||||||||
name: Bug Report | ||||||||
about: Create a report to help us improve. | ||||||||
title: Bug report | ||||||||
labels: Bug | ||||||||
assignees: "" | ||||||||
title: Bug Report | ||||||||
labels: bug | ||||||||
assignees: '' | ||||||||
|
||||||||
--- | ||||||||
|
||||||||
**Describe the bug** | ||||||||
|
@@ -27,3 +28,9 @@ A clear and concise description of how the code performed w.r.t expectations. | |||||||
If applicable, add screenshots to help explain your problem. | ||||||||
|
||||||||
**Additional details** | ||||||||
Add any other context or screenshots about the feature request here. | ||||||||
|
||||||||
Comment on lines
+31
to
+32
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛠️ Refactor suggestion Consider rephrasing the additional details section. The section seems to reference a "feature request" which is inconsistent with this being a bug report template. -Add any other context or screenshots about the feature request here.
+Add any other context or screenshots about the bug report here. 📝 Committable suggestion
Suggested change
|
||||||||
**Potential internship candidates** | ||||||||
|
||||||||
Please read this if you are planning to apply for a Palisadoes Foundation internship | ||||||||
- https://github.com/PalisadoesFoundation/talawa/issues/359 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
# Configuration for automated dependency updates using Dependabot | ||
version: 2 | ||
updates: | ||
# Define the target package ecosystem | ||
- package-ecosystem: "npm" | ||
# Specify the root directory | ||
directory: "/" | ||
# Schedule automated updates to run weekly | ||
schedule: | ||
interval: "monthly" | ||
# Labels to apply to Dependabot PRs | ||
labels: | ||
- "dependencies" | ||
# Specify the target branch for PRs | ||
target-branch: "develop-postgres" | ||
# Customize commit message prefix | ||
commit-message: | ||
prefix: "chore(deps):" |
Original file line number | Diff line number | Diff line change | ||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
@@ -1,13 +1,38 @@ | ||||||||||||||||||||
<!-- Thanks for submitting a pull request! Please provide enough information so that others can review your pull request. --> | ||||||||||||||||||||
|
||||||||||||||||||||
<!-- | ||||||||||||||||||||
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: | ||||||||||||||||||||
|
||||||||||||||||||||
- `develop`: For unstable code: New features and bug fixes. | ||||||||||||||||||||
- `master`: Where the stable production ready code lies. Only security related bugs. | ||||||||||||||||||||
|
||||||||||||||||||||
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. | ||||||||||||||||||||
|
||||||||||||||||||||
--> | ||||||||||||||||||||
|
||||||||||||||||||||
<!-- | ||||||||||||||||||||
Thanks for submitting a pull request! Please provide enough information so that others can review your pull request. | ||||||||||||||||||||
--> | ||||||||||||||||||||
|
||||||||||||||||||||
**What kind of change does this PR introduce?** | ||||||||||||||||||||
|
||||||||||||||||||||
<!-- E.g. a bugfix, feature, refactoring, etc… --> | ||||||||||||||||||||
|
||||||||||||||||||||
**Did you add tests for your changes?** | ||||||||||||||||||||
**Issue Number:** | ||||||||||||||||||||
|
||||||||||||||||||||
Fixes #<!--Add related issue number here.--> | ||||||||||||||||||||
|
||||||||||||||||||||
**Snapshots/Videos:** | ||||||||||||||||||||
|
||||||||||||||||||||
<!--Add snapshots or videos wherever possible.--> | ||||||||||||||||||||
|
||||||||||||||||||||
**If relevant, did you update the documentation?** | ||||||||||||||||||||
|
||||||||||||||||||||
<!--Add link to Talawa-Docs.--> | ||||||||||||||||||||
|
||||||||||||||||||||
**Summary** | ||||||||||||||||||||
|
||||||||||||||||||||
<!-- Explain the **motivation** for making this change. What existing problem does the pull request solve? --> | ||||||||||||||||||||
|
@@ -17,4 +42,23 @@ | |||||||||||||||||||
|
||||||||||||||||||||
<!-- If this PR introduces a breaking change, please describe the impact and a migration path for existing applications. --> | ||||||||||||||||||||
|
||||||||||||||||||||
## Checklist | ||||||||||||||||||||
|
||||||||||||||||||||
### CodeRabbit AI Review | ||||||||||||||||||||
- [ ] I have reviewed and addressed all critical issues flagged by CodeRabbit AI | ||||||||||||||||||||
- [ ] I have implemented or provided justification for each non-critical suggestion | ||||||||||||||||||||
- [ ] I have documented my reasoning in the PR comments where CodeRabbit AI suggestions were not implemented | ||||||||||||||||||||
|
||||||||||||||||||||
### Test Coverage | ||||||||||||||||||||
- [ ] I have written tests for all new changes/features | ||||||||||||||||||||
- [ ] I have verified that test coverage meets or exceeds 95% | ||||||||||||||||||||
- [ ] I have run the test suite locally and all tests pass | ||||||||||||||||||||
|
||||||||||||||||||||
|
||||||||||||||||||||
Comment on lines
+52
to
+56
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛠️ Refactor suggestion Add Vitest-specific test coverage guidance Since the project is migrating to Vitest, it would be helpful to include specific instructions for running Vitest tests. ### Test Coverage
- [ ] I have written tests for all new changes/features
- [ ] I have verified that test coverage meets or exceeds 95%
-- [ ] I have run the test suite locally and all tests pass
+- [ ] I have run `npm run test -- --coverage` locally and all tests pass
+- [ ] I have verified that tests work with Vitest (not Jest) 📝 Committable suggestion
Suggested change
|
||||||||||||||||||||
**Other information** | ||||||||||||||||||||
|
||||||||||||||||||||
<!--Add extra information about this PR here--> | ||||||||||||||||||||
|
||||||||||||||||||||
**Have you read the [contributing guide](https://github.com/PalisadoesFoundation/talawa-admin/blob/master/CONTRIBUTING.md)?** | ||||||||||||||||||||
|
||||||||||||||||||||
<!--Yes or No--> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update test framework configuration
Since we're transitioning from Jest to Vitest, the ESLint configuration should be updated accordingly.
📝 Committable suggestion