-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
NoMongo: migrate the file test/routes/graphql/Mutation/deleteChatMessage.test.ts to integration test #3273
Comments
can i work on this @palisadoes |
@bandhan-majumder Do you want to work on this issue? I am already working on a different issue, and I don’t want to be a hurdle in between, as this is a GSoC priority issue. |
You can also ask @palisadoes to get it assigned to yourself. |
I can work on this issue |
Thanks @abbi4code . I will ask to take this up if it gets no activity with in 10 days. |
How's progress on this? It needs to be completed as a GSoC prerequisite. |
I have created a PR related to this (update message), so if any help is needed, I'll be happy to help ! |
Hi, I’ve been investigating the issue, but I’m stuck because the
directory test/routes/graphql/Mutation/ does not exist in the repository. I
also ran a search for deleteChatMessage, but it only appears in Git history
and not in the actual codebase. Could you confirm the correct location or
if the file has been renamed/moved? Any guidance would be appreciated.
Thanks!
…On Sun, 9 Mar 2025 at 12:33, Peter Harrison ***@***.***> wrote:
How's progress on this? It needs to be completed as a GSoC prerequisite.
—
Reply to this email directly, view it on GitHub
<#3273 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BDSHJ6LCUF4NYKMGN24WNM32TPRV5AVCNFSM6AAAAABXUWB6MWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMBYG4YDKNJQHA>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
[image: palisadoes]*palisadoes* left a comment
(PalisadoesFoundation/talawa-api#3273)
<#3273 (comment)>
How's progress on this? It needs to be completed as a GSoC prerequisite.
—
Reply to this email directly, view it on GitHub
<#3273 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BDSHJ6LCUF4NYKMGN24WNM32TPRV5AVCNFSM6AAAAABXUWB6MWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMBYG4YDKNJQHA>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
This is for the |
Describe the bug
The
deleteChatMessage
GraphQL mutation (intest/routes/graphql/Mutation/deleteChatMessage.test.ts
) uses mocked tests, contradicting the project's requirement for integration testing via real API calls for all mutations in that directory.Expected Behavior
All GraphQL mutations, including
deleteChatMessage
, should be thoroughly integration tested using actual API calls. Mutation files should reside intest/routes/Graphql/Mutation/*
.Actual Behavior
deleteChatMessage
uses mocked tests, bypassing the real API and services.Impact
Mocked tests provide a false sense of security, risking undetected bugs and regressions. Inconsistent testing hinders maintainability.
Potential internship candidates
Please read this if you are planning to apply for a Palisadoes Foundation internship
The text was updated successfully, but these errors were encountered: