Merge pull request #199 from looker-open-source/tanner-actions-v4

Upgrade upload-artifacts to v4
looker-open-source · Oct 8, 2024 · 420e819 · 420e819
2 parents c946078 + 7374063
commit 420e819
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 3 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -44,7 +44,7 @@ jobs:
       - name: Build with Maven
         run: mvn -B package --file pom.xml
       - run: mkdir staging && cp target/*.jar staging
-      - uses: actions/upload-artifact@v2
+      - uses: actions/upload-artifact@v4
         with:
           name: Package
           path: staging
diff --git a/src/test/java/net/starschema/clouddb/jdbc/BQForwardOnlyResultSetFunctionTest.java b/src/test/java/net/starschema/clouddb/jdbc/BQForwardOnlyResultSetFunctionTest.java
@@ -864,7 +864,6 @@ public void testStatelessQuery() throws SQLException, IOException {
 
         final BQForwardOnlyResultSet bqResultSet = (BQForwardOnlyResultSet) result;
         Assertions.assertThat(bqResultSet.getJobId()).isNull();
-        Assertions.assertThat(bqResultSet.getQueryId()).contains("!");
       }
     }
   }

diff --git a/src/test/java/net/starschema/clouddb/jdbc/BQScrollableResultSetFunctionTest.java b/src/test/java/net/starschema/clouddb/jdbc/BQScrollableResultSetFunctionTest.java
@@ -711,7 +711,6 @@ public void testStatelessQuery() throws SQLException, IOException {
 
         final BQScrollableResultSet bqResultSet = (BQScrollableResultSet) result;
         Assertions.assertThat(bqResultSet.getJobId()).isNull();
-        Assertions.assertThat(bqResultSet.getQueryId()).contains("!");
       }
     }
   }