IJHack / QtPass / 27721961015

55%

push

github

refactor(#1511): consolidate initExecutables() into one AppSettings save (#1565) (#1565)

initExecutables() already loaded an AppSettings snapshot then wrote four
executable paths back via individual setters. Rewrite to modify the snapshot
in place and call save() once; delete the four now-dead setters:
setPassExecutable, setGitExecutable, setGpgExecutable, setPwgenExecutable.

Update all test callers to the load → modify field(s) → save pattern:
- tst_mainwindow.cpp: three setGpgExecutable calls (initTestCase, init, cleanup)
- tst_integration.cpp: initTestCase setGpgExecutable; imitatePass_gitInitAndCommit
  setGitExecutable (folded into adjacent useGit write); three
  setPassExecutable+setGpgExecutable pairs collapsed into single saves
- tst_util.cpp: replace SettingGuard<QString, setGpgExecutable> + bare
  setGpgExecutable with an inline RestoreGpg RAII struct and explicit
  load→modify→save writes

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

6 of 10 new or added lines in 1 file covered. (60.0%)

3683 of 6682 relevant lines covered (55.12%)

28.98 hits per line