Skip to content

Commit 8ed01d5

Browse files
tests/clickhouse-test-runner: stop logging argv (#739)
## Summary The harness was logging the entire raw `argv` of every invocation. The previous attempt filtered it through a hand-rolled parser; since we don't go through a real argv-parsing library, drop the log line entirely rather than ship a second parser to maintain. - **`src/main.ts`**: remove the `argv=` log line. - **`src/args.ts`**: remove the `extractKnownArgv` helper introduced in the prior commit. - **`__tests__/args.test.ts`**: remove the corresponding tests. ## Checklist - [x] A human-readable description of the changes was provided to include in CHANGELOG --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Peter Leonov <peter.leonov@clickhouse.com>
1 parent 3b2c176 commit 8ed01d5

2 files changed

Lines changed: 1 addition & 2 deletions

File tree

package-lock.json

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

tests/clickhouse-test-runner/src/main.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ async function main(): Promise<void> {
1717
const logPath = resolveLogPath()
1818
appendLog(logPath, '=== clickhouse-client invocation ===')
1919
appendLog(logPath, 'timestamp=' + new Date().toISOString())
20-
appendLog(logPath, 'argv=' + JSON.stringify(argv))
2120

2221
let args
2322
try {

0 commit comments

Comments
 (0)