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

Return the maven location for missing junit bundles #2867

Merged

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Sep 28, 2023

Currently we only publish the meta-data, what works for most of the time if a junit bundle is missing, but for the case where the bundle name does not match the artifact key (e.g. junit4/hamcrest) this leads to Tycho not find the file and injecting a dummy file that can confuse maven-surefire-plugin.

This now adds a new IRawArtifactFileProvider provider for such missing bundles that is asked as last resort if no other part of the system can answer for a file.

Currently we only publish the meta-data, what works for most of the time
if a junit bundle is missing, but for the case where the bundle name
does not match the artifact key (e.g. junit4/hamcrest) this leads to
Tycho not find the file and injecting a dummy file that can confuse
maven-surefire-plugin.

This now adds a new IRawArtifactFileProvider provider for such missing
bundles that is asked as last resort if no other part of the system can
answer for a file.
@laeubi laeubi mentioned this pull request Sep 28, 2023
@github-actions
Copy link

Test Results

   561 files  ±0     561 suites  ±0   4h 21m 35s ⏱️ - 14m 35s
   363 tests ±0     357 ✔️ ±0    6 💤 ±0  0 ±0 
1 089 runs  ±0  1 070 ✔️ ±0  19 💤 ±0  0 ±0 

Results for commit 841421c. ± Comparison against base commit 03e790c.

@laeubi laeubi added the backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change label Sep 28, 2023
@laeubi laeubi merged commit fb32453 into eclipse-tycho:master Sep 28, 2023
10 checks passed
@github-actions
Copy link

💚 All backports created successfully

Status Branch Result
tycho-4.0.x

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant