You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The deleteAgendaItem GraphQL mutation (in test/graphql/types/Mutation/deleteAgendaItem) uses mocked tests, contradicting the project's requirement for integration testing via real API calls for all mutations in that directory. These mutation files are also incorrectly located; they belong in test/routes/Graphql/Mutation/*.
Expected Behavior
All GraphQL mutations, including deleteAgendaItem, should be thoroughly integration tested using actual API calls. Mutation files should reside in test/routes/Graphql/Mutation/*.
Actual Behavior
deleteAgendaItem 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
Describe the bug
The
deleteAgendaItem
GraphQL mutation (intest/graphql/types/Mutation/deleteAgendaItem
) uses mocked tests, contradicting the project's requirement for integration testing via real API calls for all mutations in that directory. These mutation files are also incorrectly located; they belong intest/routes/Graphql/Mutation/*
.Expected Behavior
All GraphQL mutations, including
deleteAgendaItem
, should be thoroughly integration tested using actual API calls. Mutation files should reside intest/routes/Graphql/Mutation/*
.Actual Behavior
deleteAgendaItem
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
The text was updated successfully, but these errors were encountered: