fix(e2e): isolate telegram package artifacts · openclaw/openclaw@5820378
vincentkoc
·
2026-06-03
·
via Recent Commits to openclaw:main
| Original file line number | Diff line number | Diff line change |
|---|
@@ -56,6 +56,7 @@ Docs: https://docs.openclaw.ai
|
56 | 56 | - Release/CI/E2E: bound RPC RTT gateway readiness probes so a half-open local HTTP response cannot stall cleanup past the readiness deadline. |
57 | 57 | - Scripts/UI: stop descendant processes from wrapped non-interactive commands when `run-with-env` receives shutdown signals. |
58 | 58 | - Release/CI/E2E: write multi-node update Docker artifacts to unique per-run directories by default so parallel runs cannot overwrite evidence. |
| 59 | +- Release/CI/E2E: write package Telegram Docker artifacts to unique per-run directories by default so parallel live/RTT runs cannot overwrite evidence. |
59 | 60 | - Control UI: lazy-load the usage view so the initial app bundle stays below the chunk warning threshold. |
60 | 61 | - Build: keep Baileys optional image backends external so source builds do not warn about missing `jimp` or `sharp`. |
61 | 62 | - Build: render independent CLI startup metadata help snapshots concurrently to cut cold build-all metadata time. |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -11,7 +11,8 @@ DOCKER_TARGET="${OPENCLAW_NPM_TELEGRAM_DOCKER_TARGET:-build}"
|
11 | 11 | PACKAGE_SPEC="${OPENCLAW_NPM_TELEGRAM_PACKAGE_SPEC:-openclaw@beta}" |
12 | 12 | PACKAGE_TGZ="${OPENCLAW_NPM_TELEGRAM_PACKAGE_TGZ:-${OPENCLAW_CURRENT_PACKAGE_TGZ:-}}" |
13 | 13 | PACKAGE_LABEL="${OPENCLAW_NPM_TELEGRAM_PACKAGE_LABEL:-}" |
14 | | -OUTPUT_DIR="${OPENCLAW_NPM_TELEGRAM_OUTPUT_DIR:-.artifacts/qa-e2e/npm-telegram-live}" |
| 14 | +RUN_ID="${OPENCLAW_NPM_TELEGRAM_RUN_ID:-$(date -u +%Y%m%dT%H%M%SZ)-$$}" |
| 15 | +OUTPUT_DIR="${OPENCLAW_NPM_TELEGRAM_OUTPUT_DIR:-.artifacts/qa-e2e/npm-telegram-live/$RUN_ID}" |
15 | 16 | |
16 | 17 | resolve_credential_source() { |
17 | 18 | if [ -n "${OPENCLAW_NPM_TELEGRAM_CREDENTIAL_SOURCE:-}" ]; then |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -9,7 +9,8 @@ DOCKER_TARGET="${OPENCLAW_NPM_TELEGRAM_DOCKER_TARGET:-build}"
|
9 | 9 | PACKAGE_SPEC="${OPENCLAW_NPM_TELEGRAM_PACKAGE_SPEC:-openclaw@beta}" |
10 | 10 | PACKAGE_TGZ="${OPENCLAW_NPM_TELEGRAM_PACKAGE_TGZ:-${OPENCLAW_CURRENT_PACKAGE_TGZ:-}}" |
11 | 11 | PACKAGE_LABEL="${OPENCLAW_NPM_TELEGRAM_PACKAGE_LABEL:-}" |
12 | | -OUTPUT_DIR="${OPENCLAW_NPM_TELEGRAM_OUTPUT_DIR:-.artifacts/qa-e2e/npm-telegram-rtt}" |
| 12 | +RUN_ID="${OPENCLAW_NPM_TELEGRAM_RUN_ID:-$(date -u +%Y%m%dT%H%M%SZ)-$$}" |
| 13 | +OUTPUT_DIR="${OPENCLAW_NPM_TELEGRAM_OUTPUT_DIR:-.artifacts/qa-e2e/npm-telegram-rtt/$RUN_ID}" |
13 | 14 | |
14 | 15 | resolve_credential_source() { |
15 | 16 | if [ -n "${OPENCLAW_NPM_TELEGRAM_CREDENTIAL_SOURCE:-}" ]; then |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -103,6 +103,21 @@ describe("package Telegram live Docker E2E", () => {
|
103 | 103 | ); |
104 | 104 | }); |
105 | 105 | |
| 106 | +it("keeps live Docker artifacts isolated by default", () => { |
| 107 | +const script = readFileSync(DOCKER_SCRIPT_PATH, "utf8"); |
| 108 | + |
| 109 | +expect(script).toContain( |
| 110 | +'RUN_ID="${OPENCLAW_NPM_TELEGRAM_RUN_ID:-$(date -u +%Y%m%dT%H%M%SZ)-$$}"', |
| 111 | +); |
| 112 | +expect(script).toContain( |
| 113 | +'OUTPUT_DIR="${OPENCLAW_NPM_TELEGRAM_OUTPUT_DIR:-.artifacts/qa-e2e/npm-telegram-live/$RUN_ID}"', |
| 114 | +); |
| 115 | +expect(script).toContain('-e OPENCLAW_NPM_TELEGRAM_OUTPUT_DIR="$OUTPUT_DIR"'); |
| 116 | +expect(script).not.toContain( |
| 117 | +'OUTPUT_DIR="${OPENCLAW_NPM_TELEGRAM_OUTPUT_DIR:-.artifacts/qa-e2e/npm-telegram-live}"', |
| 118 | +); |
| 119 | +}); |
| 120 | + |
106 | 121 | it("keeps private QA harness imports local while using the installed package dist", () => { |
107 | 122 | const script = readFileSync(DOCKER_SCRIPT_PATH, "utf8"); |
108 | 123 | const preparePackage = readFileSync(PREPARE_PACKAGE_PATH, "utf8"); |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -210,6 +210,21 @@ describe("RTT harness", () => {
|
210 | 210 | expect(script).not.toContain('export TELEGRAM_BOT_TOKEN="$OPENCLAW_QA_TELEGRAM_SUT_BOT_TOKEN"'); |
211 | 211 | }); |
212 | 212 | |
| 213 | +it("keeps RTT Docker artifacts isolated by default", async () => { |
| 214 | +const script = await fs.readFile(DOCKER_SCRIPT_PATH, "utf8"); |
| 215 | + |
| 216 | +expect(script).toContain( |
| 217 | +'RUN_ID="${OPENCLAW_NPM_TELEGRAM_RUN_ID:-$(date -u +%Y%m%dT%H%M%SZ)-$$}"', |
| 218 | +); |
| 219 | +expect(script).toContain( |
| 220 | +'OUTPUT_DIR="${OPENCLAW_NPM_TELEGRAM_OUTPUT_DIR:-.artifacts/qa-e2e/npm-telegram-rtt/$RUN_ID}"', |
| 221 | +); |
| 222 | +expect(script).toContain('-e OPENCLAW_NPM_TELEGRAM_OUTPUT_DIR="$OUTPUT_DIR"'); |
| 223 | +expect(script).not.toContain( |
| 224 | +'OUTPUT_DIR="${OPENCLAW_NPM_TELEGRAM_OUTPUT_DIR:-.artifacts/qa-e2e/npm-telegram-rtt}"', |
| 225 | +); |
| 226 | +}); |
| 227 | + |
213 | 228 | it("keeps broker helper heartbeat handling aligned with QA leases", async () => { |
214 | 229 | const script = await fs.readFile(CREDENTIAL_SCRIPT_PATH, "utf8"); |
215 | 230 | |
|
此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。 原文来自 — 版权归原作者所有。
