test: add integration test for modular processor with shared dependencies

[mariuszs]

Integration test for #1039.

The underlying issue (same GAV with different types resolved to a single path) is being fixed in Maven core via apache/maven#11805. This PR adds a regression test only — no compiler plugin code changes.

[desruisseaux]

I think that the analysis is correct and this pull request seems to address the issue, but as a workaround. Ideally, it seems to me that the issue should be addressed in Maven core or in Maven resolver. It would allow to add only the dependencies that were really requested, instead of adding all dependencies that could potentially have been requested.

I was hopping that apache/maven#11380 would help, but maybe it is only a first step. If I'm guessing correctly (I have not yet verified with a step-by-step debugging), the above-cited PR 11380 fixes the type of transitive dependencies when the type is only modular-processor, but we still have the issue that if a dependency is declared with both modular-processor and modular-jar types, only one of those types win while actually we need to keep both. @cstamas do yo think that this analysis is correct?

[cstamas]

Yes, it is. Using same GAV with different types is no-go currently.

[desruisseaux]

Yes, it is. Using same GAV with different types is no-go currently.

So, it is correct to said that apache/maven#11805 tries to make that possible?

[desruisseaux]

For information, I verified that when using Maven 4.1.0-SNAPSHOT with the apache/maven#11805 patch applied, the test provided in this pull request passes with not need to change anything in the compiler code (i.e. with ToolExecutor changes reverted).

[desruisseaux]

Actually, instead of closing it, I was thinking to just discard the changes in ExecutorTool and keep the integration test.

[mariuszs]

Done.

[desruisseaux]

I pushed a maven-4.1 branch. Can you edit this pull request for targeting that branch? This is because the master branch targets Maven 4.0.x, while this test requires Maven 4.1.