Skip to content

Commit 7165a1f

Browse files
Copilotshaypal5
andauthored
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>
1 parent c36971e commit 7165a1f

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

AGENTS.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -468,7 +468,7 @@ ______________________________________________________________________
468468
- `-p, --parallel` - Run tests with `pytest-xdist`
469469
- `-w, --workers` - Set number of parallel workers (default: `auto`)
470470

471-
**Note:** External backends (MongoDB, Redis, SQL) require Docker. S3, memory, and pickle backends work without Docker.
471+
**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.
472472
You can also set cores with `CACHIER_TEST_CORES="mongo redis" ./scripts/test-local.sh`.
473473

474474
______________________________________________________________________

Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,13 +69,13 @@ test-mongo-inmemory:
6969

7070
test-mongo-docker:
7171
@echo "Running MongoDB tests against Docker MongoDB..."
72-
./scripts/test-mongo-local.sh
72+
./scripts/test-local.sh mongo
7373

7474
test-mongo-local: test-mongo-docker
7575

7676
test-mongo-also-local:
7777
@echo "Running MongoDB tests with local core tests..."
78-
./scripts/test-mongo-local.sh --mode also-local
78+
./scripts/test-local.sh mongo memory pickle
7979

8080
# New unified testing targets
8181
test-local:

0 commit comments

Comments
 (0)