Fix | Close resources and skip AE tests by default

pom.xml

@@ -54,7 +54,7 @@
 			clientCertAuth  - - For tests requiring client certificate authentication setup (excluded by default)
 			- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - 
 			Default testing enabled with SQL Server 2019 (SQLv15) -->
-		<excludedGroups>xSQLv15,NTLM,MSI,reqExternalSetup,clientCertAuth</excludedGroups>
+		<excludedGroups>xSQLv12,xSQLv15,NTLM,MSI,reqExternalSetup,clientCertAuth</excludedGroups>
 
 		<!-- Use -preview for preview release, leave empty for official release.-->
 		<releaseExt>-preview</releaseExt>

src/main/java/com/microsoft/sqlserver/jdbc/SQLServerCertificateUtils.java

@@ -81,8 +81,8 @@ static KeyManager[] getKeyManagerFromFile(String certPath, String keyPath,
     private static KeyManager[] readPKCS12Certificate(String certPath,
             String keyPassword) throws NoSuchAlgorithmException, CertificateException, FileNotFoundException, IOException, UnrecoverableKeyException, KeyStoreException, SQLServerException {
         KeyStore keystore = KeyStore.getInstance(PKCS12_ALG);
-        try {
-            keystore.load(new FileInputStream(certPath), keyPassword.toCharArray());
+        try (FileInputStream certStream = new FileInputStream(certPath)) {
+            keystore.load(certStream, keyPassword.toCharArray());
         } catch (FileNotFoundException e) {
             throw new SQLServerException(SQLServerException.getErrString("R_clientCertError"), null, 0, null);
         }
@@ -203,10 +203,12 @@ private static PrivateKey loadPrivateKeyFromPVK(String keyPath,
         }
     }
 
-    private static Certificate loadCertificate(String certificatePem) throws IOException, GeneralSecurityException, SQLServerException {
+    private static Certificate loadCertificate(
+            String certificatePem) throws IOException, GeneralSecurityException, SQLServerException {
         CertificateFactory certificateFactory = CertificateFactory.getInstance("X509");
-        InputStream certstream = fileToStream(certificatePem);
-        return certificateFactory.generateCertificate(certstream);
+        try (InputStream certStream = fileToStream(certificatePem)) {
+            return certificateFactory.generateCertificate(certStream);
+        }
     }
 
     private static PrivateKey loadPrivateKey(String privateKeyPemPath,
@@ -268,24 +270,12 @@ private static BigInteger getBigInteger(ByteBuffer buffer, int length) {
     }
 
     private static InputStream fileToStream(String fname) throws IOException, SQLServerException {
-        FileInputStream fis = null;
-        DataInputStream dis = null;
-        try {
-            fis = new FileInputStream(fname);
-            dis = new DataInputStream(fis);
+        try (FileInputStream fis = new FileInputStream(fname); DataInputStream dis = new DataInputStream(fis)) {
             byte[] bytes = new byte[dis.available()];
             dis.readFully(bytes);
-            ByteArrayInputStream bais = new ByteArrayInputStream(bytes);
-            return bais;
-        } catch (FileNotFoundException e) { 
+            return new ByteArrayInputStream(bytes);
+        } catch (FileNotFoundException e) {
             throw new SQLServerException(SQLServerException.getErrString("R_clientCertError"), null, 0, null);
-        } finally {
-            if (null != dis) {
-                dis.close();
-            }
-            if (null != fis) {
-                fis.close();
-            }
         }
     }