fixes and readme updates

Author: shaypal5

CLAUDE.md

@@ -208,15 +208,6 @@ ______________________________________________________________________
 - `make services-start` - Start all Docker containers
 - `make services-stop` - Stop all Docker containers
 
-**MongoDB-Specific Testing (Legacy):**
-
-The MongoDB-specific script still works:
-
-```bash
-./scripts/test-mongo-local.sh                    # MongoDB only
-./scripts/test-mongo-local.sh --mode also-local # MongoDB + memory, pickle, maxage
-```
-
 **Available Cores:**
 
 - `mongo` - MongoDB backend

README.rst

@@ -541,22 +541,25 @@ Running MongoDB tests against a live MongoDB instance
 
 .. code-block:: bash
 
-  ./scripts/test-mongo-local.sh                    # MongoDB tests only (default)
-  ./scripts/test-mongo-local.sh --mode also-local # MongoDB + memory, pickle, maxage tests
+  # Test MongoDB only
+  ./scripts/test-local.sh mongo
+  
+  # Test MongoDB with local backends
+  ./scripts/test-local.sh mongo memory pickle
 
 This script automatically handles Docker container lifecycle, environment variables, and cleanup. Additional options:
 
-- ``--mode also-local``: Include memory, pickle, and maxage tests alongside MongoDB tests
-- ``--keep-running``: Keep MongoDB container running after tests
-- ``--verbose``: Show verbose output
-- ``--coverage-html``: Generate HTML coverage report
+- ``-v, --verbose``: Show verbose output
+- ``-k, --keep-running``: Keep containers running after tests
+- ``-h, --html-coverage``: Generate HTML coverage report
 
 **Option 2: Using Make**
 
 .. code-block:: bash
 
-  make test-mongo-local  # Run tests with Docker MongoDB
-  make test-mongo-inmemory  # Run tests with in-memory MongoDB (default)
+  make test-mongo-local     # Run MongoDB tests with Docker
+  make test-all-local       # Run all backends with Docker
+  make test-mongo-inmemory  # Run with in-memory MongoDB (default)
 
 **Option 3: Manual setup**
 
@@ -578,6 +581,28 @@ Contributors are encouraged to test against a real MongoDB instance before submi
 **HOWEVER, the tests run against a live MongoDB instance when you submit a PR are the determining tests for deciding whether your code functions correctly against MongoDB.**
 
 
+Testing all backends locally
+-----------------------------
+
+To test all cachier backends (MongoDB, Redis, SQL, Memory, Pickle) locally with Docker:
+
+.. code-block:: bash
+
+  # Test all backends at once
+  ./scripts/test-local.sh all
+
+  # Test only external backends (MongoDB, Redis, SQL)
+  ./scripts/test-local.sh external
+
+  # Test specific combinations
+  ./scripts/test-local.sh mongo redis
+
+  # Keep containers running for debugging
+  ./scripts/test-local.sh all -k
+
+The unified test script automatically manages Docker containers, installs required dependencies, and runs the appropriate test suites. See ``scripts/README-local-testing.md`` for detailed documentation.
+
+
 Adding documentation
 --------------------
 

scripts/README-local-testing.md

@@ -177,19 +177,6 @@ The script automatically sets the required environment variables:
   docker rm cachier-test-mongo cachier-test-redis cachier-test-postgres
   ```
 
-## Integration with test-mongo-local.sh
-
-The previous `test-mongo-local.sh` script still works and is now a specialized version of the general `test-local.sh`:
-
-```bash
-# These are equivalent:
-./scripts/test-mongo-local.sh
-./scripts/test-local.sh mongo
-
-# These are equivalent:
-./scripts/test-mongo-local.sh --mode also-local
-./scripts/test-local.sh mongo memory pickle
-```
 
 ## Best Practices
 

tests/conftest.py

@@ -0,0 +1,25 @@
+"""Pytest configuration and shared fixtures for cachier tests."""
+
+import pytest
+
+
+@pytest.fixture(scope="session", autouse=True)
+def cleanup_mongo_clients():
+    """Clean up any MongoDB clients created during tests.
+    
+    This fixture runs automatically after all tests complete.
+    """
+    # Let tests run
+    yield
+    
+    # Cleanup after all tests
+    try:
+        from tests.test_mongo_core import _test_mongetter
+        if hasattr(_test_mongetter, "client"):
+            # Close the MongoDB client to avoid ResourceWarning
+            _test_mongetter.client.close()
+            # Remove the client attribute so future test runs start fresh
+            delattr(_test_mongetter, "client")
+    except (ImportError, AttributeError):
+        # If the module wasn't imported or client wasn't created, nothing to clean up
+        pass

tests/test_core_lookup.py

@@ -3,7 +3,6 @@
 import pytest
 
 from cachier import cachier, get_global_params
-from cachier.cores.mongo import MissingMongetter
 
 
 def test_get_default_params():

tests/test_mongo_core.py

@@ -25,10 +25,12 @@
     import pymongo
     from pymongo.errors import OperationFailure
     from pymongo.mongo_client import MongoClient
+    from cachier.cores.mongo import MissingMongetter
 except (ImportError, ModuleNotFoundError):
     print("pymongo is not installed; tests requiring pymongo will fail!")
     pymongo = None
     OperationFailure = None
+    MissingMongetter = None
 
     # define a mock MongoClient class that will raise an exception
     # on init, warning that pymongo is not installed