[receiver/haproxy] Unit test timing out (#32940)

I wasn't able to reproduce the exact situation from CI, but the tests I changed weren't completing at all here. Looking closely, it looks like they never closed the stream, something the first test does. After closing it, the tests start passing locally for me. While I'm not confident this will fix the CI flaky failures, this does make the test work locally for me. Fixes #32877 Signed-off-by: Juraci Paixão Kröhling <[email protected]> Signed-off-by: Juraci Paixão Kröhling <[email protected]> Co-authored-by: Alex Boten <[email protected]>
open-telemetry · May 8, 2024 · ba9ef9e · ba9ef9e
1 parent f1f2a85
commit ba9ef9e
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/receiver/haproxyreceiver/scraper_test.go b/receiver/haproxyreceiver/scraper_test.go
@@ -85,6 +85,7 @@ func Test_scraper_readStatsWithIncompleteValues(t *testing.T) {
 			require.NoError(t, err2)
 			_, err2 = c.Write(stats)
 			require.NoError(t, err2)
+			require.NoError(t, c.Close())
 		default:
 			require.Fail(t, fmt.Sprintf("invalid message: %v", data))
 		}
@@ -128,6 +129,7 @@ func Test_scraper_readStatsWithNoValues(t *testing.T) {
 			require.NoError(t, err2)
 			_, err2 = c.Write(stats)
 			require.NoError(t, err2)
+			require.NoError(t, c.Close())
 		default:
 			require.Fail(t, fmt.Sprintf("invalid message: %v", data))
 		}