Convert maven-invoker-plugin tests to regular integration tests#5412
Draft
Copilot wants to merge 1 commit into
Draft
Convert maven-invoker-plugin tests to regular integration tests#5412Copilot wants to merge 1 commit into
Copilot wants to merge 1 commit into
Conversation
Copilot
AI
changed the title
[WIP] Convert invoker tests to "regular" integration tests
Convert maven-invoker-plugin tests to regular integration tests
Sep 30, 2025
Co-authored-by: laeubi <1331477+laeubi@users.noreply.github.com>
d02340d to
dfc915c
Compare
Test Results1 020 files 1 020 suites 5h 42m 50s ⏱️ For more details on these failures, see this check. Results for commit dfc915c. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes the issue by converting all maven-invoker-plugin tests to regular integration tests in the tycho-its module, providing a single consistent testing approach across the Tycho project.
Problem
The use of maven-invoker-plugin for integration tests had several drawbacks:
Solution
Converted all maven-invoker-plugin tests to standard JUnit integration tests in the tycho-its module following the established pattern.
Changes Made
tycho-buildtimestamp-jgit:
BuildTimestampJgitTest.javaextendingAbstractTychoIntegrationTesttycho-its/projects/buildtimestamp.jgit/src/it/directory with groovy scriptsUnzipFileutility classtycho-extras:
tycho-its:
plexus-archiverdependency for extracting test project zip filesBenefits
✅ Tests now appear as proper JUnit tests in CI reports and GitHub Actions
✅ Easier to debug with standard Java debugging tools
✅ Full Maven build logs visible in CI output
✅ Single consistent test pattern - all integration tests now in tycho-its
✅ Clear that integration tests only run after full Tycho build
✅ Removed 560+ lines of code (groovy scripts, invoker configs, etc.)
The conversion maintains full test coverage while improving developer experience and CI visibility.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.