Merge pull request #1785 from marklogic/feature/17505-test-fix

MLE-17505 Updated test for MarkLogic 12

Author: rjrudin

marklogic-client-api/src/test/java/com/marklogic/client/test/ssl/OneWaySSLTest.java

@@ -13,7 +13,10 @@
 import com.marklogic.client.test.junit5.RequiresML12;
 import com.marklogic.mgmt.ManageClient;
 import com.marklogic.mgmt.resource.appservers.ServerManager;
-import org.junit.jupiter.api.*;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.condition.DisabledOnJre;
 import org.junit.jupiter.api.condition.JRE;
 import org.junit.jupiter.api.extension.ExtendWith;
@@ -119,7 +122,9 @@ void noSslContext() {
 
 		DatabaseClient.ConnectionResult result = client.checkConnection();
 		assertEquals("Forbidden", result.getErrorMessage(), "MarkLogic is expected to return a 403 Forbidden when the " +
-			"user tries to access an HTTPS app server using HTTP");
+			"user tries to access an HTTPS app server using HTTP. This behavior changes in MarkLogic 12, and it may " +
+			"be considered a bit surprising with MarkLogic 11 and earlier - that is, the user probably shouldn't get " +
+			"any response back since a connection cannot be made without using SSL.");
 		assertEquals(403, result.getStatusCode());
 
 		ForbiddenUserException ex = assertThrows(ForbiddenUserException.class,
@@ -133,6 +138,17 @@ void noSslContext() {
 		);
 	}
 
+	@ExtendWith(RequiresML12.class)
+	@Test
+	void noSslContextWithMarkLogic12() {
+		DatabaseClient client = Common.newClientBuilder().build();
+
+		MarkLogicIOException ex = assertThrows(MarkLogicIOException.class, () -> client.checkConnection());
+		assertTrue(ex.getMessage().contains("unexpected end of stream"), "Per MLE-17505, a change in the openssl " +
+			"library used by the server results in an IO exception when the client tries to connect to an " +
+			"app server that requires SSL, but the client does not use SSL. Actual message: " + ex.getMessage());
+	}
+
 	@Test
 	void tLS13ClientWithTLS12Server() {
 		DatabaseClient client = buildTrustAllClientWithSSLProtocol(SSLUtil.DEFAULT_PROTOCOL);