Skip to content

Conversation

@vogella
Copy link
Contributor

@vogella vogella commented Dec 28, 2025

Migrates 10 tests in org.eclipse.ui.tests from CloseTestWindowsRule (JUnit 4) to CloseTestWindowsExtension (JUnit 5).

Changes include:

  • Replacing CloseTestWindowsRule with CloseTestWindowsExtension.
  • Updating imports to JUnit 5.
  • Converting annotations: @Rule -> @ExtendWith, @Ignore -> @Disabled, @Before -> @BeforeEach.
  • Updating assertions to match JUnit 5 argument order (message last).

Tests migrated:

  • Bug43321Test
  • Bug53489Test
  • Bug44460Test
  • Bug41931Test
  • ActionExpressionTest
  • Bug40023Test
  • Bug36537Test
  • PerspectiveSwitcherTest
  • TextHandlerTest
  • StickyViewManagerTest

Migrated 10 tests in org.eclipse.ui.tests from using CloseTestWindowsRule
(JUnit 4) to CloseTestWindowsExtension (JUnit 5).
Updates include replacing imports, annotations (@rule -> @ExtendWith,
@ignore -> @disabled, @before -> @beforeeach), and updating assertion
argument order.
public final CloseTestWindowsRule closeTestWindowsRule = new CloseTestWindowsRule();

@Rule
public final PreferenceMementoRule preferenceMemento = new PreferenceMementoRule();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This one should also be migrated to Junit 5

@vogella
Copy link
Contributor Author

vogella commented Dec 28, 2025

PreferenceMementoRule should also be migrated to Junit 5

@github-actions
Copy link
Contributor

Test Results

 3 015 files  ± 0   3 015 suites  ±0   2h 4m 3s ⏱️ - 11m 44s
 8 258 tests ± 0   8 009 ✅  -  1  248 💤 ±0  0 ❌ ±0  1 🔥 +1 
23 574 runs   - 24  22 780 ✅  - 27  791 💤 ±0  0 ❌ ±0  3 🔥 +3 

For more details on these errors, see this check.

Results for commit 11532d5. ± Comparison against base commit 447e516.

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