fix: parse matrix qa env timeouts strictly · openclaw/openclaw@d23e411
steipete
·
2026-05-29
·
via Recent Commits to openclaw:main
File tree
extensions/qa-matrix/src/runners/contract
| Original file line number | Diff line number | Diff line change |
|---|
@@ -98,6 +98,10 @@ describe("matrix live qa runtime", () => {
|
98 | 98 | expect(liveTesting.createMatrixQaRunDeadline().timeoutMs).toBe(12345); |
99 | 99 | process.env.OPENCLAW_QA_MATRIX_TIMEOUT_MS = "nope"; |
100 | 100 | expect(liveTesting.createMatrixQaRunDeadline().timeoutMs).toBe(30 * 60_000); |
| 101 | +process.env.OPENCLAW_QA_MATRIX_TIMEOUT_MS = "1e3"; |
| 102 | +expect(liveTesting.createMatrixQaRunDeadline().timeoutMs).toBe(30 * 60_000); |
| 103 | +process.env.OPENCLAW_QA_MATRIX_TIMEOUT_MS = "1.5"; |
| 104 | +expect(liveTesting.createMatrixQaRunDeadline().timeoutMs).toBe(30 * 60_000); |
101 | 105 | } finally { |
102 | 106 | if (previous === undefined) { |
103 | 107 | delete process.env.OPENCLAW_QA_MATRIX_TIMEOUT_MS; |
@@ -147,6 +151,8 @@ describe("matrix live qa runtime", () => {
|
147 | 151 | expect(liveTesting.resolveMatrixQaCanaryTimeoutMs()).toBe(90_000); |
148 | 152 | process.env.OPENCLAW_QA_MATRIX_CANARY_TIMEOUT_MS = "nope"; |
149 | 153 | expect(liveTesting.resolveMatrixQaCanaryTimeoutMs()).toBe(45_000); |
| 154 | +process.env.OPENCLAW_QA_MATRIX_CANARY_TIMEOUT_MS = "0x1000"; |
| 155 | +expect(liveTesting.resolveMatrixQaCanaryTimeoutMs()).toBe(45_000); |
150 | 156 | } finally { |
151 | 157 | if (previous === undefined) { |
152 | 158 | delete process.env.OPENCLAW_QA_MATRIX_CANARY_TIMEOUT_MS; |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -195,11 +195,12 @@ function parsePositiveMatrixQaEnvMs(name: string, fallback: number) {
|
195 | 195 | if (raw === undefined) { |
196 | 196 | return fallback; |
197 | 197 | } |
198 | | -const parsed = Number(raw); |
199 | | -if (!Number.isFinite(parsed) || parsed < 1) { |
| 198 | +const trimmed = raw.trim(); |
| 199 | +const parsed = /^\d+$/.test(trimmed) ? Number(trimmed) : Number.NaN; |
| 200 | +if (!Number.isSafeInteger(parsed) || parsed < 1) { |
200 | 201 | return fallback; |
201 | 202 | } |
202 | | -return Math.floor(parsed); |
| 203 | +return parsed; |
203 | 204 | } |
204 | 205 | |
205 | 206 | function createMatrixQaRunDeadline() { |
|
此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。 原文来自 — 版权归原作者所有。
