Added some protective code here and there

Author: chernser

.github/workflows/build.yml

@@ -258,20 +258,16 @@ jobs:
         run: mvn --also-make --batch-mode --no-transfer-progress -DskipTests=true -Dmaven.javadoc.skip=true install
       - name: Generate Database Name
         run: |
-          # Combine the GitHub Run ID and a random number to guarantee uniqueness
           RANDOM_DB="clickhouse_java_test_${GITHUB_RUN_ID}_${RANDOM}"
-          
-          # Write it to GITHUB_ENV so future steps can access it as $DB_NAME
           echo "TEST_DB_NAME=$RANDOM_DB" >> $GITHUB_ENV
           echo "Generated database name: $RANDOM_DB"
-
       - name: Create Temporary Database
         env:
           CLICKHOUSE_CLOUD_HOST: ${{ secrets.INTEGRATIONS_TEAM_TESTS_CLOUD_HOST_SMT_PROD }}
           CLICKHOUSE_CLOUD_PASSWORD: ${{ secrets.INTEGRATIONS_TEAM_TESTS_CLOUD_PASSWORD_SMT_PROD }}
         run: |
           echo "Creating database: \`$TEST_DB_NAME\`"
-          curl -s -u "default:$CLICKHOUSE_CLOUD_PASSWORD" "https://$CLICKHOUSE_CLOUD_HOST:8443" --data-binary "CREATE DATABASE IF NOT EXISTS $TEST_DB_NAME"
+          curl --fail-with-body -s -u "default:$CLICKHOUSE_CLOUD_PASSWORD" "https://$CLICKHOUSE_CLOUD_HOST:8443" --data-binary "CREATE DATABASE IF NOT EXISTS $TEST_DB_NAME"
       - name: Test http client
         env:
           CLICKHOUSE_CLOUD_HOST: ${{ secrets.INTEGRATIONS_TEAM_TESTS_CLOUD_HOST_SMT_PROD }}
@@ -298,7 +294,7 @@ jobs:
           if [ -n "$TEST_DB_NAME" ]; then
             DROP_STMT="DROP DATABASE IF EXISTS \`$TEST_DB_NAME\`" 
             echo "Cleaning up database: \`$TEST_DB_NAME\` with statement '$DROP_STMT'";
-            curl -s -u "default:$CLICKHOUSE_CLOUD_PASSWORD" "https://$CLICKHOUSE_CLOUD_HOST:8443" --data-binary "$DROP_STMT";
+            curl --fail-with-body -s -u "default:$CLICKHOUSE_CLOUD_PASSWORD" "https://$CLICKHOUSE_CLOUD_HOST:8443" --data-binary "$DROP_STMT";
           else
             echo "No database name was generated; skipping cleanup.";
           fi;

clickhouse-client/src/test/java/com/clickhouse/client/ClickHouseServerForTest.java

@@ -68,6 +68,9 @@ public class ClickHouseServerForTest {
         properties = new Properties(System.getProperties());
         String externalDatabase = System.getenv("TEST_DB_NAME"); // see build.yaml workflow
         if (externalDatabase != null && !externalDatabase.trim().isEmpty()) {
+            if (!externalDatabase.startsWith("clickhouse_java_test_")) {
+                throw new RuntimeException("external database for tests should start with 'clickhouse_java_test_'");
+            }
             localDatabase = false;
             database = externalDatabase;
         } else {

client-v2/src/test/java/com/clickhouse/client/HttpTransportTests.java

@@ -1220,7 +1220,8 @@ public void testJWTWithCloud() throws Exception {
         }
         final String jwt = System.getenv("CLIENT_JWT");
         final String host = System.getenv("JWT_TEST_HOST");
-        Assert.assertTrue(jwt != null && !jwt.trim().isEmpty(), "JWT is missing");
+        Assert.assertTrue(jwt != null && !jwt.trim().isEmpty(), "CLIENT_JWT is not set.");
+        Assert.assertTrue(host != null && !host.trim().isEmpty(), "JWT_TEST_HOST is not set");
         Assert.assertFalse(jwt.contains("\n") || jwt.contains("-----"), "JWT should be single string ready for HTTP header");
         try (Client client = new Client.Builder()
                 .addEndpoint(Protocol.HTTP, host, 8443, true)