Skip to content

Migrate org.eclipse.jface.tests to JUnit 5#3708

Open
vogella wants to merge 1 commit intoeclipse-platform:masterfrom
vogella:migrate-jface-tests-junit5
Open

Migrate org.eclipse.jface.tests to JUnit 5#3708
vogella wants to merge 1 commit intoeclipse-platform:masterfrom
vogella:migrate-jface-tests-junit5

Conversation

@vogella
Copy link
Contributor

@vogella vogella commented Jan 27, 2026

Migrate remaining org.eclipse.jface.tests to JUnit 5

@github-actions
Copy link
Contributor

github-actions bot commented Jan 27, 2026

Test Results

 3 024 files   3 024 suites   2h 16m 50s ⏱️
 8 232 tests  7 984 ✅ 248 💤 0 ❌
23 520 runs  22 729 ✅ 791 💤 0 ❌

Results for commit 35d19e4.

♻️ This comment has been updated with latest results.

@vogella vogella force-pushed the migrate-jface-tests-junit5 branch 2 times, most recently from 2cfa6af to aeeceea Compare January 30, 2026 09:28
@vogella
Copy link
Contributor Author

vogella commented Jan 30, 2026

New compiler warning looks unrelated to me but I wait until someone else confirms.

Internal inconsistency: Inappropriate operand stack size encountered during translation

@vogella vogella force-pushed the migrate-jface-tests-junit5 branch from aeeceea to 9520cbc Compare February 2, 2026 09:20
@vogella vogella force-pushed the migrate-jface-tests-junit5 branch 4 times, most recently from 630a75b to d6fda42 Compare February 3, 2026 12:17
@vogella vogella force-pushed the migrate-jface-tests-junit5 branch from d6fda42 to 35d19e4 Compare February 5, 2026 11:38
@Test
public void testSimpleContentProposal() {
proposal = new ContentProposal(content);
assertEquals("1.0", content, proposal.getContent());
Copy link
Member

Choose a reason for hiding this comment

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

These messages "1.0" are totally useless. It's better to remove them. Here and in many other places.

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.

2 participants