From b1d9b3e53b6ee7cd87d7801542b8bb2f4d0055c2 Mon Sep 17 00:00:00 2001 From: lilgreenbird Date: Thu, 6 Feb 2025 22:06:29 -0800 Subject: [PATCH] Add "requireSecret" exclude tag for tests which require adding a secret to app registration (#2596) * add secretRequired tag * more * update * add test --- pom.xml | 110 ++++++++++++------ ...pySendTemporalDataTypesAsStringAETest.java | 1 + .../CallableStatementTest.java | 1 + .../jdbc/AlwaysEncrypted/EnclaveTest.java | 1 + .../JDBCEncryptionDecryptionTest.java | 1 + .../jdbc/AlwaysEncrypted/MSITest.java | 1 + .../AlwaysEncrypted/MultiUserAKVTest.java | 1 + .../ParameterMetaDataCacheTest.java | 1 + .../AlwaysEncrypted/PrecisionScaleTest.java | 1 + .../RegressionAlwaysEncryptedTest.java | 1 + .../sqlserver/jdbc/EnclavePackageTest.java | 1 + .../jdbc/fedauth/ConcurrentLoginTest.java | 1 + .../fedauth/ConnectionEncryptionTest.java | 1 + .../fedauth/ConnectionSuspensionTest.java | 1 + .../jdbc/fedauth/ErrorMessageTest.java | 1 + .../sqlserver/jdbc/fedauth/FedauthTest.java | 16 +-- .../sqlserver/jdbc/fedauth/FedauthWithAE.java | 1 + .../jdbc/fedauth/PooledConnectionTest.java | 1 + .../sqlserver/testframework/Constants.java | 1 + 19 files changed, 99 insertions(+), 44 deletions(-) diff --git a/pom.xml b/pom.xml index 026144f46..321c4226d 100644 --- a/pom.xml +++ b/pom.xml @@ -42,14 +42,16 @@ xAzureSQLDB - - - - For tests not compatible with Azure SQL Database - - xAzureSQLDW - - - - For tests not compatible with Azure Data Warehouse - xAzureSQLMI - - - - For tests not compatible with Azure SQL Managed Instance - NTLM - - - - - - - For tests using NTLM Authentication mode (excluded by default) - kerberos - - - - - For tests using Kerberos authentication (excluded by default) + NTLM - - - For tests using NTLM Authentication mode (excluded by default) + kerberos - - - - For tests using Kerberos authentication (excluded by default) reqExternalSetup - For tests requiring external setup (excluded by default) clientCertAuth - - For tests requiring client certificate authentication - setup (excluded by default) - - - - - - - - - - - - - - - - - - - - - - - + setup (excluded by default) - - - - - - - - - - - - - - - - - - - - - - - + requireSecret - For tests requiring setting up secrets manually - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - Default testing enabled with SQL Server 2019 (SQLv15) --> - xSQLv12,xSQLv15,NTLM,MSI,reqExternalSetup,clientCertAuth,fedAuth,kerberos + + xSQLv12,xSQLv15,NTLM,MSI,reqExternalSetup,clientCertAuth,fedAuth,kerberos,requireSecret -preview @@ -228,7 +230,8 @@ central - https://sqlclientdrivers.pkgs.visualstudio.com/public/_packaging/mssql-jdbc/maven/v1 + + https://sqlclientdrivers.pkgs.visualstudio.com/public/_packaging/mssql-jdbc/maven/v1 true @@ -240,7 +243,8 @@ central - https://sqlclientdrivers.pkgs.visualstudio.com/public/_packaging/mssql-jdbc/maven/v1 + + https://sqlclientdrivers.pkgs.visualstudio.com/public/_packaging/mssql-jdbc/maven/v1 true @@ -253,7 +257,8 @@ jre8 - ${project.artifactId}-${project.version}.jre8${releaseExt} + + ${project.artifactId}-${project.version}.jre8${releaseExt} org.apache.maven.plugins @@ -261,14 +266,20 @@ 3.8.0 - **/com/microsoft/sqlserver/jdbc/ISQLServerConnection43.java - **/com/microsoft/sqlserver/jdbc/SQLServerConnection43.java - **/com/microsoft/sqlserver/jdbc/SQLServerJdbc43.java + + **/com/microsoft/sqlserver/jdbc/ISQLServerConnection43.java + + **/com/microsoft/sqlserver/jdbc/SQLServerConnection43.java + + **/com/microsoft/sqlserver/jdbc/SQLServerJdbc43.java - **/com/microsoft/sqlserver/jdbc/connection/ConnectionWrapper43Test.java - **/com/microsoft/sqlserver/jdbc/connection/RequestBoundaryMethodsTest.java - **/com/microsoft/sqlserver/jdbc/JDBC43Test.java + + **/com/microsoft/sqlserver/jdbc/connection/ConnectionWrapper43Test.java + + **/com/microsoft/sqlserver/jdbc/connection/RequestBoundaryMethodsTest.java + + **/com/microsoft/sqlserver/jdbc/JDBC43Test.java 1.8 1.8 @@ -287,7 +298,8 @@ 3.1.1 - ${project.build.outputDirectory}/META-INF/MANIFEST.MF + + ${project.build.outputDirectory}/META-INF/MANIFEST.MF @@ -296,7 +308,8 @@ maven-surefire-plugin 3.0.0-M1 - + ${excludedGroups}, xJDBC42 @@ -306,7 +319,8 @@ jre11 - ${project.artifactId}-${project.version}.jre11${releaseExt} + + ${project.artifactId}-${project.version}.jre11${releaseExt} org.apache.maven.plugins @@ -314,7 +328,8 @@ 3.8.0 - **/com/microsoft/sqlserver/jdbc/SQLServerJdbc42.java + + **/com/microsoft/sqlserver/jdbc/SQLServerJdbc42.java 11 11 @@ -326,9 +341,11 @@ 3.1.1 - ${project.build.outputDirectory}/META-INF/MANIFEST.MF + + ${project.build.outputDirectory}/META-INF/MANIFEST.MF - com.microsoft.sqlserver.jdbc + + com.microsoft.sqlserver.jdbc @@ -339,7 +356,8 @@ jre17 - ${project.artifactId}-${project.version}.jre17${releaseExt} + + ${project.artifactId}-${project.version}.jre17${releaseExt} org.apache.maven.plugins @@ -347,7 +365,8 @@ 3.8.0 - **/com/microsoft/sqlserver/jdbc/SQLServerJdbc42.java + + **/com/microsoft/sqlserver/jdbc/SQLServerJdbc42.java 17 17 @@ -359,9 +378,11 @@ 3.1.1 - ${project.build.outputDirectory}/META-INF/MANIFEST.MF + + ${project.build.outputDirectory}/META-INF/MANIFEST.MF - com.microsoft.sqlserver.jdbc + + com.microsoft.sqlserver.jdbc @@ -372,7 +393,8 @@ jre21 - ${project.artifactId}-${project.version}.jre21${releaseExt} + + ${project.artifactId}-${project.version}.jre21${releaseExt} org.apache.maven.plugins @@ -380,7 +402,8 @@ 3.8.0 - **/com/microsoft/sqlserver/jdbc/SQLServerJdbc42.java + + **/com/microsoft/sqlserver/jdbc/SQLServerJdbc42.java 21 21 @@ -392,9 +415,11 @@ 3.1.1 - ${project.build.outputDirectory}/META-INF/MANIFEST.MF + + ${project.build.outputDirectory}/META-INF/MANIFEST.MF - com.microsoft.sqlserver.jdbc + + com.microsoft.sqlserver.jdbc @@ -408,7 +433,8 @@ true - ${project.artifactId}-${project.version}.jre23${releaseExt} + + ${project.artifactId}-${project.version}.jre23${releaseExt} org.apache.maven.plugins @@ -416,7 +442,8 @@ 3.8.0 - **/com/microsoft/sqlserver/jdbc/SQLServerJdbc42.java + + **/com/microsoft/sqlserver/jdbc/SQLServerJdbc42.java 23 23 @@ -428,9 +455,11 @@ 3.1.1 - ${project.build.outputDirectory}/META-INF/MANIFEST.MF + + ${project.build.outputDirectory}/META-INF/MANIFEST.MF - com.microsoft.sqlserver.jdbc + + com.microsoft.sqlserver.jdbc @@ -477,12 +506,16 @@ - + WARN - org.apache.maven.plugins:maven-verifier-plugin + + org.apache.maven.plugins:maven-verifier-plugin - Please consider using the maven-invoker-plugin + + Please consider using the + maven-invoker-plugin (http://maven.apache.org/plugins/maven-invoker-plugin/)! @@ -535,8 +568,10 @@ com.microsoft.sqlserver.jdbc.dataclassification, microsoft.sql - !microsoft.sql,jdk.net;resolution:=optional,* - com.microsoft.sqlserver.jdbc.osgi.Activator + + !microsoft.sql,jdk.net;resolution:=optional,* + + com.microsoft.sqlserver.jdbc.osgi.Activator @@ -573,7 +608,8 @@ true outdated-dependencies.txt - file:///${session.executionRootDirectory}/maven-version-rules.xml + + file:///${session.executionRootDirectory}/maven-version-rules.xml diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/BulkCopySendTemporalDataTypesAsStringAETest.java b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/BulkCopySendTemporalDataTypesAsStringAETest.java index 6dd177ab5..47f717f54 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/BulkCopySendTemporalDataTypesAsStringAETest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/BulkCopySendTemporalDataTypesAsStringAETest.java @@ -58,6 +58,7 @@ @Tag(Constants.xAzureSQLDB) @Tag(Constants.xAzureSQLDW) @Tag(Constants.reqExternalSetup) +@Tag(Constants.requireSecret) public class BulkCopySendTemporalDataTypesAsStringAETest extends AESetup { static String inputFile = "BulkCopyCSVSendTemporalDataTypesAsStringForBulkCopy.csv"; static String encoding = "UTF-8"; diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/CallableStatementTest.java b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/CallableStatementTest.java index d259b35a5..8215250ee 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/CallableStatementTest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/CallableStatementTest.java @@ -54,6 +54,7 @@ @Tag(Constants.xAzureSQLDW) @Tag(Constants.xAzureSQLDB) @Tag(Constants.reqExternalSetup) +@Tag(Constants.requireSecret) public class CallableStatementTest extends AESetup { private static String multiStatementsProcedure = AbstractSQLGenerator diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/EnclaveTest.java b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/EnclaveTest.java index 4b55573af..43dc7ed66 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/EnclaveTest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/EnclaveTest.java @@ -44,6 +44,7 @@ @Tag(Constants.xAzureSQLDW) @Tag(Constants.xAzureSQLDB) @Tag(Constants.reqExternalSetup) +@Tag(Constants.requireSecret) public class EnclaveTest extends AESetup { /** * Tests basic connection. diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/JDBCEncryptionDecryptionTest.java b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/JDBCEncryptionDecryptionTest.java index 6f3502933..b7cfac547 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/JDBCEncryptionDecryptionTest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/JDBCEncryptionDecryptionTest.java @@ -65,6 +65,7 @@ @Tag(Constants.xAzureSQLDW) @Tag(Constants.xAzureSQLDB) @Tag(Constants.reqExternalSetup) +@Tag(Constants.requireSecret) public class JDBCEncryptionDecryptionTest extends AESetup { private boolean nullable = false; diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/MSITest.java b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/MSITest.java index 9aeb0da8f..360e172a1 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/MSITest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/MSITest.java @@ -42,6 +42,7 @@ */ @RunWith(JUnitPlatform.class) @Tag(Constants.MSI) +@Tag(Constants.requireSecret) public class MSITest extends AESetup { /* diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/MultiUserAKVTest.java b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/MultiUserAKVTest.java index 2619a79b5..614895608 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/MultiUserAKVTest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/MultiUserAKVTest.java @@ -59,6 +59,7 @@ @Tag(Constants.xAzureSQLDW) @Tag(Constants.xAzureSQLDB) @Tag(Constants.reqExternalSetup) +@Tag(Constants.requireSecret) public class MultiUserAKVTest extends AESetup { private static Map requiredKeyStoreProvider = new HashMap<>(); diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/ParameterMetaDataCacheTest.java b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/ParameterMetaDataCacheTest.java index cef219d4a..ca4162154 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/ParameterMetaDataCacheTest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/ParameterMetaDataCacheTest.java @@ -33,6 +33,7 @@ @Tag(Constants.xSQLv12) @Tag(Constants.xSQLv14) @Tag(Constants.reqExternalSetup) +@Tag(Constants.requireSecret) public class ParameterMetaDataCacheTest extends AESetup { @BeforeAll diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/PrecisionScaleTest.java b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/PrecisionScaleTest.java index 162fe8cef..dcbe907b5 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/PrecisionScaleTest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/PrecisionScaleTest.java @@ -43,6 +43,7 @@ @Tag(Constants.xAzureSQLDW) @Tag(Constants.xAzureSQLDB) @Tag(Constants.reqExternalSetup) +@Tag(Constants.requireSecret) public class PrecisionScaleTest extends AESetup { private static java.util.Date date = null; private static int offsetFromGMT = 0; diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/RegressionAlwaysEncryptedTest.java b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/RegressionAlwaysEncryptedTest.java index d8725cdbe..c8f33573a 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/RegressionAlwaysEncryptedTest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/RegressionAlwaysEncryptedTest.java @@ -30,6 +30,7 @@ @Tag(Constants.xAzureSQLDW) @Tag(Constants.xAzureSQLDB) @Tag(Constants.reqExternalSetup) +@Tag(Constants.requireSecret) public class RegressionAlwaysEncryptedTest extends AESetup { static String numericTable[][] = {{"Bit", "bit"}, {"Tinyint", "tinyint"}, {"Smallint", "smallint"},}; diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/EnclavePackageTest.java b/src/test/java/com/microsoft/sqlserver/jdbc/EnclavePackageTest.java index 5f3c1165c..174755d4b 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/EnclavePackageTest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/EnclavePackageTest.java @@ -33,6 +33,7 @@ @Tag(Constants.xAzureSQLDW) @Tag(Constants.xAzureSQLDB) @Tag(Constants.reqExternalSetup) +@Tag(Constants.requireSecret) public class EnclavePackageTest extends AbstractTest { private static String connectionStringEnclave = null; diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ConcurrentLoginTest.java b/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ConcurrentLoginTest.java index 72dae99f9..7c7f2240d 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ConcurrentLoginTest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ConcurrentLoginTest.java @@ -23,6 +23,7 @@ @RunWith(JUnitPlatform.class) @Tag(Constants.fedAuth) +@Tag(Constants.requireSecret) public class ConcurrentLoginTest extends FedauthCommon { final AtomicReference throwableRef = new AtomicReference(); diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ConnectionEncryptionTest.java b/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ConnectionEncryptionTest.java index a1a2a5fd4..b7e307654 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ConnectionEncryptionTest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ConnectionEncryptionTest.java @@ -29,6 +29,7 @@ @RunWith(JUnitPlatform.class) @Tag(Constants.fedAuth) +@Tag(Constants.requireSecret) public class ConnectionEncryptionTest extends FedauthCommon { static String charTable = TestUtils.escapeSingleQuotes( diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ConnectionSuspensionTest.java b/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ConnectionSuspensionTest.java index 94e70a142..c9e34d8b2 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ConnectionSuspensionTest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ConnectionSuspensionTest.java @@ -31,6 +31,7 @@ @RunWith(JUnitPlatform.class) @Tag("slow") @Tag(Constants.fedAuth) +@Tag(Constants.requireSecret) public class ConnectionSuspensionTest extends FedauthCommon { static String charTable = TestUtils.escapeSingleQuotes( diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ErrorMessageTest.java b/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ErrorMessageTest.java index dd4d95239..ebe87e65c 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ErrorMessageTest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ErrorMessageTest.java @@ -25,6 +25,7 @@ @RunWith(JUnitPlatform.class) @Tag(Constants.fedAuth) +@Tag(Constants.requireSecret) public class ErrorMessageTest extends FedauthCommon { String badUserName = "abc" + azureUserName; diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/FedauthTest.java b/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/FedauthTest.java index f3e4e957f..436ca8e17 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/FedauthTest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/FedauthTest.java @@ -42,6 +42,7 @@ @RunWith(JUnitPlatform.class) @Tag(Constants.fedAuth) +@Tag(Constants.requireSecret) public class FedauthTest extends FedauthCommon { static String charTable = TestUtils .escapeSingleQuotes(AbstractSQLGenerator.escapeIdentifier(RandomUtil.getIdentifier("JDBC_FedAuthTest"))); @@ -366,8 +367,8 @@ public void testActiveDirectoryPasswordFailureOnSubsequentConnectionsWithInvalid public void testAADServicePrincipalCertAuthFailureOnSubsequentConnectionsWithInvalidatedTokenCacheWithInvalidPassword() throws Exception { // Should succeed on valid cert field values String url = "jdbc:sqlserver://" + azureServer + ";database=" + azureDatabase + ";authentication=" - + SqlAuthentication.ActiveDirectoryServicePrincipalCertificate + ";Username=" + servicePrincipalCertificateApplicationClientId - + ";clientCertificate=" + clientCertificate; + + SqlAuthentication.ActiveDirectoryServicePrincipalCertificate + ";Username=" + + servicePrincipalCertificateApplicationClientId + ";clientCertificate=" + clientCertificate; // Should fail on invalid cert field values String invalidPasswordUrl = "jdbc:sqlserver://" + azureServer + ";database=" + azureDatabase @@ -428,8 +429,8 @@ public void testAADServicePrincipalAuthWrong() { public void testAADServicePrincipalCertAuth() { // certificate from AKV has no password String url = "jdbc:sqlserver://" + azureServer + ";database=" + azureDatabase + ";authentication=" - + SqlAuthentication.ActiveDirectoryServicePrincipalCertificate + ";Username=" + servicePrincipalCertificateApplicationClientId - + ";clientCertificate=" + clientCertificate; + + SqlAuthentication.ActiveDirectoryServicePrincipalCertificate + ";Username=" + + servicePrincipalCertificateApplicationClientId + ";clientCertificate=" + clientCertificate; String urlEncrypted = url + ";encrypt=false;trustServerCertificate=true;"; SQLServerDataSource ds = new SQLServerDataSource(); @@ -450,7 +451,8 @@ public void testAADServicePrincipalCertAuth() { @Test public void testAADServicePrincipalCertAuthWrong() { String baseUrl = "jdbc:sqlserver://" + azureServer + ";database=" + azureDatabase + ";authentication=" - + SqlAuthentication.ActiveDirectoryServicePrincipalCertificate + ";userName=" + servicePrincipalCertificateApplicationClientId; + + SqlAuthentication.ActiveDirectoryServicePrincipalCertificate + ";userName=" + + servicePrincipalCertificateApplicationClientId; // no certificate provided. String url = baseUrl; @@ -469,8 +471,8 @@ public void testAADServicePrincipalCertAuthWrong() { validateException(url, "R_readCertError"); // wrong certificate key or password - url = baseUrl + ";password=" + azurePassword + ";clientCertificate=" + clientCertificate + ";clientKey=wrongKey;" - + "clientPassword=wrongPassword"; + url = baseUrl + ";password=" + azurePassword + ";clientCertificate=" + clientCertificate + + ";clientKey=wrongKey;" + "clientPassword=wrongPassword"; validateException(url, "R_readCertError"); } diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/FedauthWithAE.java b/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/FedauthWithAE.java index 1e0112f02..e42cc9f56 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/FedauthWithAE.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/FedauthWithAE.java @@ -37,6 +37,7 @@ @RunWith(JUnitPlatform.class) @Tag(Constants.fedAuth) +@Tag(Constants.requireSecret) public class FedauthWithAE extends FedauthCommon { static String cmkName1 = Constants.CMK_NAME + "fedauthAE1"; diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/PooledConnectionTest.java b/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/PooledConnectionTest.java index 85a232fd0..604782eab 100644 --- a/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/PooledConnectionTest.java +++ b/src/test/java/com/microsoft/sqlserver/jdbc/fedauth/PooledConnectionTest.java @@ -58,6 +58,7 @@ @RunWith(JUnitPlatform.class) @Tag("slow") @Tag(Constants.fedAuth) +@Tag(Constants.requireSecret) public class PooledConnectionTest extends FedauthCommon { static String charTable = TestUtils.escapeSingleQuotes( diff --git a/src/test/java/com/microsoft/sqlserver/testframework/Constants.java b/src/test/java/com/microsoft/sqlserver/testframework/Constants.java index 8ed409945..3009d2213 100644 --- a/src/test/java/com/microsoft/sqlserver/testframework/Constants.java +++ b/src/test/java/com/microsoft/sqlserver/testframework/Constants.java @@ -47,6 +47,7 @@ private Constants() {} public static final String reqExternalSetup = "reqExternalSetup"; public static final String clientCertAuth = "clientCertAuth"; public static final String fedAuth = "fedAuth"; + public static final String requireSecret = "requireSecret"; public static final ThreadLocalRandom RANDOM = ThreadLocalRandom.current(); public static final Logger LOGGER = Logger.getLogger("AbstractTest");