fix(e2e): forward codex media log limit

This commit is contained in:
Vincent Koc
2026-06-19 03:10:01 +02:00
parent ccc1ad4c74
commit 2c6bf1a5d8
2 changed files with 19 additions and 3 deletions
+2
View File
@@ -7,6 +7,7 @@ source "$ROOT_DIR/scripts/lib/docker-e2e-image.sh"
IMAGE_NAME="$(docker_e2e_resolve_image "openclaw-codex-media-path-e2e" OPENCLAW_CODEX_MEDIA_PATH_E2E_IMAGE)"
PORT="$(docker_e2e_read_tcp_port_env OPENCLAW_CODEX_MEDIA_PATH_PORT 18790)"
TIMEOUT_SECONDS="$(docker_e2e_read_positive_int_env OPENCLAW_CODEX_MEDIA_PATH_TIMEOUT_SECONDS 180)"
LOG_TAIL_MAX_BYTES="$(docker_e2e_read_positive_int_env OPENCLAW_CODEX_MEDIA_PATH_LOG_TAIL_MAX_BYTES 2097152)"
TOKEN="codex-media-path-e2e-$$"
CODEX_PLUGIN_SPEC="${OPENCLAW_CODEX_MEDIA_PATH_PLUGIN_SPEC:-npm:@openclaw/codex}"
@@ -17,6 +18,7 @@ echo "Running Codex media-path Docker E2E..."
docker_e2e_run_logged_with_harness codex-media-path \
-e COREPACK_ENABLE_DOWNLOAD_PROMPT=0 \
-e "OPENCLAW_CODEX_MEDIA_PATH_PLUGIN_SPEC=$CODEX_PLUGIN_SPEC" \
-e "OPENCLAW_CODEX_MEDIA_PATH_LOG_TAIL_MAX_BYTES=$LOG_TAIL_MAX_BYTES" \
-e "OPENCLAW_CODEX_MEDIA_PATH_TIMEOUT_SECONDS=$TIMEOUT_SECONDS" \
-e "OPENCLAW_ALLOW_INSECURE_PRIVATE_WS=1" \
-e "OPENCLAW_GATEWAY_TOKEN=$TOKEN" \
+17 -3
View File
@@ -3374,21 +3374,35 @@ output="$(cat "$sampler_log")"
expect(result.stderr).not.toContain("OPENAI_API_KEY was not available");
});
it("rejects invalid Codex media path Docker timeouts before Docker setup", () => {
it.each([
["timeout", "OPENCLAW_CODEX_MEDIA_PATH_TIMEOUT_SECONDS", "180s"],
["log tail cap", "OPENCLAW_CODEX_MEDIA_PATH_LOG_TAIL_MAX_BYTES", "64kb"],
])("rejects invalid Codex media path Docker %s before Docker setup", (_label, envName, value) => {
const result = spawnSync("bash", [CODEX_MEDIA_PATH_DOCKER_E2E_PATH], {
encoding: "utf8",
env: {
...process.env,
OPENCLAW_CODEX_MEDIA_PATH_TIMEOUT_SECONDS: "180s",
[envName]: value,
OPENCLAW_SKIP_DOCKER_BUILD: "1",
},
});
expect(result.status).toBe(2);
expect(result.stderr).toContain("invalid OPENCLAW_CODEX_MEDIA_PATH_TIMEOUT_SECONDS: 180s");
expect(result.stderr).toContain(`invalid ${envName}: ${value}`);
expect(result.stderr).not.toContain("Docker image not found");
});
it("forwards Codex media path client limits into Docker", () => {
const runner = readFileSync(CODEX_MEDIA_PATH_DOCKER_E2E_PATH, "utf8");
expect(runner).toContain(
'LOG_TAIL_MAX_BYTES="$(docker_e2e_read_positive_int_env OPENCLAW_CODEX_MEDIA_PATH_LOG_TAIL_MAX_BYTES 2097152)"',
);
expect(runner).toContain(
'-e "OPENCLAW_CODEX_MEDIA_PATH_LOG_TAIL_MAX_BYTES=$LOG_TAIL_MAX_BYTES"',
);
});
it.each([
[MCP_CODE_MODE_GATEWAY_DOCKER_E2E_PATH, "OPENCLAW_MCP_CODE_MODE_CLIENT_TIMEOUT_MS", "1e3"],
[