From f4de35bc66f76cc10bd99fce65aa2c5d0734a4a5 Mon Sep 17 00:00:00 2001
From: Simon Cropp <simon.cropp@gmail.com>
Date: Wed, 23 Sep 2026 22:09:21 +1000
Subject: [PATCH] Serialise the tests that probe a released port, so their
 unowned record cannot land on another test's owner

---
 src/DiffEngine.Tests/ViewerProtocolTests.cs | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/DiffEngine.Tests/ViewerProtocolTests.cs b/src/DiffEngine.Tests/ViewerProtocolTests.cs
index e16e3e35..e236638f 100644
--- a/src/DiffEngine.Tests/ViewerProtocolTests.cs
+++ b/src/DiffEngine.Tests/ViewerProtocolTests.cs
@@ -1053,6 +1053,9 @@ await Assert.That(await ViewerClient.SendAsync(new(ViewerVerb.List), default, se
     }
 
     [Test]
+    // Serialised: the port it releases and probes is recorded as unowned process wide, and
+    // the OS may already have handed that number to another test's owner.
+    [NotInParallel]
     public async Task AnAbsentOwnerReportsNoOwner()
     {
         ViewerServer.TryBind(0, out var server);
@@ -1119,6 +1122,8 @@ void Record(object? sender, UnobservedTaskExceptionEventArgs args)
     }
 
     [Test]
+    // Serialised for the same reason as AnAbsentOwnerReportsNoOwner.
+    [NotInParallel]
     public async Task AnAbsentOwnerIsNotAnError()
     {
         ViewerServer.TryBind(0, out var server);