-
Notifications
You must be signed in to change notification settings - Fork 29
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
Remove outdated tests #85
Labels
aspect-Testing
category.Chore
a task that needs to be done
p.Medium
to be done when there are no other higher priority issues
Comments
Issue is related to #72 |
chunweii
added
the
p.Medium
to be done when there are no other higher priority issues
label
Apr 13, 2023
@Eclipse-Dominator Are you working on this issue? If not you can unassign yourself and let others work on this |
This was referenced Feb 3, 2024
This was referenced Feb 19, 2024
This was referenced Mar 2, 2024
Hi, since the outdated tests have been refactored and the pre-push hook for npm run test is also enabled, I think we can close this issue now. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
aspect-Testing
category.Chore
a task that needs to be done
p.Medium
to be done when there are no other higher priority issues
There are many failing unit tests in WATcher. These tests are inherited from CATcher and require updating.
Reasons for failure fall under these 2 categories:
It should be relatively safe to fix category 1 since we are removing tests for files that don't exist anymore.
Fixes for category 2 should be done carefully, and should try to preserve tests cases as much as possible
To refactor test:
To remove test:
The text was updated successfully, but these errors were encountered: