aelassas / servy / 27973125169

86%

push

github

fix(infra): ServiceRepository.cs - read methods arbitrarily mix CommandDefinition-wrapping vs direct-parameter calls, and 'const string' vs 'var' for fixed SQL (#3275)
fix(desktop): ServiceCommands.cs / ServiceConfigurationValidator.cs (Servy) - import path validates account credentials against the live OS; no importMode bypass like the Manager (#3306) (#3737)
fix(service): StartOptions.cs - EnableConsoleUI summary deviates from the 'Gets or sets ...' convention and mis-attributes the console to the service (parallel to #3958) (#3967)
fix(manager): IAppConfiguration.cs - SearchDebounceDelayMs/MaxBulkOperationParallelism <summary> docs use a bare noun-phrase instead of the "Gets the…" style of the other 9 members (#3952)
fix(manager): HistoryResult.cs - constructor 'lines' param is non-nullable List<LogLine> but is null-coalesced (test must pass null!); should be List<LogLine>? (#3812)
fix(manager): MainViewModel.cs vs LogsViewModel.cs - same GetRowsInfo helper called via partial qualification in one and a namespace import in the other (#3335)
fix(manager): LogsViewModel.cs - Search failure is logged but never surfaced to the user, indistinguishable from zero results (siblings show a message box) (#3314)
fix(manager): ServiceCommands.cs (Manager) - UninstallServiceAsync wraps RemoveService in Task.Run while RemoveServiceAsync calls it directly; the callee already marshals to the dispatcher (#3294)
fix(core): Domain/Service.cs - Post-Launch/Pre-Stop/Post-Stop hook properties use terse XML docs that drop the dominant 'Gets or sets' prefix (#4050)
fix(service): ProcessLaunchOptions.cs - TimeoutMs and EnableConsoleUI <summary> docs deviate from the "Gets or sets ..." convention (EnableConsoleUI also mis-attributes the console to the service) (#3958)

1928 of 2192 branches covered (87.96%)

Branch coverage included in aggregate %.

15 of 15 new or added lines in 7 files covered. (100.0%)

13067 of 15375 relevant lines covered (84.99%)

92.18 hits per line