fix: correct conditions in namespace

Author: dingo4dev

Makefile

@@ -117,7 +117,7 @@ test-integration-rebuild: ## Rebuild integration Docker services from scratch
 	docker compose -f dev/docker-compose-integration.yml rm -f
 	docker compose -f dev/docker-compose-integration.yml build --no-cache
 
-test-integration-trino:
+test-integration-trino: ## Run tests marked with @pytest.mark.integration_trino
 	sh ./dev/run-trino.sh
 	$(TEST_RUNNER) pytest tests/ -m integration_trino $(PYTEST_ARGS)
 
@@ -134,7 +134,7 @@ test-gcs: ## Run tests marked with @pytest.mark.gcs
 	$(TEST_RUNNER) pytest tests/ -m gcs $(PYTEST_ARGS)
 
 test-coverage: COVERAGE=1
-test-coverage: test test-integration test-s3 test-adls test-gcs coverage-report ## Run all tests with coverage and report
+test-coverage: test test-integration test-integration-trino test-s3 test-adls test-gcs coverage-report ## Run all tests with coverage and report
 
 coverage-report: ## Combine and report coverage
 	uv run $(PYTHON_ARG) coverage combine

dev/docker-compose-trino.yml

@@ -1,3 +1,19 @@
+# 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.
 services:
   rest:
     image: apache/iceberg-rest-fixture

dev/trino/catalog/warehouse_hive.properties

@@ -1,3 +1,19 @@
+# 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.
 connector.name=iceberg
 iceberg.catalog.type=hive_metastore
 iceberg.expire-snapshots.min-retention=0d

dev/trino/catalog/warehouse_rest.properties

@@ -1,3 +1,19 @@
+# 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.
 connector.name=iceberg
 iceberg.catalog.type=rest
 iceberg.rest-catalog.uri=http://rest:8181

tests/integration/test_rest_catalog.py

@@ -75,7 +75,7 @@ def test_schema_exists_in_trino(trino_rest_conn: Connection, catalog: RestCatalo
     catalog, it becomes visible as a schema in the Trino environment.
     """
 
-    if not catalog.namespace_exists(TEST_NAMESPACE_IDENTIFIER):
+    if catalog.namespace_exists(TEST_NAMESPACE_IDENTIFIER):
         catalog.drop_namespace(TEST_NAMESPACE_IDENTIFIER)
     catalog.create_namespace_if_not_exists(TEST_NAMESPACE_IDENTIFIER)