fix test final

Netflix · Dec 4, 2024 · fd0085f · fd0085f
1 parent 8275c09
commit fd0085f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/.../src/main/java/com/netflix/metacat/connector/polaris/PolarisConnectorDatabaseService.java b/.../src/main/java/com/netflix/metacat/connector/polaris/PolarisConnectorDatabaseService.java
@@ -86,11 +86,8 @@ public void delete(final ConnectorRequestContext context, final QualifiedName na
             throw new DatabaseNotFoundException(name);
         }
         try {
-            System.out.println("BEFORE deleting polaris database " + name.getDatabaseName());
             this.polarisStoreService.deleteDatabase(name.getDatabaseName());
-            System.out.println("AFTER deleting polaris database " + name.getDatabaseName());
         } catch (DataIntegrityViolationException exception) {
-            System.out.println("CAUGHT THE DB DELETE ERROR");
             if (exception.getCause() instanceof org.hibernate.exception.ConstraintViolationException) {
                 throw new DatabasePreconditionFailedException(
                     name,

diff --git a/.../src/test/java/com/netflix/metacat/connector/polaris/store/PolarisStoreConnectorTest.java b/.../src/test/java/com/netflix/metacat/connector/polaris/store/PolarisStoreConnectorTest.java
@@ -1,7 +1,6 @@
 package com.netflix.metacat.connector.polaris.store;
 
 
-import com.netflix.metacat.common.server.connectors.exception.DatabasePreconditionFailedException;
 import com.netflix.metacat.connector.polaris.common.PolarisUtils;
 import com.netflix.metacat.connector.polaris.configs.PolarisPersistenceConfig;
 import com.netflix.metacat.connector.polaris.store.entities.PolarisDatabaseEntity;
@@ -17,6 +16,7 @@
 import org.springframework.boot.test.autoconfigure.orm.jpa.AutoConfigureDataJpa;
 import org.springframework.boot.test.context.SpringBootTest;
 import org.springframework.dao.DataAccessException;
+import org.springframework.dao.DataIntegrityViolationException;
 import org.springframework.dao.OptimisticLockingFailureException;
 import org.springframework.test.context.ActiveProfiles;
 import org.springframework.test.context.junit.jupiter.SpringExtension;
@@ -138,7 +138,7 @@ public void testDbDeletionNoCascade() {
         final PolarisDatabaseEntity dbEntity = createDB(dbName);
         final PolarisTableEntity tblEntity = createTable(dbName, tblName);
 
-        Assertions.assertThrows(DatabasePreconditionFailedException.class, () ->
+        Assertions.assertThrows(DataIntegrityViolationException.class, () ->
             polarisConnector.deleteDatabase(dbName));
         Assert.assertTrue(polarisConnector.databaseExists(dbName));
     }