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

jbonofre pushed a commit to branch release/0.10.x
in repository https://gitbox.apache.org/repos/asf/polaris.git


The following commit(s) were added to refs/heads/release/0.10.x by this push:
     new 6a203aa07 Fix a failing task with the release profile (#1693)
6a203aa07 is described below

commit 6a203aa07596f8af41c25a61d73c07a65e39a984
Author: Yufei Gu <yu...@apache.org>
AuthorDate: Thu May 29 14:30:45 2025 -0700

    Fix a failing task with the release profile (#1693)
    
    (cherry picked from commit 046fa7076203ddfc66574323adb2996535cd1d8c)
---
 .../apache/polaris/version/TestPolarisVersion.java | 22 +++++++++++++++-------
 1 file changed, 15 insertions(+), 7 deletions(-)

diff --git 
a/tools/version/src/jarTest/java/org/apache/polaris/version/TestPolarisVersion.java
 
b/tools/version/src/jarTest/java/org/apache/polaris/version/TestPolarisVersion.java
index 63982e31f..695959b34 100644
--- 
a/tools/version/src/jarTest/java/org/apache/polaris/version/TestPolarisVersion.java
+++ 
b/tools/version/src/jarTest/java/org/apache/polaris/version/TestPolarisVersion.java
@@ -57,13 +57,21 @@ public class TestPolarisVersion {
   @Order(1)
   public void versionAvailable() {
     
soft.assertThat(polarisVersionString()).isEqualTo(System.getProperty("polarisVersion"));
-    soft.assertThat(isReleaseBuild()).isFalse();
-    soft.assertThat(getBuildReleasedVersion()).isEmpty();
-    soft.assertThat(getBuildTimestamp()).isEmpty();
-    soft.assertThat(getBuildGitHead()).isEmpty();
-    soft.assertThat(getBuildGitTag()).isEmpty();
-    soft.assertThat(getBuildSystem()).isEmpty();
-    soft.assertThat(getBuildJavaVersion()).isEmpty();
+    if (isReleaseBuild()) {
+      soft.assertThat(getBuildReleasedVersion()).isNotEmpty();
+      soft.assertThat(getBuildTimestamp()).isNotEmpty();
+      soft.assertThat(getBuildGitHead()).isNotEmpty();
+      soft.assertThat(getBuildGitTag()).isNotEmpty();
+      soft.assertThat(getBuildSystem()).isNotEmpty();
+      soft.assertThat(getBuildJavaVersion()).isNotEmpty();
+    } else {
+      soft.assertThat(getBuildReleasedVersion()).isEmpty();
+      soft.assertThat(getBuildTimestamp()).isEmpty();
+      soft.assertThat(getBuildGitHead()).isEmpty();
+      soft.assertThat(getBuildGitTag()).isEmpty();
+      soft.assertThat(getBuildSystem()).isEmpty();
+      soft.assertThat(getBuildJavaVersion()).isEmpty();
+    }
   }
 
   /**

Reply via email to