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

Make win32 embedded tests package protected #1303

Closed
wants to merge 1 commit into from

Conversation

merks
Copy link
Contributor

@merks merks commented Jun 25, 2024

Fixes #1298

Copy link
Contributor

Test Results

   422 files   - 48     422 suites   - 48   7m 41s ⏱️ - 1m 28s
 4 124 tests  - 11   4 116 ✅  - 11   8 💤 ±0  0 ❌ ±0 
16 325 runs   - 11  16 233 ✅  - 11  92 💤 ±0  0 ❌ ±0 

Results for commit 04c7622. ± Comparison against base commit f40502d.

This pull request removes 11 tests.
org.eclipse.swt.graphics.GCWin32Tests ‑ drawnElementsShouldScaleUpToTheRightZoomLevel
org.eclipse.swt.graphics.GCWin32Tests ‑ gcZoomLevelMustChangeOnShellZoomChange
org.eclipse.swt.graphics.PathWin32Tests ‑ testBoundsAreScaledWRTZoomLevel
org.eclipse.swt.graphics.PathWin32Tests ‑ testHandlesExistForEachZoomLevelInHashMap
org.eclipse.swt.graphics.PathWin32Tests ‑ testPathMustBeScaledOnZoomLevelChange
org.eclipse.swt.graphics.TextLayoutWin32Tests ‑ testGetBoundPublicAPIshouldReturnTheSameValueRegardlessOfZoomLevel
org.eclipse.swt.graphics.TransformWin32Tests ‑ testScaledTrasformMustHaveScaledValues
org.eclipse.swt.graphics.TransformWin32Tests ‑ testShouldHaveDifferentHandlesAtDifferentZoomLevels
org.eclipse.swt.widgets.ControlWin32Tests ‑ testDoNotScaleFontCorrectlyInNoAutoScaleSzenario
org.eclipse.swt.widgets.ControlWin32Tests ‑ testScaleFontCorrectlyInAutoScaleSzenario
…

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.

Unfortunately, I think this will not work as package-protected classes will not be picked as test classes by a JUnit 4 runner. GitHub also reports the tests as removed as they don't appear as executed in the logs:
image

My understanding is that a migration to JUnit 5 is required to allow the visibility reduction.

@vogella
Copy link
Contributor

vogella commented Jun 26, 2024

In the past we removed the exported API from test packages. Tests are not API (which was confirmed a longer time ago in the PMC IIRC (before Ed joined).

@merks
Copy link
Contributor Author

merks commented Jun 26, 2024

Then let's abandon this.

@merks merks closed this Jun 26, 2024
@merks merks deleted the issue-1298 branch June 26, 2024 13:02
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.

SWT tests for win32 have API errors
3 participants