Fix, replace, or remove Cypress tests
Closed this issue · 4 comments
Currently, the Cypress tests don't run as is following the provided instructions. It seems like there is a dependency issue with cypress-plugin-snapshots
(which is unmaintained and not compatible with the latest cypress
major version). See meinaart/cypress-plugin-snapshots#215 about possible updates. But even if you pin it to the compatible version, many tests don't pass. This is likely either because the dump data changed or snapshots haven't been updated.
I'll add a note about the issues in the relevant docs, but at some point, it would be nice to get this working, replace it with something like Playwright, or just remove it if we don't plan to keep it up to date.
Additional cleanup:
-
There is also documentation that would be updated to remove this information on the testing page: https://github.com/tl-its-umich-edu/my-learning-analytics/blob/master/docs/testing.md
-
There is one line in package.json for a script to run
-
There is a cypress.json file on the root level that wouldn't be needed.
Remove the Cypress tests.
I see that they're removed, I don't see any references to cypress.