From 09a2a1048d83ba098c40e05fc01a0c7128e80866 Mon Sep 17 00:00:00 2001
From: "M. J. Fromberger" <fromberger@tailscale.com>
Date: Tue, 28 Oct 2025 14:20:19 -0700
Subject: [PATCH] derp: fix an unchecked error in a test (#17694)

Found by staticcheck, the test was calling derphttp.NewClient but not checking
its error result before doing other things to it.

Updates #cleanup

Change-Id: I4ade35a7de7c473571f176e747866bc0ab5774db
Signed-off-by: M. J. Fromberger <fromberger@tailscale.com>
---
 derp/derphttp/derphttp_test.go | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/derp/derphttp/derphttp_test.go b/derp/derphttp/derphttp_test.go
index 76681d498..5208481ed 100644
--- a/derp/derphttp/derphttp_test.go
+++ b/derp/derphttp/derphttp_test.go
@@ -620,6 +620,9 @@ func TestURLDial(t *testing.T) {
 	}
 	netMon := netmon.NewStatic()
 	c, err := derphttp.NewClient(key.NewNode(), "https://"+hostname+"/", t.Logf, netMon)
+	if err != nil {
+		t.Errorf("NewClient: %v", err)
+	}
 	defer c.Close()
 
 	if err := c.Connect(context.Background()); err != nil {