feat: add Spanner guards, fix group commit filtering, clean up artifacts

- JdbcAdminCaseSensitivityIntegrationTest: disable rename, FLOAT index,
  alter column type, BLOB index tests for Spanner
- JdbcAdminImportTableIntegrationTest: disable alter column type and
  widening conversion tests for Spanner
- Remove group_commit_tests duplication: filter tests list at runtime to
  only include *ConsensusCommit* classes in group_commit flavor
- Skip group_commit jobs entirely when no ConsensusCommit tests exist
- Prefix binary result artifacts with internal_ for clarity
- Upload per-job HTML reports alongside binary results
- Delete internal artifacts after successful aggregation

Author: Torch3333

.github/workflows/spanner-adapter-check.yaml

@@ -50,32 +50,26 @@ jobs:
           - name: JdbcAdminIntegrationTest
             label: spanner_admin
             tests: "JdbcAdminIntegrationTest"
-            group_commit_tests: ""
 
           - name: ConsensusCommitSpecific
             label: spanner_cc_specific
             tests: "ConsensusCommitSpecificIntegrationTestWithJdbcDatabase"
-            group_commit_tests: "ConsensusCommitSpecificIntegrationTestWithJdbcDatabase"
 
           - name: ConsensusCommitSpecific (highest isolation)
             label: spanner_cc_specific_highest
             tests: "ConsensusCommitSpecificIntegrationTestWithJdbcDatabaseInHighestIsolation"
-            group_commit_tests: "ConsensusCommitSpecificIntegrationTestWithJdbcDatabaseInHighestIsolation"
 
           - name: ConsensusCommitSpecific (metadata decoupling)
             label: spanner_cc_specific_md
             tests: "ConsensusCommitSpecificWithMetadataDecouplingIntegrationTestWithJdbcDatabase"
-            group_commit_tests: "ConsensusCommitSpecificWithMetadataDecouplingIntegrationTestWithJdbcDatabase"
 
           - name: MultipleClusteringKeyScan
             label: spanner_multi_ck_scan
             tests: "JdbcDatabaseMultipleClusteringKeyScanIntegrationTest"
-            group_commit_tests: ""
 
           - name: ConditionalMutation
             label: spanner_cond_mutation
             tests: "JdbcDatabaseConditionalMutationIntegrationTest"
-            group_commit_tests: ""
 
           # --- Group 1: Admin and import tests (~26m) ---
 
@@ -92,13 +86,6 @@ jobs:
               JdbcAdminRepairIntegrationTest
               JdbcSchemaLoaderImportIntegrationTest
               JdbcSchemaLoaderImportWithMetadataDecouplingIntegrationTest
-            group_commit_tests: >-
-              ConsensusCommitAdminIntegrationTestWithJdbcDatabase
-              ConsensusCommitAdminRepairIntegrationTestWithJdbcDatabase
-              ConsensusCommitAdminImportTableIntegrationTestWithJdbcDatabase
-              ConsensusCommitAdminImportTableWithMetadataDecouplingIntegrationTestWithJdbcDatabase
-              ConsensusCommitImportTableIntegrationTestWithJdbcDatabase
-              ConsensusCommitImportTableWithMetadataDecouplingIntegrationTestWithJdbcDatabase
 
           # --- Group 2: Core JDBC database tests (~12m) ---
 
@@ -112,7 +99,6 @@ jobs:
               JdbcDatabaseSinglePartitionKeyIntegrationTest
               JdbcDatabaseMultiplePartitionKeyIntegrationTest
               JdbcDatabaseSecondaryIndexIntegrationTest
-            group_commit_tests: ""
 
           # --- Group 3: Schema, i18n, virtual tables (~25m) ---
 
@@ -127,9 +113,9 @@ jobs:
               JdbcAdminCaseSensitivityIntegrationTest
               JdbcSchemaLoaderIntegrationTest
               JdbcSchemaLoaderWithMetadataDecouplingIntegrationTest
-            group_commit_tests: ""
 
           # --- Group 4: ConsensusCommit base + TwoPhase + SingleCrud (~15m) ---
+          # Group commit filters to *ConsensusCommit* only (excludes TwoPhase and SingleCrud)
 
           - name: "CC base & TwoPhase & SingleCrud"
             label: spanner_group4
@@ -146,22 +132,24 @@ jobs:
               TwoPhaseConsensusCommitWithIncludeMetadataEnabledIntegrationTestWithJdbcDatabase
               SingleCrudOperationTransactionIntegrationTestWithJdbcDatabase
               SingleCrudOperationTransactionAdminIntegrationTestWithJdbcDatabase
