Fix race in TestStreamClose introduced by #238

Fixes #242
pion · Oct 18, 2022 · 38d26c7 · 38d26c7
1 parent d13d723
commit 38d26c7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/AUTHORS.txt b/AUTHORS.txt
@@ -12,6 +12,7 @@ chenkaiC4 <[email protected]>
 Hugo Arregui <[email protected]>
 Hugo Arregui <[email protected]>
 Jerko Steiner <[email protected]>
+Jerry Tao <[email protected]>
 John Bradley <[email protected]>
 Konstantin Itskov <[email protected]>
 Lukas Herman <[email protected]>

diff --git a/vnet_test.go b/vnet_test.go
@@ -463,8 +463,8 @@ func TestStreamClose(t *testing.T) {
 				log.Infof("server: received %d bytes (%d)", n, numServerReceived)
 				assert.Equal(t, 0, bytes.Compare(buf[:n], messages[numServerReceived]), "should receive HELLO")
 
-				_, err = stream.Write(buf[:n])
-				assert.NoError(t, err, "should succeed")
+				_, err2 := stream.Write(buf[:n])
+				assert.NoError(t, err2, "should succeed")
 
 				numServerReceived++
 			}