RFR: JDK-8304063: tools/jpackage/share/AppLauncherEnvTest.java fails when checking LD_LIBRARY_PATH [v2]

Matthias Baesken mbaesken at openjdk.org
Thu Mar 16 09:08:10 UTC 2023


On Wed, 15 Mar 2023 18:37:51 GMT, Alexey Semenyuk <asemenyuk at openjdk.org> wrote:

>> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   redo check
>
> test/jdk/tools/jpackage/share/AppLauncherEnvTest.java line 93:
> 
>> 91:         } else {
>> 92:             TKit.assertEquals(expectedEnvVarValue, actualEnvVarValue, msg);
>> 93:         }
> 
> I'd keep the check as a single statement:
> 
> TKit.assertTextStream(expectedEnvVarValue)
>     .predicate(TKit.isLinux() ? String::endsWith : String::equals)
>     .label(String.format("value of %s env variable", envVarName))
>     .apply(Stream.of(actualEnvVarValue));
> 
> 
> It also will produce a nicer log record than `TKit.assertTrue(false, msg);`

Hi Alexey, I adjusted the check following your advice .

-------------

PR: https://git.openjdk.org/jdk/pull/13041


More information about the core-libs-dev mailing list