Skip to content

Compute an auxiliary manifest for test launches with junit container#1832

Merged
laeubi merged 1 commit intoeclipse-pde:masterfrom
laeubi:compute_auxilliary_test_manifest
Jun 22, 2025
Merged

Compute an auxiliary manifest for test launches with junit container#1832
laeubi merged 1 commit intoeclipse-pde:masterfrom
laeubi:compute_auxilliary_test_manifest

Conversation

@laeubi
Copy link
Copy Markdown
Contributor

@laeubi laeubi commented Jun 22, 2025

Currently if the user uses a JUnit Classpath container one still needs to add import package/require bundle to the manifest to make these usable inside the running OSGi framework. This is not only cumbersome but also not desired if for example the project uses a testfolder inside the plugin. Also other libraries might be affected in a similar way if they come from other classpath containers.

This now detects if a JUnit Bundle Container is used and then compute an auxiliary manifest that is attached to the testprobe with a fragment, this is similar to how Tycho behaves already. With that it is now possible to much more smoothly use such setups and will enabled advanced use-cases in the future.

this is a polished variant extracted from

and can be merged already to make changes more focused and small.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 22, 2025

Test Results

   765 files  ±0     765 suites  ±0   55m 45s ⏱️ + 2m 58s
 3 611 tests ±0   3 535 ✅ +1   76 💤 ±0  0 ❌  - 1 
10 833 runs  ±0  10 602 ✅ +1  231 💤 ±0  0 ❌  - 1 

Results for commit aea7a5e. ± Comparison against base commit 386d0a5.

♻️ This comment has been updated with latest results.

Currently if the user uses a JUnit Classpath container one still needs
to add import package/require bundle to the manifest to make these
usable inside the running OSGi framework. This is not only cumbersome
but also not desired if for example the project uses a testfolder inside
the plugin. Also other libraries might be affected in a similar way if
they come from other classpath containers.

This now detects if a JUnit Bundle Container is used and then compute an
auxiliary manifest that is attached to the testprobe with a fragment,
this is similar to how Tycho behaves already. With that it is now
possible to much more smoothly use such setups and will enabled advanced
use-cases in the future.
@laeubi laeubi force-pushed the compute_auxilliary_test_manifest branch from 71afba2 to aea7a5e Compare June 22, 2025 16:36
@laeubi laeubi merged commit f702744 into eclipse-pde:master Jun 22, 2025
19 checks passed
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