umputun / revdiff / 24880187567

89%

master: 90%

Build Type

Pull #149

github

Committed by umputun

Commit Message

fix(themes): heal misplaced theme line and round-trip tests

Code-review followup to the #148 fix.

The replace-in-place branch of patchConfigTheme was matching the first
`theme = ...` line anywhere in the file and rewriting it at the same
location. If a user's config was already corrupted by the pre-fix persist
path (theme line stuck inside `[color options]`), the new binary kept
rewriting at the wrong location and go-flags still reported
`unknown option: theme` on next startup.

Make the scan section-aware: track the active INI section while walking
lines; replace in place only when the found line lives in the default
scope ([Application Options] or the unnamed top-of-file section);
otherwise delete the stray line and run the insertion path so the new
entry lands before the first non-[Application Options] header.

Also split the scan and insertion-position logic into two helper methods
on *themeCatalog to keep cyclomatic complexity within the linter limit.

Add an end-to-end regression test that round-trips patched files through
flags.NewIniParser and asserts opts.Theme resolves without error,
covering three testdata fixtures (good / no-theme-yet / corrupted). This
is the assertion that would have caught #148 originally.

Related to #148

Pull Request Pull Request #149: fix(themes): persist theme into Application Options section

Coverage Stats

39 of 41 new or added lines in 1 file covered. (95.12%)

1 existing line in 1 file now uncovered.

8432 of 9426 relevant lines covered (89.45%)

551.88 hits per line