eclipse-platform / eclipse.platform

https://eclipse.dev/eclipse/
Eclipse Public License 2.0
84 stars 113 forks source link

Migrate resources propertytester tests to JUnit 5 #903 #1555

Closed HeikoKlare closed 1 month ago

HeikoKlare commented 2 months ago

Migrates the tests in org.eclipse.core.tests.interal.propertytester to JUnit 5.

Contributes to https://github.com/eclipse-platform/eclipse.platform/issues/903

github-actions[bot] commented 2 months ago

Test Results

 1 758 files  ± 0   1 758 suites  ±0   1h 26m 29s :stopwatch: +51s  4 170 tests +12   4 148 :white_check_mark: +12   22 :zzz: ±0  0 :x: ±0  13 107 runs  +36  12 943 :white_check_mark: +36  164 :zzz: ±0  0 :x: ±0 

Results for commit 50cc2d04. ± Comparison against base commit 1016b364.

This pull request removes 4 and adds 16 tests. Note that renamed tests count towards both. ``` AutomatedResourceTests AllPropertytesterTests FilePropertyTesterTest ‑ testExistingNsRootElementFile AutomatedResourceTests AllPropertytesterTests FilePropertyTesterTest ‑ testExistingTextFile AutomatedResourceTests AllPropertytesterTests FilePropertyTesterTest ‑ testNonExistingNsRootElementFile AutomatedResourceTests AllPropertytesterTests FilePropertyTesterTest ‑ testNonExistingTextFile ``` ``` AutomatedResourceTests AllPropertytesterTests FilePropertyTesterTest ‑ testExistingNsRootElementFile(List)[1] [] AutomatedResourceTests AllPropertytesterTests FilePropertyTesterTest ‑ testExistingNsRootElementFile(List)[2] [kindOf] AutomatedResourceTests AllPropertytesterTests FilePropertyTesterTest ‑ testExistingNsRootElementFile(List)[3] [useFilenameOnly] AutomatedResourceTests AllPropertytesterTests FilePropertyTesterTest ‑ testExistingNsRootElementFile(List)[4] [kindOf, useFilenameOnly] AutomatedResourceTests AllPropertytesterTests FilePropertyTesterTest ‑ testExistingTextFile(List)[1] [] AutomatedResourceTests AllPropertytesterTests FilePropertyTesterTest ‑ testExistingTextFile(List)[2] [kindOf] AutomatedResourceTests AllPropertytesterTests FilePropertyTesterTest ‑ testExistingTextFile(List)[3] [useFilenameOnly] AutomatedResourceTests AllPropertytesterTests FilePropertyTesterTest ‑ testExistingTextFile(List)[4] [kindOf, useFilenameOnly] AutomatedResourceTests AllPropertytesterTests FilePropertyTesterTest ‑ testNonExistingNsRootElementFile(List)[1] [] AutomatedResourceTests AllPropertytesterTests FilePropertyTesterTest ‑ testNonExistingNsRootElementFile(List)[2] [kindOf] … ```

:recycle: This comment has been updated with latest results.