Skip to content

Commit 7375feb

Browse files
JamesNKCopilot
andcommitted
Apply suggestion from @Copilot
Co-authored-by: Copilot <[email protected]>
1 parent dc03618 commit 7375feb

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

tests/Aspire.Dashboard.Tests/Integration/StartupTests.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -756,6 +756,8 @@ await ServerRetryHelper.BindPortsWithRetry(async ports =>
756756
otlpHttpPort = ports[3];
757757

758758
// Reset sink writes. Required to clear out data from a previous failed retry.
759+
// The following cast relies on the internal implementation detail that TestSink.Writes is a ConcurrentQueue<WriteContext>.
760+
// If the implementation of TestSink changes, this may break. There is no public API to clear the writes.
759761
var writes = (ConcurrentQueue<Microsoft.Extensions.Logging.Testing.WriteContext>)testSink.Writes;
760762
writes.Clear();
761763

0 commit comments

Comments
 (0)