aelassas / servy / 29438746069

push

github

test(service): EnvironmentVariableHelperTests.cs - Dispose 'restores' modified OS variables by deleting them; a pre-existing value would be lost, contradicting the tracking comment (#3108)
test(service): EventHandlerTests.cs - both writer tests' 'IgnoresNullOrEmpty' half is under-pinned: an empty-string WriteLine would pass unverified (#3105)
test(service): TestLogger.cs (Service.IntegrationTests) - log lists are plain List<string> but are written from async process-event threads; concurrent Adds can corrupt or lose entries (#3102)
test(service): ProcessWrapperIntegrationTests.cs - RedirectStreams test bypasses the wrapper and calls UnderlyingProcess.WaitForExit() directly, with no comment explaining the output-drain intent (#3101)
test(service): ProcessWrapperIntegrationTests.cs - PropertySetters_UpdateUnderlyingProcess never reads the underlying process; the round-trip cannot detect a wrapper that stores values locally (#3099)

1912 of 2160 branches covered (88.52%)

Branch coverage included in aggregate %.

13252 of 15377 relevant lines covered (86.18%)

96.07 hits per line