aelassas / servy / 27934443767

push

github

fix(core): IServiceValidationRules.cs - Validate optional confirmPassword defaults to empty string while sibling wrapperExePath defaults to null (undocumented sentinel divergence) (#3643)
fix(core): InstallServiceOptions.cs - MaxRestartAttempts / RecoveryOnCleanExit docs omit the "Only used when EnableHealthMonitoring is true" caveat their recovery/health siblings carry (#3938)
fix(desktop): MainViewModel.cs (Servy) - AppConfig_PropertyChanged null-checks 'e' (e?.PropertyName) but force-unwraps 'sender' (sender!) in the same handler (#3850)
fix(service): ProcessWrapper.cs - raw 1000.0 ms→s literal instead of AppConfig.MillisecondsPerSecond (used in ServiceHelper) (#3669)
fix(service): IServiceHelper.cs / ServiceHelper.cs (Service) - four methods declare non-nullable IServyLogger but dereference it with logger?. (siblings use IServyLogger?) (#3664)
fix(core): EventLogLogger.cs - null message mishandled (swallowed NRE + misleading 'EventLog write failed' warn / junk entry), unlike the null-tolerant static Logger (#3926)
fix(service): IProcessWrapper.cs - MainWindowHandle and PriorityClass properties are declared in the middle of the method block, breaking the properties-first grouping (#3969)
fix(manager): ServiceSearchViewModelBase.cs - ServiceCommands is a public mutable property (null-checked in SearchServicesAsync) while sibling deps are readonly ctor-injected fields (#3965)
fix(core): JsonServiceSerializer.cs - null-guard comment references a unit-test method name (Deserialize_NullOrEmpty_ReturnsNull), unlike its XmlServiceSerializer sibling (#2601)

1928 of 2192 branches covered (87.96%)

Branch coverage included in aggregate %.

9 of 10 new or added lines in 5 files covered. (90.0%)

13097 of 15403 relevant lines covered (85.03%)

93.09 hits per line