Address PR comments

microsoft · Aug 19, 2024 · 4b89d7c · 4b89d7c
1 parent 296bb02
commit 4b89d7c
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/src/test/java/com/microsoft/sqlserver/jdbc/unit/statement/BatchExecutionTest.java b/src/test/java/com/microsoft/sqlserver/jdbc/unit/statement/BatchExecutionTest.java
@@ -149,12 +149,12 @@ public void testSqlServerBulkCopyCachingPstmtLevel() throws Exception {
 
     @Test
     public void testSqlServerBulkCopyCachingConnectionLevel() throws Exception {
-        Calendar gmtCal = Calendar.getInstance(TimeZone.getTimeZone("GMT"));
-        long ms = 1578743412000L;
-
         // Needs to be on a JDK version greater than 8
         assumeTrue(TestUtils.getJVMVersion() > 8);
 
+        Calendar gmtCal = Calendar.getInstance(TimeZone.getTimeZone("GMT"));
+        long ms = 1578743412000L;
+
         try (SQLServerConnection con = (SQLServerConnection) DriverManager.getConnection(connectionString
                 + ";useBulkCopyForBatchInsert=true;cacheBulkCopyMetadata=true;sendTemporalDataTypesAsStringForBulkCopy=false;");
                 Statement stmt = con.createStatement()) {
@@ -213,14 +213,14 @@ public void testSqlServerBulkCopyCachingConnectionLevel() throws Exception {
 
     @Test
     public void testSqlServerBulkCopyCachingConnectionLevelMultiThreaded() throws Exception {
+        // Needs to be on a JDK version greater than 8
+        assumeTrue(TestUtils.getJVMVersion() > 8);
+
         Calendar gmtCal = Calendar.getInstance(TimeZone.getTimeZone("GMT"));
         long ms = 1578743412000L;
-        long timeOut = 30000; // 30 seconds
+        long timeOut = 30000;
         int NUMBER_SIMULTANEOUS_INSERTS = 5;
 
-        // Needs to be on a JDK version greater than 8
-        assumeTrue(TestUtils.getJVMVersion() > 8);
-
         try (SQLServerConnection con = (SQLServerConnection) DriverManager.getConnection(connectionString
                 + ";useBulkCopyForBatchInsert=true;cacheBulkCopyMetadata=true;sendTemporalDataTypesAsStringForBulkCopy=false;");
                 Statement stmt = con.createStatement()) {