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

Move specific FindReplaceDialog Tests into the separated test-class #1875

Merged

Conversation

Wittmaxi
Copy link
Contributor

@Wittmaxi Wittmaxi commented May 6, 2024

Some tests in FindReplaceUITest.java are not general to all find/replace interfaces and are specific for the find/replace dialog. This commit moves these tests into the dedicated test class and provides the necessary interface in FindReplaceUITest

Copy link
Contributor

github-actions bot commented May 6, 2024

Test Results

 1 812 files  +  304   1 812 suites  +304   1h 33m 48s ⏱️ + 29m 39s
 7 614 tests +    1   7 385 ✅ ±    0  228 💤 ±  0  1 ❌ +1 
23 994 runs  +1 841  23 244 ✅ +1 637  749 💤 +203  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 95439d7. ± Comparison against base commit 7372e60.

♻️ This comment has been updated with latest results.

@Wittmaxi Wittmaxi force-pushed the MW_moveDialogTestsToProperClas branch from 75460fa to 47cd1b1 Compare May 7, 2024 16:01
@Wittmaxi Wittmaxi requested a review from HeikoKlare May 7, 2024 16:02
Copy link
Contributor

@HeikoKlare HeikoKlare left a comment

Choose a reason for hiding this comment

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

Thank you!

Some tests in FindReplaceUITest.java are not general to all
find/replace interfaces and are specific for the find/replace dialog.
This commit moves these tests into the dedicated test class and provides
the necessary interface in FindReplaceUITest
@HeikoKlare HeikoKlare force-pushed the MW_moveDialogTestsToProperClas branch from 47cd1b1 to 95439d7 Compare May 13, 2024 09:41
@HeikoKlare
Copy link
Contributor

Failing test is unrelated and documented: #275

@HeikoKlare HeikoKlare merged commit deb75d4 into eclipse-platform:master May 13, 2024
14 of 16 checks passed
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.

None yet

2 participants