PatternWin32ManualTest: suppress Discouraged access warning

[jukzi]

The type 'DPIUtil' is not API (restriction on required project 'org.eclipse.swt.win32.win32.x86_64')

[github-actions]

Test Results

   383 files  ±0     383 suites  ±0   5m 5s ⏱️ +15s
 4 286 tests ±0   4 272 ✅ ±0  14 💤 ±0  0 ❌ ±0 
12 150 runs  ±0  12 065 ✅ ±0  85 💤 ±0  0 ❌ ±0 

Results for commit 423d30b. ± Comparison against base commit 3c1065b.

♻️ This comment has been updated with latest results.

[jukzi]

CI complains about 10 unnecessary @SuppressWarnings("restriction") - what has changed?

[jukzi]

CI complains about 10 unnecessary

created #1648

[jukzi]

3 Failing Test_org_eclipse_swt_browser_Browser are documented in #1564,
Warnings are gone since #1650

However i wonder that the warnings did not decrease by 1 with this PR. The warning is not shown in the CI build on master

[akurtakov]

That's because the plugin is not enabled in maven build. If you look at

eclipse.platform.swt/pom.xml

Line 233 in 3c1065b

<module>tests/org.eclipse.swt.tests</module>

there is only one test bundle enabled.

[jukzi]

Is it possible and good to add org.eclipse.swt.tests.win32 there?

[akurtakov]

From releng POV it would probably have to be added in a maven profile as IIRC it uses SWT methods that are internal for the win32 port.
With the above said it's up to people with windows access to evaluate whether it would be beneficial for GHA as Jenkins runs on Linux thus the module will be skipped there.

[jukzi]

lets keep it simple by now: i know the warning is reduced in my workspace - lets merge?

[akurtakov]

Merge it.