Skip to content

Conversation

@vogella
Copy link
Contributor

@vogella vogella commented Jan 11, 2026

Migrate the ConformanceTestSuite and its dependent conformance test classes in org.eclipse.jface.tests.databinding to JUnit 5.
Verified with local tests.

@vogella vogella force-pushed the migrate-jface-databinding-conformance-junit5 branch from 34c11eb to 6b7b229 Compare January 11, 2026 14:35
@vogella vogella marked this pull request as draft January 11, 2026 14:35
@github-actions
Copy link
Contributor

github-actions bot commented Jan 11, 2026

Test Results

 2 502 files   -   513   2 502 suites   - 513   2h 5m 34s ⏱️ - 1m 32s
 7 189 tests  - 1 069   6 943 ✅  - 1 067  246 💤  - 2  0 ❌ ±0 
20 391 runs   - 3 207  19 606 ✅  - 3 201  785 💤  - 6  0 ❌ ±0 

Results for commit a95e1d5. ± Comparison against base commit e81ba1c.

This pull request removes 1069 tests.
org.eclipse.core.tests.databinding.AggregateValidationStatusTest ‑ testAggregateValidationStatusValueType
org.eclipse.core.tests.databinding.AggregateValidationStatusTest ‑ testConstructor_DefaultRealm
org.eclipse.core.tests.databinding.BindingTest ‑ testDisposeBindingThenModel
org.eclipse.core.tests.databinding.BindingTest ‑ testDisposeBindingThenObservables
org.eclipse.core.tests.databinding.BindingTest ‑ testDisposeBindingThenTarget
org.eclipse.core.tests.databinding.BindingTest ‑ testDisposeModelDisposesBinding
org.eclipse.core.tests.databinding.BindingTest ‑ testDisposeModelThenBinding
org.eclipse.core.tests.databinding.BindingTest ‑ testDisposeObservablesThenBinding
org.eclipse.core.tests.databinding.BindingTest ‑ testDisposeTargetDisposesBinding
org.eclipse.core.tests.databinding.BindingTest ‑ testDisposeTargetThenBinding
…

♻️ This comment has been updated with latest results.

- Migrate org.eclipse.jface.tests.databinding.conformance tests to JUnit 5.
- Update ConformanceTestSuite to use @testfactory.
- Update MANIFEST.MF to import junit.jupiter.api.function.
@vogella vogella force-pushed the migrate-jface-databinding-conformance-junit5 branch from 6b7b229 to a95e1d5 Compare January 12, 2026 09:03
@vogella vogella marked this pull request as ready for review January 12, 2026 09:03
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