Closed dscpsyl closed 1 year ago
Should we just delete the code to avoid confusion for future developers or commenting out the code?
Yes. There’s very few cases where commented code is beneficial to readers, and this is not one of them.
That is my bad. This PR is not ready yet. I'll close this PR and recreate one. There's still more testing to be done and I'm still experimenting
Overview
In this PR, the test job function is removed from the user-facing front end. It is still available in the Swagger backend.
Screenshots (Optional)
Validation (Optional)
Tests
npm test
) passnpm run coverage
) 100%npx stryker run
) 100%npx eslint --fix src
)Linked Issues
Closes #25