From: Anderson A. Giacomolli <redacted>
Date: Wed, 11 Nov 2026 15:04:03 +0000 (-0300)
Subject: Check forced-disconnect test setup failures
X-Git-Url: https://git.giacomolli.uk/onechat/commitdiff_plain/8707a09fee94c97e29d668d24fb9713373913e0b?ds=sidebyside

Check forced-disconnect test setup failures

The HTTP failure fixture checks whether it can take over the test
connection before closing it. A setup failure cannot silently become a
different network failure. Only the deliberate disconnect close remains
a justified nonfatal discard.

Issue: 3cdc678
---

diff --git a/telegram_api_test.go b/telegram_api_test.go
index 0da1395..4395d6b 100644
--- a/telegram_api_test.go
+++ b/telegram_api_test.go
@@ -149,11 +149,13 @@ func TestBotErrorsDoNotHoldTheToken(t *testing.T) {
 			name: "refused connection is retried",
 			reply: func(w http.ResponseWriter) {
 				conn, _, err := http.NewResponseController(w).Hijack()
-				if err == nil {
-					// This forced disconnect has no
-					// response or retry after close.
-					conn.Close() // nolint: errcheck
+				if err != nil {
+					panic(err)
 				}
+
+				// This forced disconnect has no response or
+				// retry after close.
+				conn.Close() // nolint: errcheck
 			},
 			retry:     true,
 			errorText: "no answer",