You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@polaris.apache.org by di...@apache.org on 2025/01/10 15:40:46 UTC

(polaris) branch main updated: Make testCreateCatalogWithUnparsableJson() more generic (#667)

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

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


The following commit(s) were added to refs/heads/main by this push:
     new a8d88fe0 Make testCreateCatalogWithUnparsableJson() more generic (#667)
a8d88fe0 is described below

commit a8d88fe0b9e170eca47d2864ce3059446ee73f16
Author: Dmitri Bourlatchkov <dm...@dremio.com>
AuthorDate: Fri Jan 10 10:40:42 2025 -0500

    Make testCreateCatalogWithUnparsableJson() more generic (#667)
    
    Remove assertion on too specific JSON parsing error message.
    The exact message text may vary in different server builds.
---
 .../service/dropwizard/admin/PolarisServiceImplIntegrationTest.java | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/dropwizard/service/src/test/java/org/apache/polaris/service/dropwizard/admin/PolarisServiceImplIntegrationTest.java b/dropwizard/service/src/test/java/org/apache/polaris/service/dropwizard/admin/PolarisServiceImplIntegrationTest.java
index 2aa5bd6c..f2120c36 100644
--- a/dropwizard/service/src/test/java/org/apache/polaris/service/dropwizard/admin/PolarisServiceImplIntegrationTest.java
+++ b/dropwizard/service/src/test/java/org/apache/polaris/service/dropwizard/admin/PolarisServiceImplIntegrationTest.java
@@ -554,11 +554,7 @@ public class PolarisServiceImplIntegrationTest {
       assertThat(response)
           .returns(Response.Status.BAD_REQUEST.getStatusCode(), Response::getStatus);
       ErrorResponse error = response.readEntity(ErrorResponse.class);
-      assertThat(error)
-          .isNotNull()
-          .extracting(ErrorResponse::message)
-          .asString()
-          .startsWith("Invalid JSON: Unexpected character");
+      assertThat(error).isNotNull().extracting(ErrorResponse::message).isNotNull();
     }
   }