Skip to content

[UI, #230] Implement undo delete test case functionality#484

Merged
arksap2002 merged 2 commits intodevelopmentfrom
danielrendox/feature/undo_delete_test_case
May 2, 2025
Merged

[UI, #230] Implement undo delete test case functionality#484
arksap2002 merged 2 commits intodevelopmentfrom
danielrendox/feature/undo_delete_test_case

Conversation

@DanielRendox
Copy link
Collaborator

Description of changes made

Closes #230

  • I have checked that I am merging into correct branch

@github-actions
Copy link

🚨 PR Title Needs Formatting

Please specify the modified module and, preferrably, include a GitHub or YouTrack issue that this PR closes in the title. By module, we mean any logical part of the project — it doesn’t have to be a Gradle module or anything like that.
Use the following format: [<Module(s)>] and(or) [<ISSUE-ID>] brief title. Examples:

  • [LLM] Add support for Google AI as an LLM
  • [Build] Update IntelliJ Platform version
  • [#290] Fix bug with LLM response parsing`
  • [PromptBuilder] [JBRes-2860] Refactor the PromptBuilder component

@DanielRendox DanielRendox changed the title Implement undo delete test case functionality [UI, #230] Implement undo delete test case functionality Apr 10, 2025
Copy link
Collaborator

@stephanlukasczyk stephanlukasczyk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While I agree on the changes in general, one conceptual question, for which consider the following scenario:

I have generated tests and executed them. Now the "Run all" button is disabled (so far, so good). Afterwards, I delete one of the test cases (permanently, but does not matter IMHO). My assumption now was that I should be able to rerun the tests again, because presumably coverage might have changed. However, that is not possible, the button is still disabled.

IMHO allowing re-execution of the tests in such a scenario is independent of this PR, however, I would like to know, what do you think @DanielRendox @pderakhshanfar about activating the run buttons in such a case?

@DanielRendox
Copy link
Collaborator Author

@stephanlukasczyk Agreed, sounds reasonable

@arksap2002 arksap2002 force-pushed the danielrendox/feature/undo_delete_test_case branch from 39c5e7c to 6f327c3 Compare May 2, 2025 16:53
@arksap2002 arksap2002 merged commit e2ca277 into development May 2, 2025
3 checks passed
@arksap2002 arksap2002 deleted the danielrendox/feature/undo_delete_test_case branch May 2, 2025 22:39
vsantele pushed a commit that referenced this pull request May 23, 2025
* Implement undo delete test case functionality

* Fix ktlint issues
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add undo to test case removing

3 participants