Add Assert.fail() to test that should expected an exception

java/flight/flight-jdbc-driver/src/test/java/org/apache/arrow/driver/jdbc/test/ConnectionTest.java

@@ -38,6 +38,7 @@
 import org.apache.arrow.memory.RootAllocator;
 import org.apache.arrow.util.AutoCloseables;
 import org.junit.After;
+import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
 
@@ -149,7 +150,7 @@ public void testUnencryptedConnectionWithEmptyHost()
 
     try (Connection connection = DriverManager
         .getConnection(invalidUrl, properties)) {
-      // Shouldn't reach this since we are expecting an Exception.
+      Assert.fail();
     }
   }
 
@@ -188,7 +189,7 @@ public void testUnencryptedConnectionProvidingInvalidPort()
 
     try (Connection connection = DriverManager
         .getConnection(invalidUrl, properties)) {
-      // Shouldn't reach this since we are expecting an Exception.
+      Assert.fail();
     }
   }
 
@@ -244,7 +245,7 @@ public void testUnencryptedConnectionShouldThrowExceptionWhenProvidedWithInvalid
 
     try (Connection connection = DriverManager.getConnection(serverUrl,
         properties)) {
-      // Shouldn't reach this.
+      Assert.fail();
     }
   }
 }

java/flight/flight-jdbc-driver/src/test/java/org/apache/arrow/driver/jdbc/test/ConnectionTlsTest.java

@@ -39,6 +39,7 @@
 import org.apache.arrow.util.AutoCloseables;
 import org.apache.calcite.avatica.org.apache.http.auth.UsernamePasswordCredentials;
 import org.junit.After;
+import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
 
@@ -159,7 +160,7 @@ public void testGetEncryptedClientWithNoCertificateOnKeyStore() throws Exception
             allocator, flightTestUtils.getLocalhost(), this.tlsServer.getPort(),
             null, noCertificateKeyStorePath,
             noCertificateKeyStorePassword)) {
-      // Shouldn't reach this since we are expecting an Exception.
+      Assert.fail();
     }
   }
 
@@ -197,7 +198,7 @@ public void testGetEncryptedClientWithKeyStoreBadPasswordAndNoAuth() throws Exce
             allocator, flightTestUtils.getLocalhost(), this.tlsServer.getPort(),
             null, keyStorePath,
             keyStoreBadPassword)) {
-      // Shouldn't reach this since we are expecting an Exception.
+      Assert.fail();
     }
   }
 
@@ -243,7 +244,7 @@ public void testGetAuthenticatedEncryptedConnectionWithKeyStoreBadPassword() thr
 
     try (Connection connection = DriverManager
         .getConnection(serverUrl, properties)) {
-      // Shouldn't reach this since we are expecting an Exception.
+      Assert.fail();
     }
   }