aelassas / servy / 29438748380

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
test(service): EventHandlerTests.cs - both writer tests' 'IgnoresNullOrEmpty' half is under-pinned: an empty-string WriteLine would pass unverified
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
test(service): ProcessWrapperIntegrationTests.cs - RedirectStreams test bypasses the wrapper and calls UnderlyingProcess.WaitForExit() directly, with no comment explaining the output-drain intent
test(service): ProcessWrapperIntegrationTests.cs - PropertySetters_UpdateUnderlyingProcess never reads the underlying process; the round-trip cannot detect a wrapper that stores values locally

1906 of 2151 branches covered (88.61%)

Branch coverage included in aggregate %.

13287 of 15422 relevant lines covered (86.16%)

105.96 hits per line