On Thu, 23 Dec 2021 21:06:51 GMT, Valerie Peng <valer...@openjdk.org> wrote:
>> Hmm, this "ADD" case should be covered by existing regression tests. I will >> take a look. Thanks. > > Hmm, existing test does add an alias, the legacyMap.get() would returns null > upon 'aliasAlg' (type: String). To test this particular scenario, I enhanced > existing regression test to re-assign an alias and then check if the returned > impl is as expected. The proposed test patch is attached. If we are including > this test patch into this PR, then good to add the bug id to the test too. > [patch.txt](https://github.com/openjdk/jdk18/files/7771389/patch.txt) Thanks, @valeriepeng! Do you see the current JDK 18 failing with that test addition? Because this still passes for me without the fix from this PR: `CONF=linux-x86_64-server-fastdebug make run-test TEST=java/security/Provider/CaseSensitiveServices.java` ------------- PR: https://git.openjdk.java.net/jdk18/pull/70
