From c36971eafdd664e1df2f86b958e70c4a1a08ea71 Mon Sep 17 00:00:00 2001 From: Shay Palachy Date: Fri, 13 Mar 2026 14:23:22 +0200 Subject: [PATCH 1/3] docs: update AGENTS local testing workflows --- AGENTS.md | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/AGENTS.md b/AGENTS.md index 50fa2af3..9af47bc7 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -439,10 +439,13 @@ ______________________________________________________________________ - `make test-all-local` - Test all backends with Docker - `make test-external` - Test all external backends - `make test-mongo-local` - Test MongoDB only +- `make test-mongo-inmemory` - Test MongoDB marker set with local/in-memory setup +- `make test-mongo-also-local` - Test MongoDB together with local core tests - `make test-redis-local` - Test Redis only - `make test-sql-local` - Test SQL only - `make services-start` - Start all Docker containers - `make services-stop` - Stop all Docker containers +- `make services-logs` - Tail logs for Dockerized test services **Available Cores:** @@ -462,8 +465,11 @@ ______________________________________________________________________ - `-k, --keep-running` - Keep containers running after tests - `-h, --html-coverage` - Generate HTML coverage report - `-f, --files` - Run only specific test files +- `-p, --parallel` - Run tests with `pytest-xdist` +- `-w, --workers` - Set number of parallel workers (default: `auto`) **Note:** External backends (MongoDB, Redis, SQL) require Docker. S3, memory, and pickle backends work without Docker. +You can also set cores with `CACHIER_TEST_CORES="mongo redis" ./scripts/test-local.sh`. ______________________________________________________________________ From 7165a1f3ada47deff4c6e36569deb560c78ca472 Mon Sep 17 00:00:00 2001 From: Copilot <198982749+Copilot@users.noreply.github.com> Date: Sat, 14 Mar 2026 23:57:28 +0200 Subject: [PATCH 2/3] fix: replace non-existent test-mongo-local.sh calls in Makefile and clarify MongoDB Docker requirement (#366) * Initial plan * docs: fix broken Makefile mongo targets and clarify MongoDB Docker requirement in AGENTS.md Co-authored-by: shaypal5 <917954+shaypal5@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: shaypal5 <917954+shaypal5@users.noreply.github.com> --- AGENTS.md | 2 +- Makefile | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index 9af47bc7..ccafcc92 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -468,7 +468,7 @@ ______________________________________________________________________ - `-p, --parallel` - Run tests with `pytest-xdist` - `-w, --workers` - Set number of parallel workers (default: `auto`) -**Note:** External backends (MongoDB, Redis, SQL) require Docker. S3, memory, and pickle backends work without Docker. +**Note:** Redis and SQL backends require Docker. MongoDB can run either in-memory (default, no Docker needed) or against a Dockerized instance when using `make test-mongo-local` / `./scripts/test-local.sh mongo`. S3, memory, and pickle backends work without Docker. You can also set cores with `CACHIER_TEST_CORES="mongo redis" ./scripts/test-local.sh`. ______________________________________________________________________ diff --git a/Makefile b/Makefile index 9369a313..3fba9e7f 100644 --- a/Makefile +++ b/Makefile @@ -69,13 +69,13 @@ test-mongo-inmemory: test-mongo-docker: @echo "Running MongoDB tests against Docker MongoDB..." - ./scripts/test-mongo-local.sh + ./scripts/test-local.sh mongo test-mongo-local: test-mongo-docker test-mongo-also-local: @echo "Running MongoDB tests with local core tests..." - ./scripts/test-mongo-local.sh --mode also-local + ./scripts/test-local.sh mongo memory pickle # New unified testing targets test-local: From e4e1d9b24acdd7ec61b2de6d1d97d4cd4d0e3de4 Mon Sep 17 00:00:00 2001 From: Shay Palachy-Affek Date: Sun, 15 Mar 2026 00:16:02 +0200 Subject: [PATCH 3/3] Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- AGENTS.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index ccafcc92..4d41cd59 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -468,8 +468,8 @@ ______________________________________________________________________ - `-p, --parallel` - Run tests with `pytest-xdist` - `-w, --workers` - Set number of parallel workers (default: `auto`) -**Note:** Redis and SQL backends require Docker. MongoDB can run either in-memory (default, no Docker needed) or against a Dockerized instance when using `make test-mongo-local` / `./scripts/test-local.sh mongo`. S3, memory, and pickle backends work without Docker. -You can also set cores with `CACHIER_TEST_CORES="mongo redis" ./scripts/test-local.sh`. +**Note:** Redis and SQL backends require Docker. MongoDB tests run in-memory by default (no Docker needed) when invoked directly (for example, `pytest -m mongo` or `make test-mongo-inmemory` without `CACHIER_TEST_VS_DOCKERIZED_MONGO` set). When using `./scripts/test-local.sh mongo` or including `mongo` in the core list, MongoDB is always run via a Docker container and requires Docker. S3, memory, and pickle backends work without Docker. +You can also set cores with `CACHIER_TEST_CORES="mongo redis" ./scripts/test-local.sh`, in which case both MongoDB and Redis will run via Docker. ______________________________________________________________________