Add warning in docs

Author: rambleraptor

mkdocs/docs/contributing.md

@@ -157,9 +157,18 @@ To rebuild the containers from scratch.
 
 #### Running Integration Tests against REST Catalogs
 
+!!! warning "Do not run against production catalogs"
+    The integration tests will delete data throughout the entirety of your catalog. Running these integration tests against production catalogs will result in data loss.
+
 PyIceberg supports the ability to run our catalog tests against an arbitrary REST Catalog.
 
-You can set the test catalog in the ~/.pyiceberg.yaml file:
+In order to run the test catalog, you will need to specify which REST catalog to run against with the `PYICEBERG_TEST_CATALOG` environment variable
+
+```sh
+export PYICEBERG_TEST_CATALOG=test_catalog
+```
+
+The catalog in question can be configured either through the ~/.pyiceberg.yaml file or through environment variables.
 
 ```yaml
 catalog:
@@ -168,8 +177,6 @@ catalog:
     credential: t-1234:secret
 ```
 
-You can additionally set the properties using environment variables
-
 ```sh
 export PYICEBERG_CATALOG__TEST_CATALOG__URI=thrift://localhost:9083
 export PYICEBERG_CATALOG__TEST_CATALOG__ACCESS_KEY_ID=username

tests/integration/test_catalog.py

@@ -21,7 +21,7 @@
 
 import pytest
 
-from pyiceberg.catalog import Catalog, MetastoreCatalog
+from pyiceberg.catalog import Catalog, MetastoreCatalog, load_catalog
 from pyiceberg.catalog.hive import HiveCatalog
 from pyiceberg.catalog.memory import InMemoryCatalog
 from pyiceberg.catalog.rest import RestCatalog
@@ -78,7 +78,7 @@ def rest_catalog() -> Generator[Catalog, None, None]:
 @pytest.fixture(scope="function")
 def test_catalog() -> Generator[Catalog, None, None]:
     if test_catalog_name := os.environ.get("PYICEBERG_TEST_CATALOG"):
-        test_catalog = RestCatalog(test_catalog_name)
+        test_catalog = load_catalog(test_catalog_name)
         yield test_catalog
         clean_up(test_catalog)
     else: