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

Feature: Undo test case deletion #405

Open
wants to merge 3 commits into
base: development
Choose a base branch
from

Conversation

hoanglam-nguyen
Copy link

Description of changes made

This PR implements the ability for a user to undo the deletion of a test case if they change their mind. The general procedure is as follows:

  • When the user decides to delete a test case, instead of irrevocably removing the test from the UI and cache, the test is marked as deleted in the UI and the cache data is moved to a separate cache for "deleted" test cases
  • In place of the bottom panel of the "deleted" test, the user is presented with a button to undo this operation; the original test code is still visible for inspection
  • If the user decides to undo the deletion, the test together with its UI is enabled again, and the cache data is moved back to the original cache
  • The top bar with the current stats as well as the coverage indicators on the gutter are updated accordingly

Screenshot

undo-test-case-removal-example

Other notes

Closes #230

What is missing?

When the number of deleted test cases is large, they may clutter the list of test cases. In addition to an undo button, one could add a button to permanently remove the test case from the UI.

  • I have checked that I am merging into correct branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add undo to test case removing
2 participants