Skip to content

Commit a8f35ff

Browse files
committed
Additional comments regarding Jinja pre-processing required before running a test.
1 parent afe2421 commit a8f35ff

1 file changed

Lines changed: 5 additions & 12 deletions

File tree

src/tests/multi-server/all.mk

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
# done on purpose to allow multiple test variants to share the same base docker compose
99
# environment, while still having unique test YAML files.
1010
#
11-
# Examples:
1211
# make test-5hs-autoaccept
1312
# -> TEST_FILENAME = test-5hs-autoaccept.yml
1413
# -> ENV_COMPOSE_PATH = environments/docker-compose/env-5hs-autoaccept.yml
@@ -21,11 +20,11 @@
2120
# -> TEST_FILENAME = test-2p-2p-4hs-sql-mycustomvariantstring.yml
2221
# -> ENV_COMPOSE_PATH = environments/docker-compose/env-2p-2p-4hs-sql.yml
2322
#
24-
# Compose env selection rule:
25-
# - Start with the full test target name (TEST_NAME).
26-
# - Convert "test-..." to "environments/docker-compose/env-....yml".
27-
# - If that env file does not exist, strip the last "-suffix" segment and try again,
28-
# repeating until a matching env file is found.
23+
# Jinja2 Template Pre-Processing:
24+
#
25+
# Prior to running a test, .j2 files are processed using the config_builder.py script
26+
# of the test-framework.
27+
#
2928

3029
# Find ENV compose file by stripping trailing "-suffix" chunks until a match exists.
3130
# Returns: environments/docker-compose/env-<base-without-test->.yml
@@ -145,9 +144,3 @@ all: $(TEST_NAMES)
145144
# Ensure the target directory exists
146145
$(MULTI_SERVER_BUILD_DIR_REL_PATH):
147146
@mkdir -p "$@"
148-
149-
# Create .venv if it doesn't exist (in the build directory)
150-
$(VENV_DIR): | $(MULTI_SERVER_BUILD_DIR_REL_PATH)
151-
python3 -m venv "$(VENV_DIR)"
152-
153-
venv: $(VENV_DIR)

0 commit comments

Comments
 (0)