[QUIC] Move 0 byte read before semaphore release on test (#100128)

liveans · web-flow · commit a2ce8db5fb14 · 2024-03-22T11:05:35.000+01:00
diff --git a/src/libraries/System.Net.Quic/tests/FunctionalTests/QuicStreamTests.cs b/src/libraries/System.Net.Quic/tests/FunctionalTests/QuicStreamTests.cs
@@ -1249,11 +1249,12 @@ await RunClientServer(
                     {
                         await stream.WritesClosed;
                     }
-                    serverSem.Release();
-                    await clientSem.WaitAsync();
 
                     var _ = await stream.ReadAsync(new byte[0]);
 
+                    serverSem.Release();
+                    await clientSem.WaitAsync();
+
                     if (closeServer)
                     {
                         expectedError = QuicError.OperationAborted;
@@ -1281,11 +1282,12 @@ await RunClientServer(
                     {
                         await stream.WritesClosed;
                     }
-                    clientSem.Release();
-                    await serverSem.WaitAsync();
 
                     var _ = await stream.ReadAsync(new byte[0]);
 
+                    clientSem.Release();
+                    await serverSem.WaitAsync();
+
                     if (!closeServer)
                     {
                         expectedError = QuicError.OperationAborted;

Original file line number	Diff line number	Diff line change
`@@ -1249,11 +1249,12 @@ await RunClientServer(`
`1249`	`1249`	`{`
`1250`	`1250`	`await stream.WritesClosed;`
`1251`	`1251`	`}`
`1252`		`- serverSem.Release();`
`1253`		`- await clientSem.WaitAsync();`
`1254`	`1252`
`1255`	`1253`	`var _ = await stream.ReadAsync(new byte[0]);`
`1256`	`1254`
	`1255`	`+ serverSem.Release();`
	`1256`	`+ await clientSem.WaitAsync();`
	`1257`	`+`
`1257`	`1258`	`if (closeServer)`
`1258`	`1259`	`{`
`1259`	`1260`	`expectedError = QuicError.OperationAborted;`
`@@ -1281,11 +1282,12 @@ await RunClientServer(`
`1281`	`1282`	`{`
`1282`	`1283`	`await stream.WritesClosed;`
`1283`	`1284`	`}`
`1284`		`- clientSem.Release();`
`1285`		`- await serverSem.WaitAsync();`
`1286`	`1285`
`1287`	`1286`	`var _ = await stream.ReadAsync(new byte[0]);`
`1288`	`1287`
	`1288`	`+ clientSem.Release();`
	`1289`	`+ await serverSem.WaitAsync();`
	`1290`	`+`
`1289`	`1291`	`if (!closeServer)`
`1290`	`1292`	`{`
`1291`	`1293`	`expectedError = QuicError.OperationAborted;`