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

Add OSGi test classpath support #1619

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

laeubi
Copy link
Contributor

@laeubi laeubi commented Feb 16, 2025

Similar to what we have for OSGi annotations, PDE should have OSGi Testing Support as it is a great library for testing OSGi applications. The most hindering thing in this regard is that it is rater complex to setup until one can make the first steps.

This now adds a new classpath contributor that detects if a PDE project is already using JUNIT classpath container and then adds OSGi test dependencies automatically as test dependencies if they are part of the target platform or alternatively from the running platform.

See

Requires:

Copy link

github-actions bot commented Feb 16, 2025

Test Results

   279 files   -   6     279 suites   - 6   46m 18s ⏱️ +3s
 3 608 tests ±  0   3 531 ✅  -   1   76 💤 ± 0  1 ❌ +1 
10 822 runs   - 194  10 620 ✅  - 165  201 💤  - 30  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 5f07b7b. ± Comparison against base commit 338779a.

♻️ This comment has been updated with latest results.

@laeubi laeubi force-pushed the add_osgi_test_classpath_support branch 2 times, most recently from 0bafd3c to aef6d99 Compare March 11, 2025 14:31
@laeubi laeubi marked this pull request as ready for review March 11, 2025 14:31
Similar to what we have for OSGi annotations, PDE should have OSGi
Testing Support as it is a great library for testing OSGi applications.
The most hindering thing in this regard is that it is rater complex to
setup until one can make the first steps.

This now adds a new classpath contributor that detects if a PDE project
is already using JUNIT classpath container and then adds OSGi test
dependencies automatically as test dependencies if they are part of the
target platform or alternatively from the running platform.

See eclipse-pde#877
@laeubi laeubi force-pushed the add_osgi_test_classpath_support branch from aef6d99 to 5f07b7b Compare March 11, 2025 14:33
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