fix: fix failing tests

- Fix Glue integration test tearDown NPE when credentials unavailable
- Fix Makefile test patterns (!*IntegrationTest -> !*Integration)
- Add Iceberg config unit tests
- Remove warehouse from Python Iceberg test config

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: jackye1995

java/Makefile

@@ -42,7 +42,7 @@ build-hive2:
 
 .PHONY: test-hive2
 test-hive2:
-	./mvnw test -pl lance-namespace-hive2 -Dtest="!*IntegrationTest"
+	./mvnw test -pl lance-namespace-hive2 -Dtest="!*Integration"
 
 # ============================================================================
 # Hive3
@@ -59,7 +59,7 @@ build-hive3:
 
 .PHONY: test-hive3
 test-hive3:
-	./mvnw test -pl lance-namespace-hive3 -Dtest="!*IntegrationTest"
+	./mvnw test -pl lance-namespace-hive3 -Dtest="!*Integration"
 
 # ============================================================================
 # Iceberg
@@ -76,7 +76,7 @@ build-iceberg:
 
 .PHONY: test-iceberg
 test-iceberg:
-	./mvnw test -pl lance-namespace-iceberg -Dtest="!*IntegrationTest"
+	./mvnw test -pl lance-namespace-iceberg -Dtest="!*Integration"
 
 # ============================================================================
 # Polaris
@@ -93,7 +93,7 @@ build-polaris:
 
 .PHONY: test-polaris
 test-polaris:
-	./mvnw test -pl lance-namespace-polaris -Dtest="!*IntegrationTest"
+	./mvnw test -pl lance-namespace-polaris -Dtest="!*Integration"
 
 # ============================================================================
 # Unity
@@ -110,7 +110,7 @@ build-unity:
 
 .PHONY: test-unity
 test-unity:
-	./mvnw test -pl lance-namespace-unity -Dtest="!*IntegrationTest"
+	./mvnw test -pl lance-namespace-unity -Dtest="!*Integration"
 
 # ============================================================================
 # All

java/lance-namespace-glue/src/test/java/org/lance/namespace/glue/TestGlueNamespaceIntegration.java

@@ -137,11 +137,13 @@ public void setUp() {
   @AfterEach
   public void tearDown() {
     // Clean up test resources
-    for (String dbName : createdDatabases) {
-      try {
-        cleanupDatabase(dbName);
-      } catch (Exception e) {
-        // Ignore cleanup errors
+    if (createdDatabases != null) {
+      for (String dbName : createdDatabases) {
+        try {
+          cleanupDatabase(dbName);
+        } catch (Exception e) {
+          // Ignore cleanup errors
+        }
       }
     }
 

java/lance-namespace-iceberg/src/test/java/org/lance/namespace/iceberg/TestIcebergNamespaceConfig.java

@@ -0,0 +1,99 @@
+/*
+ * Licensed 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.
+ */
+package org.lance.namespace.iceberg;
+
+import org.junit.jupiter.api.Test;
+
+import java.util.HashMap;
+import java.util.Map;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNull;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+
+public class TestIcebergNamespaceConfig {
+
+  @Test
+  public void testConfigWithEndpoint() {
+    Map<String, String> properties = new HashMap<>();
+    properties.put("endpoint", "http://localhost:8181");
+
+    IcebergNamespaceConfig config = new IcebergNamespaceConfig(properties);
+
+    assertEquals("http://localhost:8181", config.getEndpoint());
+    assertEquals(System.getProperty("user.dir"), config.getRoot());
+    assertNull(config.getAuthToken());
+    assertNull(config.getCredential());
+    assertEquals(10000, config.getConnectTimeout());
+    assertEquals(30000, config.getReadTimeout());
+    assertEquals(3, config.getMaxRetries());
+  }
+
+  @Test
+  public void testConfigWithAllProperties() {
+    Map<String, String> properties = new HashMap<>();
+    properties.put("endpoint", "https://iceberg.example.com");
+    properties.put("auth_token", "test_token");
+    properties.put("credential", "client_id:client_secret");
+    properties.put("connect_timeout", "5000");
+    properties.put("read_timeout", "60000");
+    properties.put("max_retries", "5");
+    properties.put("root", "/data/lance");
+
+    IcebergNamespaceConfig config = new IcebergNamespaceConfig(properties);
+
+    assertEquals("https://iceberg.example.com", config.getEndpoint());
+    assertEquals("test_token", config.getAuthToken());
+    assertEquals("client_id:client_secret", config.getCredential());
+    assertEquals(5000, config.getConnectTimeout());
+    assertEquals(60000, config.getReadTimeout());
+    assertEquals(5, config.getMaxRetries());
+    assertEquals("/data/lance", config.getRoot());
+  }
+
+  @Test
+  public void testMissingEndpointThrowsException() {
+    Map<String, String> properties = new HashMap<>();
+
+    assertThrows(IllegalArgumentException.class, () -> new IcebergNamespaceConfig(properties));
+  }
+
+  @Test
+  public void testEmptyEndpointThrowsException() {
+    Map<String, String> properties = new HashMap<>();
+    properties.put("endpoint", "");
+
+    assertThrows(IllegalArgumentException.class, () -> new IcebergNamespaceConfig(properties));
+  }
+
+  @Test
+  public void testGetBaseApiUrlStripsTrailingSlash() {
+    Map<String, String> properties = new HashMap<>();
+    properties.put("endpoint", "http://localhost:8181/");
+
+    IcebergNamespaceConfig config = new IcebergNamespaceConfig(properties);
+
+    assertEquals("http://localhost:8181", config.getBaseApiUrl());
+  }
+
+  @Test
+  public void testGetBaseApiUrlWithoutTrailingSlash() {
+    Map<String, String> properties = new HashMap<>();
+    properties.put("endpoint", "http://localhost:8181");
+
+    IcebergNamespaceConfig config = new IcebergNamespaceConfig(properties);
+
+    assertEquals("http://localhost:8181", config.getBaseApiUrl());
+  }
+}

python/tests/test_iceberg.py

@@ -39,15 +39,13 @@ def test_config_initialization(self):
             "endpoint": "https://iceberg.example.com",
             "root": "/data/lance",
             "auth_token": "test_token",
-            "warehouse": "test_warehouse",
         }
 
         config = IcebergNamespaceConfig(properties)
 
         self.assertEqual(config.endpoint, "https://iceberg.example.com")
         self.assertEqual(config.root, "/data/lance")
         self.assertEqual(config.auth_token, "test_token")
-        self.assertEqual(config.warehouse, "test_warehouse")
 
     def test_config_defaults(self):
         """Test configuration with default values."""
@@ -59,7 +57,6 @@ def test_config_defaults(self):
 
         self.assertEqual(config.root, os.getcwd())
         self.assertIsNone(config.auth_token)
-        self.assertIsNone(config.warehouse)
         self.assertEqual(config.connect_timeout, 10000)
         self.assertEqual(config.read_timeout, 30000)
         self.assertEqual(config.max_retries, 3)