Fix for issues/533 (dual licensed dependency with one license on whitelist and the other on blacklist)

[wuwu2000]

See #533

If a project dependency has multiple licences I can choose the licence I want to use. So only 1 of the licences must be included in the whitelist.

[wuwu2000]

Hey @slawekjaranowski is something off with my pull request?

[Copilot]

Pull Request Overview

This PR fixes issue #533 by modifying the license validation logic to properly handle dual-licensed dependencies. When a dependency has multiple licenses, the new logic treats it as compliant if at least one of its licenses is whitelisted, even if another license appears on the blacklist.

Key Changes:

• Refactored the checkForbiddenLicenses() method to store unsafe licenses in a LicenseMap instead of a Set<String>, enabling project-level tracking
• Extracted duplicate whitelist checking logic into a new isDependencyWhitelisted() helper method
• Added comprehensive test coverage for various license validation scenarios including dual-licensed dependencies

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 9 comments.

File	Description
AbstractAddThirdPartyMojo.java	Refactored license checking logic to handle dual-licensed dependencies by tracking projects per license and validating each project individually against whitelist
AbstractAddThirdPartyMojoTest.java	Added comprehensive test suite covering whitelist/blacklist scenarios, including edge cases for dual-licensed dependencies
pom.xml	Added Mockito 5.20.0 test dependency to support the new test cases

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[wuwu2000]

thank you @slawekjaranowski for checking my pull request. I changed Britsh English licence to American English license in both files and downgraded mockito from 5 to 4 to be jdk 8 compatible.
Then I ran mvn --errors --batch-mode --show-version -D"invoker.streamLogsOnFailures" -P run-its verify with different JDKs to verify it's working, hopefully that way I verified all checks in advance

[wuwu2000]

Hello @slawekjaranowski
Do you plan to accept the pr? Is there something I can do for you? :)

[slawekjaranowski]

Hello @slawekjaranowski Do you plan to accept the pr? Is there something I can do for you? :)

I will try to look 😄

[slawekjaranowski]

merged with current master in order to run build