On Sat, 20 Jul 2024 15:32:37 GMT, SendaoYan <s...@openjdk.org> wrote:
> Hi all, > The testcase > `test/jdk/security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#actalisauthenticationrootca` > run Error, because the `@run` set conflicted with `/manual` and `/timeout`. > I think the `/timeout` not needed for manual test. > > Only this testcase has this issue, the search shell command is: > > for i in `grep "@run" test -rnl` ; do grep "/manual" $i | grep -q "/timeout" > && echo $i ; done > > > Only change the testcase, the change has been verified, no risk. It is fine to remove timeout from this test. IIRC, it was added quite long time back when we had issues with this external CA. ------------- Marked as reviewed by rhalade (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/20270#pullrequestreview-2192239080