[MBUILDCACHE-88] Tests in failure when ran on jdk21

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.1.0
• Fix Version/s: 1.2.0
• Labels: pull-request-available
• Assignee: Olivier Lamy
• Reporter: Kevin Buntrock
• Created: 22/Apr/24 21:05
• Updated: 03/May/24 01:55
• Resolved: 03/May/24 01:55

Description

The project tests cannot be run on jdk21. Result is :

[INFO]
[INFO] Results:
[INFO]
[ERROR] Failures:
[ERROR]   CacheConfigImplTest.testInitializationDisabledInXML:234 expected: <DISABLED> but was: <INITIALIZED>
[ERROR]   CacheConfigImplTest.testRemoteDisableByUserPropertyOverride:330->assertDefaults:137->assertDefaults:201->lambda$testRemoteDisableByUserPropertyOverride$39:330 expected: <dummy.url.xyz> but was: <null>
[ERROR]   CacheConfigImplTest.testRemoteEnableByUserPropertyOverrideWithURL:313->assertDefaults:137->assertDefaults:201->lambda$testRemoteEnableByUserPropertyOverrideWithURL$38:315 expected: <true> but was: <false>
[ERROR]   CacheConfigImplTest.testRemoteEnableInXMLWithURL:288->assertDefaults:137->assertDefaults:201->lambda$testRemoteEnableInXMLWithURL$36:290 expected: <true> but was: <false>
[ERROR]   CacheConfigImplTest.testRemoteSaveIgnoredWhenRemoteDisabledByUserPropertyOverride:420->assertDefaults:137->assertDefaults:201->lambda$testRemoteSaveIgnoredWhenRemoteDisabledByUserPropertyOverride$48:420 expected: <dummy.url.xyz> but was: <null>
[ERROR]   CacheConfigImplTest.testRemoveSaveDisabledByUserProperty:381->assertDefaults:137->assertDefaults:201->lambda$testRemoveSaveDisabledByUserProperty$47:383 expected: <true> but was: <false>
[ERROR]   CacheConfigImplTest.testRemoveSaveEnabledByUserProperty:362->assertDefaults:137->assertDefaults:201->lambda$testRemoveSaveEnabledByUserProperty$45:365 expected: <true> but was: <false>
[ERROR]   CacheConfigImplTest.testRemoveSaveEnabledInXML:344->assertDefaults:137->assertDefaults:201->lambda$testRemoveSaveEnabledInXML$42:347 expected: <true> but was: <false>
[ERROR]   CacheConfigImplTest.testRemoveSaveFinalEnabledByUserProperty:436->assertDefaults:137->assertDefaults:201->lambda$testRemoveSaveFinalEnabledByUserProperty$51:439 expected: <true> but was: <false>
[ERROR]   CacheConfigImplTest.testRemoveSaveFinalIgnoredWhenRemoteSaveDisabled:455->assertDefaults:137->assertDefaults:201->lambda$testRemoveSaveFinalIgnoredWhenRemoteSaveDisabled$54:457 expected: <true> but was: <false>
[INFO]
[ERROR] Tests run: 71, Failures: 10, Errors: 0, Skipped: 4
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------

In class "CacheConfigImplTest", a method "deepMockConfigFile" mocks the result of the call to java.nio.file.Files.exists (via "FileSystemProvider.checkAccess").

In jdk21 version, "Files.exists" does not rely on the same underlying "FileSystemProvider" method, therefore breaking the mocking purpose.

Issue Links

• links to: GitHub Pull Request #147