Skip to content

Isolate request-id logging from console-sensitive validation tests #4798

Description

@Widthdom

Summary

The full Release net8 suite can leak a request-id log line into a console-sensitive validation test, causing a non-deterministic stderr assertion failure.

Evidence

At commit 2eac2999c62bc5c6fa9bec51144622faa82229de, this command produced 10,231 passes, 7 skips, and one failure:

dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Release --framework net8.0 --no-build -p:UseSharedCompilation=false

The failing test was CdidxConfigFileTests.RunValidate_PositionalJson_ReturnsStructuredError_Issue3892. Its Assert.Empty() for stderr received a line beginning with [rid=rid:v1:...]. An immediate isolated rerun passed 1/1 in 17 ms.

Expected behavior

Request-id/global logging from concurrently executing tests should not enter console streams owned by a console-sensitive test. The full suite should be deterministic without relying on an isolated retry.

Environment

  • macOS arm64
  • Release / net8.0
  • full CodeIndex.Tests suite

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions