RFR: 8298108: Add a regression test for JDK-8297684 [v3]
Sean Mullan
mullan at openjdk.org
Mon Dec 5 21:09:53 UTC 2022
On Mon, 5 Dec 2022 17:56:49 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:
>> Please review this test addition as it would have helped discover a regression when https://bugs.openjdk.org/browse/JDK-8269039 got introduced and subsequently backported. What's more, it might help discover similar issues going forward. Thoughts?
>>
>> Testing: Manually produced a build without [JDK-8269039](https://bugs.openjdk.org/browse/JDK-8269039). Test fails without the fix, passes after.
>
> Severin Gehwolf has updated the pull request incrementally with one additional commit since the last revision:
>
> Remove unused Path arg for test invocation
A regression test was added as part of the fix for https://bugs.openjdk.org/browse/JDK-8280890. See the commit log for more details: https://github.com/openjdk/jdk/commit/a0f6f2409ea61ff9ed9dc2e2b46e309c751d456d#diff-6de8303e80bab41e34a29b272cff64a68d3b255f0f147ee16ef21680abf7f41a
Since this is a different way of testing the same underlying issue and the problem shows up in more than one way, I agree it is a good idea to add another regression test. However, I would put in the same test directory as 8280890: test/jdk/java/security/SignedJar
-------------
PR: https://git.openjdk.org/jdk/pull/11515
More information about the security-dev
mailing list