TestStreamStress doesn't wait for the test to finish

[Stebalien]

makePinger and makePonger shouldn't launch goroutines as we don't end up waiting for these goroutines which means we finish the test before it's actually done.

go-libp2p/p2p/net/mock/mock_test.go

Lines 322 to 365 in 198633b

	func makePinger(st string, n int) func(network.Stream) {
	return func(s network.Stream) {
	go func() {
	defer helpers.FullClose(s)
	for i := 0; i < n; i++ {
	b := make([]byte, 4+len(st))
	if _, err := s.Write([]byte("ping" + st)); err != nil {
	panic(err)
	}
	if _, err := io.ReadFull(s, b); err != nil {
	panic(err)
	}
	if !bytes.Equal(b, []byte("pong"+st)) {
	panic("bytes mismatch")
	}
	}
	}()
	}
	}
	func makePonger(st string) func(network.Stream) {
	return func(s network.Stream) {
	go func() {
	defer helpers.FullClose(s)
	for {
	b := make([]byte, 4+len(st))
	if _, err := io.ReadFull(s, b); err != nil {
	if err == io.EOF {
	return
	}
	panic(err)
	}
	if !bytes.Equal(b, []byte("ping"+st)) {
	panic("bytes mismatch")
	}
	if _, err := s.Write([]byte("pong" + st)); err != nil {
	panic(err)
	}
	}
	}()
	}
	}