(gravitino) branch main updated: [#5724] fix(bug): testCatalogDetailsCommand() failed in the CI (#5725)

[mchades]

This is an automated email from the ASF dual-hosted git repository.

mchades pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/main by this push:
     new 0e4a0dcaa [#5724] fix(bug): testCatalogDetailsCommand() failed in the 
CI (#5725)
0e4a0dcaa is described below

commit 0e4a0dcaa58612e7d6a2699f668124b2b7f63cdc
Author: Xun <x...@datastrato.com>
AuthorDate: Tue Dec 3 15:20:57 2024 +0800

    [#5724] fix(bug): testCatalogDetailsCommand() failed in the CI (#5725)
    
    ### What changes were proposed in this pull request?
    
    Add `catalog-jdbc-postgresql` in the CLI Gradle build scripts.
    
    ### Why are the changes needed?
    
    Fix: #5724
    
    ### Does this PR introduce _any_ user-facing change?
    
    N/A
    
    ### How was this patch tested?
    
    CI Passed.
---
 clients/cli/build.gradle.kts                     | 11 ++++++
 clients/cli/src/test/resources/log4j2.properties | 47 ++++++++++++++++++++++++
 2 files changed, 58 insertions(+)

diff --git a/clients/cli/build.gradle.kts b/clients/cli/build.gradle.kts
index ae45fc968..8d42712ea 100644
--- a/clients/cli/build.gradle.kts
+++ b/clients/cli/build.gradle.kts
@@ -67,3 +67,14 @@ tasks.jar {
   from(dependencies)
   duplicatesStrategy = DuplicatesStrategy.EXCLUDE
 }
+
+tasks.test {
+  val skipITs = project.hasProperty("skipITs")
+  if (skipITs) {
+    // Exclude integration tests
+    exclude("**/integration/test/**")
+  } else {
+    dependsOn(tasks.jar)
+    dependsOn(":catalogs:catalog-jdbc-postgresql:jar", 
":catalogs:catalog-jdbc-postgresql:runtimeJars")
+  }
+}
diff --git a/clients/cli/src/test/resources/log4j2.properties 
b/clients/cli/src/test/resources/log4j2.properties
new file mode 100644
index 000000000..e2e88964c
--- /dev/null
+++ b/clients/cli/src/test/resources/log4j2.properties
@@ -0,0 +1,47 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#  http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+#
+
+# Set to debug or trace if log4j initialization is failing
+status = info
+
+# Name of the configuration
+name = ConsoleLogConfig
+
+# Console appender configuration
+appender.console.type = Console
+appender.console.name = consoleLogger
+appender.console.layout.type = PatternLayout
+appender.console.layout.pattern = %d{yyyy-MM-dd HH:mm:ss} %-5p [%t] %c{1}:%L - 
%m%n
+
+# Log files location
+property.logPath = ${sys:gravitino.log.path:-build/clients-cli.log}
+
+# File appender configuration
+appender.file.type = File
+appender.file.name = fileLogger
+appender.file.fileName = ${logPath}
+appender.file.layout.type = PatternLayout
+appender.file.layout.pattern = %d{yyyy-MM-dd HH:mm:ss.SSS} [%t] %-5p %c - %m%n
+
+# Root logger level
+rootLogger.level = info
+
+# Root logger referring to console and file appenders
+rootLogger.appenderRef.stdout.ref = consoleLogger
+rootLogger.appenderRef.file.ref = fileLogger