Skip to content
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 file test/routes/graphql/Mutation/updateChatMessage to integration test #3274

Open
Piyushk8 opened this issue Feb 22, 2025 · 0 comments
Labels
bug Something isn't working security Security fix test Testing application unapproved Unapproved for Pull Request

Comments

@Piyushk8
Copy link

Describe the bug

The updateChatMessage GraphQL mutation (in test/routes/graphql/Mutation/updateChatMessage.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 updateChatMessage, should be thoroughly integration tested using actual API calls. Mutation files should reside in test/routes/Graphql/Mutation/*.

Actual Behavior

updateChatMessage uses mocked tests, bypassing the real API and services. The file is in the wrong directory.

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

@Piyushk8 Piyushk8 added the bug Something isn't working label Feb 22, 2025
@github-actions github-actions bot added security Security fix test Testing application unapproved Unapproved for Pull Request labels Feb 22, 2025
@Piyushk8 Piyushk8 changed the title NoMongo: migrate file 'test/routes/graphql/updateChatMessage' to integration test NoMongo: migrate file test/routes/graphql/Mutation/updateChatMessage to integration test Feb 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working security Security fix test Testing application unapproved Unapproved for Pull Request
Projects
Status: Backlog
Status: Backlog
Development

No branches or pull requests

1 participant