Simplify test: use invalid host instead of socket dance

Author: dandavison

internal/temporalcli/commands_test.go

@@ -6,7 +6,6 @@ import (
 	"fmt"
 	"io"
 	"log/slog"
-	"net"
 	"regexp"
 	"slices"
 	"strings"
@@ -632,19 +631,15 @@ func TestErrorReporting_IndependentOfLogLevel(t *testing.T) {
 	for _, logLevel := range []string{"never", "error", "info"} {
 		t.Run("log-level="+logLevel, func(t *testing.T) {
 			h := NewCommandHarness(t)
-			ln, err := net.Listen("tcp", "127.0.0.1:0")
-			require.NoError(t, err)
-			ln.Close() // close immediately so connection is refused
-
 			res := h.Execute(
 				"workflow", "list",
-				"--address", ln.Addr().String(),
+				"--address", "not-a-valid-host:1",
 				"--log-level", logLevel,
 			)
 			require.Error(t, res.Err)
 
 			stderr := res.Stderr.String()
-			assert.NotContains(t, stderr, "level=ERROR",
+			assert.NotContains(t, stderr, "level=",
 				"errors should not appear as structured log messages on stderr")
 		})
 	}