tpagectl: add test for flag precedence over config file #541

New issue

Closed

opened 2026-08-05 06:11:02 +00:00 by the.auditor · 1 comment

the.auditor commented

2026-08-05 06:11:02 +00:00

Owner

Config precedence chain (defaults → file → env → flags) is only partially tested. Tests cover defaults, file, env-over-file, env-without-file, and flag-over-env, but not flag-over-file (e.g. --server beating a value from a loaded --config file).

Suggested fix: add a case asserting CLI flags override values from an explicit --config file.

Ref: PR #537

Config precedence chain (defaults → file → env → flags) is only partially tested. Tests cover defaults, file, env-over-file, env-without-file, and flag-over-env, but not flag-over-file (e.g. `--server` beating a value from a loaded `--config` file). Suggested fix: add a case asserting CLI flags override values from an explicit `--config` file. Ref: PR #537

the.auditor added the

Kind/Testing

Priority

Low

labels

2026-08-05 06:11:09 +00:00

fuzzy referenced this issue from a commit

2026-08-11 12:32:47 +00:00

test(tpagectl): close tests-lump audit gaps

automation referenced this issue from a pull request that will close it,

2026-08-11 12:32:52 +00:00

test(tpagectl): close tests-lump audit gaps #635

fuzzy referenced this issue from a commit

2026-08-11 12:36:02 +00:00

test(tpagectl): close tests-lump audit gaps

automation closed this issue

2026-08-11 12:36:10 +00:00

automation commented

2026-08-11 12:36:10 +00:00

Owner

Fixed in #635: TestConfigFlagOverridesFile asserts --server/--token override values from an explicit --config file (completing the defaults → file → env → flags precedence chain).

Fixed in #635: `TestConfigFlagOverridesFile` asserts `--server`/`--token` override values from an explicit `--config` file (completing the defaults → file → env → flags precedence chain).