Skip to content

Migrate QuickAccessDialogTest to JUnit 5#3911

Merged
vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:junit5/quickaccess-dialog-test-upstream
Apr 21, 2026
Merged

Migrate QuickAccessDialogTest to JUnit 5#3911
vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:junit5/quickaccess-dialog-test-upstream

Conversation

@vogella
Copy link
Copy Markdown
Contributor

@vogella vogella commented Apr 20, 2026

Replace JUnit 4 annotations and rule with JUnit 5 equivalents:

  • `@Before` / `@After` -> `@BeforeEach` / `@AfterEach`
  • `@Rule CloseTestWindowsRule` -> `@ExtendWith(CloseTestWindowsExtension.class)`
  • `org.junit.Assert` -> `org.junit.jupiter.api.Assertions` (message
    argument moved from first to last position per JUnit 5 signature)
  • `org.junit.Test` -> `org.junit.jupiter.api.Test`

No behavior change.

Test plan

  • `mvn clean verify -pl :org.eclipse.ui.tests -Pbuild-individual-bundles` runs the class on CI and all existing tests pass.

Replace JUnit 4 annotations and rule with JUnit 5 equivalents:
- @before / @after -> @beforeeach / @AfterEach
- @rule CloseTestWindowsRule -> @ExtendWith(CloseTestWindowsExtension.class)
- org.junit.Assert -> org.junit.jupiter.api.Assertions (message argument
  moved from first to last position per JUnit 5 signature)
- org.junit.Test -> org.junit.jupiter.api.Test

No behavior change; prepares the test for JUnit 5 features such as
@RepeatedTest, which will be used to investigate the flaky
testPreviousChoicesAvailableForExtension.
@github-actions
Copy link
Copy Markdown
Contributor

Test Results

   852 files  ±0     852 suites  ±0   56m 10s ⏱️ + 3m 57s
 7 899 tests ±0   7 656 ✅ ±0  243 💤 ±0  0 ❌ ±0 
20 199 runs  ±0  19 544 ✅ ±0  655 💤 ±0  0 ❌ ±0 

Results for commit cf1bb8e. ± Comparison against base commit 27696dd.

@vogella vogella merged commit 92e6244 into eclipse-platform:master Apr 21, 2026
18 checks passed
@vogella vogella deleted the junit5/quickaccess-dialog-test-upstream branch April 21, 2026 06:21
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.

1 participant