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

Remove abstract OSGiTest class and dependency to CoreTest #458

Merged

Conversation

HeikoKlare
Copy link
Contributor

@HeikoKlare HeikoKlare commented Dec 21, 2023

This removes all dependencies to the to-be-removed JUnit 3-specific CoreTest class. To this end, the OSGiTest class, which only contains utility methods, is removed and the remaining utility fields and methods moved to the existing OSGiTestActivator class.

@HeikoKlare HeikoKlare force-pushed the remove-coretest_osgitest branch from 62743a7 to f2db26c Compare January 2, 2024 08:13
Copy link

github-actions bot commented Jan 2, 2024

Test Results

   25 files  ±0     25 suites  ±0   10m 56s ⏱️ -28s
2 156 tests ±0  2 110 ✅ ±0  46 💤 ±0  0 ❌ ±0 
2 200 runs  ±0  2 154 ✅ ±0  46 💤 ±0  0 ❌ ±0 

Results for commit a5fe976. ± Comparison against base commit 8b33666.

♻️ This comment has been updated with latest results.

@akurtakov akurtakov force-pushed the remove-coretest_osgitest branch from f2db26c to 1fa6887 Compare January 10, 2024 15:10
@akurtakov
Copy link
Member

@HeikoKlare You should be able to continue here as all other PRs are merged in

@HeikoKlare HeikoKlare force-pushed the remove-coretest_osgitest branch from 1fa6887 to 824884d Compare January 12, 2024 11:21
@HeikoKlare
Copy link
Contributor Author

Thanks, @akurtakov! I've update this PR accordingly.

@HeikoKlare HeikoKlare marked this pull request as ready for review January 12, 2024 11:49
This removes all dependencies to the to-be-removed JUnit 3-specific
CoreTest class. To this end, the OSGiTest class, which only contains
utility methods, is removed and the remaining utility fields and methods
moved to the existing OSGiTestActivator class.
@akurtakov akurtakov force-pushed the remove-coretest_osgitest branch from 824884d to a5fe976 Compare January 12, 2024 11:54
@akurtakov akurtakov merged commit 4ddfedd into eclipse-equinox:master Jan 12, 2024
23 of 24 checks passed
@HeikoKlare HeikoKlare deleted the remove-coretest_osgitest branch January 12, 2024 17:11
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