Skip to content

Commit 8107933

Browse files
author
Andrew Brookins
committed
Use default REDIS_OM_URL if not set
1 parent 3b9a2c1 commit 8107933

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

Makefile

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ NAME := aredis_om
22
SYNC_NAME := redis_om
33
INSTALL_STAMP := .install.stamp
44
POETRY := $(shell command -v poetry 2> /dev/null)
5+
REDIS_OM_URL ?= "redis://localhost:6380?decode_responses=True"
56

67
.DEFAULT_GOAL := help
78

@@ -56,16 +57,16 @@ lint: $(INSTALL_STAMP) dist
5657
$(POETRY) run twine check dist/*
5758

5859
.PHONY: format
59-
format: $(INSTALL_STAMP) sync
60+
format: $(INSTALL_STAMP) sync redis
6061
$(POETRY) run isort --profile=black --lines-after-imports=2 ./tests/ $(NAME) $(SYNC_NAME)
6162
$(POETRY) run black ./tests/ $(NAME) $(SYNC_NAME)
6263

6364
.PHONY: test
6465
test: $(INSTALL_STAMP) sync
65-
REDIS_OM_URL="redis://localhost:6380?decode_responses=True" $(POETRY) run pytest -n auto -vv ./tests/ ./tests_sync/ --cov-report term-missing --cov $(NAME) $(SYNC_NAME)
66+
REDIS_OM_URL="$(REDIS_OM_URL)" $(POETRY) run pytest -n auto -vv ./tests/ ./tests_sync/ --cov-report term-missing --cov $(NAME) $(SYNC_NAME)
6667

6768
.PHONY: test_oss
68-
test_oss: $(INSTALL_STAMP) sync
69+
test_oss: $(INSTALL_STAMP) sync redis
6970
# Specifically tests against a local OSS Redis instance via
7071
# docker-compose.yml. Do not use this for CI testing, where we should
7172
# instead have a matrix of Docker images.

0 commit comments

Comments
 (0)