This is an automated email from the ASF dual-hosted git repository. dsoumis pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git
commit 12a260a0d2f0f711b68454f9a36d02fe8fe8596f Author: Dimitris Soumis <[email protected]> AuthorDate: Thu Oct 9 21:40:25 2025 +0300 Refactor multiple instances of same code block in TestOcspIntegration --- .../tomcat/util/net/ocsp/TestOcspIntegration.java | 44 ++++++++++------------ 1 file changed, 20 insertions(+), 24 deletions(-) diff --git a/test/org/apache/tomcat/util/net/ocsp/TestOcspIntegration.java b/test/org/apache/tomcat/util/net/ocsp/TestOcspIntegration.java index 71effa10b6..8b8d0e62d6 100644 --- a/test/org/apache/tomcat/util/net/ocsp/TestOcspIntegration.java +++ b/test/org/apache/tomcat/util/net/ocsp/TestOcspIntegration.java @@ -118,21 +118,13 @@ public class TestOcspIntegration extends TomcatBaseTest { } @Test public void testOcspGood_Mutual() throws Exception { - final int ocspResponderPortForClient = 8889; - Assume.assumeTrue("Port " + ocspResponderPortForClient + " is not available.", isPortAvailable(ocspResponderPortForClient)); - try (FakeOcspResponder fakeOcspResponder = new FakeOcspResponder(Files.readAllBytes(new File(getPath(OCSP_CLIENT_CERT_GOOD_RESPONSE)).toPath()), ocspResponderPortForClient)){ - fakeOcspResponder.start(); - Assert.assertEquals(HttpServletResponse.SC_OK, testOCSP(OCSP_SERVER_CERT_GOOD_RESPONSE, true, true, ffm)); - } + testOCSPWithClientResponder(OCSP_CLIENT_CERT_GOOD_RESPONSE, + () -> Assert.assertEquals(HttpServletResponse.SC_OK, testOCSP(OCSP_SERVER_CERT_GOOD_RESPONSE, true, true, ffm))); } @Test public void testOcspGood_ServerVerifiesClientCertificateOnly() throws Exception { - final int ocspResponderPortForClient = 8889; - Assume.assumeTrue("Port " + ocspResponderPortForClient + " is not available.", isPortAvailable(ocspResponderPortForClient)); - try (FakeOcspResponder fakeOcspResponder = new FakeOcspResponder(Files.readAllBytes(new File(getPath(OCSP_CLIENT_CERT_GOOD_RESPONSE)).toPath()), ocspResponderPortForClient)){ - fakeOcspResponder.start(); - Assert.assertEquals(HttpServletResponse.SC_OK, testOCSP(OCSP_SERVER_CERT_REVOKED_RESPONSE, true, false, ffm)); - } + testOCSPWithClientResponder(OCSP_CLIENT_CERT_GOOD_RESPONSE, + () -> Assert.assertEquals(HttpServletResponse.SC_OK, testOCSP(OCSP_SERVER_CERT_REVOKED_RESPONSE, true, false, ffm))); } @Test(expected = CertificateRevokedException.class) public void testOcspRevoked_ClientVerifiesServerCertificateOnly() throws Exception { @@ -153,21 +145,13 @@ public class TestOcspIntegration extends TomcatBaseTest { } @Test(expected = SSLHandshakeException.class) public void testOcspRevoked_ServerVerifiesClientCertificateOnly() throws Exception { - final int ocspResponderPortForClient = 8889; - Assume.assumeTrue("Port " + ocspResponderPortForClient + " is not available.", isPortAvailable(ocspResponderPortForClient)); - try (FakeOcspResponder fakeOcspResponder = new FakeOcspResponder(Files.readAllBytes(new File(getPath(OCSP_CLIENT_CERT_REVOKED_RESPONSE)).toPath()), ocspResponderPortForClient)){ - fakeOcspResponder.start(); - testOCSP(OCSP_SERVER_CERT_GOOD_RESPONSE, true, false, ffm); - } + testOCSPWithClientResponder(OCSP_CLIENT_CERT_REVOKED_RESPONSE, + () -> testOCSP(OCSP_SERVER_CERT_GOOD_RESPONSE, true, false, ffm)); } @Test public void testOcsp_NoVerification() throws Exception { - final int ocspResponderPortForClient = 8889; - Assume.assumeTrue("Port " + ocspResponderPortForClient + " is not available.", isPortAvailable(ocspResponderPortForClient)); - try (FakeOcspResponder fakeOcspResponder = new FakeOcspResponder(Files.readAllBytes(new File(getPath(OCSP_CLIENT_CERT_REVOKED_RESPONSE)).toPath()), ocspResponderPortForClient)){ - fakeOcspResponder.start(); - Assert.assertEquals(HttpServletResponse.SC_OK, testOCSP(OCSP_SERVER_CERT_REVOKED_RESPONSE, false, false, ffm)); - } + testOCSPWithClientResponder(OCSP_CLIENT_CERT_REVOKED_RESPONSE, + () -> Assert.assertEquals(HttpServletResponse.SC_OK, testOCSP(OCSP_SERVER_CERT_REVOKED_RESPONSE, false, false, ffm))); } @Test public void testOcspResponderUrlDiscoveryViaCertificateAIA() throws Exception { @@ -176,6 +160,18 @@ public class TestOcspIntegration extends TomcatBaseTest { Assert.assertEquals(HttpServletResponse.SC_OK, testOCSP(OCSP_SERVER_CERT_GOOD_RESPONSE, false, true, ffm, true, ocspPort)); } + @FunctionalInterface + private interface TestOCSPAction { + void execute() throws Exception; + } + private void testOCSPWithClientResponder(String clientResponsePath, TestOCSPAction testOCSPAction) throws Exception { + final int ocspResponderPortForClient = 8889; + Assume.assumeTrue("Port " + ocspResponderPortForClient + " is not available.", isPortAvailable(ocspResponderPortForClient)); + try (FakeOcspResponder fakeOcspResponder = new FakeOcspResponder(Files.readAllBytes(new File(getPath(clientResponsePath)).toPath()), ocspResponderPortForClient)){ + fakeOcspResponder.start(); + testOCSPAction.execute(); + } + } private int testOCSP(String pathToOcspResponse, boolean serverSideVerificationEnabled, boolean clientSideOcspVerificationEnabled, boolean ffm) throws Exception { return testOCSP(pathToOcspResponse, serverSideVerificationEnabled, clientSideOcspVerificationEnabled, ffm, false, 0); --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