-            # TwoPhaseConsensusCommit excluded: group commit is incompatible with two-phase commit interface
-            group_commit_tests: >-
-              ConsensusCommitIntegrationTestWithJdbcDatabase
-              ConsensusCommitIntegrationTestWithJdbcDatabaseInHighestIsolation
-              ConsensusCommitCrossPartitionScanIntegrationTestWithJdbcDatabase
-              ConsensusCommitNullMetadataIntegrationTestWithJdbcDatabase
-              ConsensusCommitWithIncludeMetadataEnabledIntegrationTestWithJdbcDatabase
-              ConsensusCommitWithMetadataDecouplingIntegrationTestWithJdbcDatabase
 
     steps:
       - name: Check if job should run
         id: should-run
         run: |
-          if [ "${{ matrix.group_commit_enabled }}" = "true" ] && [ -z "${{ matrix.test_group.group_commit_tests }}" ]; then
-            echo "skip=true" >> "$GITHUB_OUTPUT"
-            echo "Skipping: no ConsensusCommit tests in this group"
+          # For group_commit flavor, filter tests to only *ConsensusCommit* classes
+          # Skip entirely if no ConsensusCommit tests exist in this group
+          if [ "${{ matrix.group_commit_enabled }}" = "true" ]; then
+            CC_TESTS=""
+            for test_class in ${{ matrix.test_group.tests }}; do
+              case "${test_class}" in *ConsensusCommit*) CC_TESTS="${CC_TESTS} ${test_class}" ;; esac
+            done
+            if [ -z "${CC_TESTS}" ]; then
+              echo "skip=true" >> "$GITHUB_OUTPUT"
+              echo "Skipping: no ConsensusCommit tests in this group"
+            else
+              echo "skip=false" >> "$GITHUB_OUTPUT"
+            fi
           else
             echo "skip=false" >> "$GITHUB_OUTPUT"
           fi
@@ -239,16 +227,12 @@ jobs:
           ORG_GRADLE_PROJECT_integrationTestJavaRuntimeVersion: "${{ env.INT_TEST_JAVA_RUNTIME_VERSION }}"
           ORG_GRADLE_PROJECT_integrationTestJavaRuntimeVendor: "${{ env.INT_TEST_JAVA_RUNTIME_VENDOR }}"
         run: |
-          # Select test list: use group_commit_tests when group commit is enabled, otherwise use tests
-          if [ "${{ matrix.group_commit_enabled }}" = "true" ]; then
-            TEST_LIST="${{ matrix.test_group.group_commit_tests }}"
-          else
-            TEST_LIST="${{ matrix.test_group.tests }}"
-          fi
-
-          # Build the --tests arguments from the space-separated test list
+          # Build the --tests arguments; for group_commit, filter to *ConsensusCommit* only
           TESTS_ARGS=""
-          for test_class in ${TEST_LIST}; do
+          for test_class in ${{ matrix.test_group.tests }}; do
+            if [ "${{ matrix.group_commit_enabled }}" = "true" ]; then
+              case "${test_class}" in *ConsensusCommit*) ;; *) continue ;; esac
+            fi
             TESTS_ARGS="${TESTS_ARGS} --tests ${test_class}"
           done
 
@@ -267,7 +251,7 @@ jobs:
         if: ${{ always() && steps.should-run.outputs.skip == 'false' }}
         uses: actions/upload-artifact@v7
         with:
-          name: test-results_${{ matrix.test_group.label }}_${{ matrix.group_commit_enabled == 'true' && 'group_commit' || 'default' }}
+          name: internal_test-results_${{ matrix.test_group.label }}_${{ matrix.group_commit_enabled == 'true' && 'group_commit' || 'default' }}
           path: core/build/test-results/integrationTestJdbc/
 
       - name: Upload HTML test report
@@ -303,10 +287,11 @@ jobs:
       - name: Download all ${{ matrix.flavor }} test results
         uses: actions/download-artifact@v4
         with:
-          pattern: test-results_*_${{ matrix.flavor }}
+          pattern: internal_test-results_*_${{ matrix.flavor }}
           path: downloaded-results/
 
       - name: Generate aggregated HTML report
+        id: aggregate-report
         run: |
           # Create an init script that registers an aggregate TestReport task
           cat > /tmp/aggregate-init.gradle << 'EOF'
@@ -333,3 +318,12 @@ jobs:
         with:
           name: spanner_ci_${{ matrix.flavor }}_report_${{ env.ARTIFACT_SUFFIX }}_run_#${{ github.run_attempt }}
           path: build/reports/tests/aggregated/
+
+      - name: Delete per-job binary and HTML report artifacts
+        if: steps.aggregate-report.outcome == 'success'
+        uses: geekyeggo/delete-artifact@v6
+        with:
+          name: |
+            internal_test-results_*_${{ matrix.flavor }}
+            *_${{ matrix.flavor }}_report_${{ env.ARTIFACT_SUFFIX }}
+          failOnError: false