diff --git a/pkg/client/client_test.go b/pkg/client/client_test.go index 4d655ec..759bfd4 100644 --- a/pkg/client/client_test.go +++ b/pkg/client/client_test.go @@ -159,7 +159,7 @@ func TestGetDeviceInfo_Success(t *testing.T) { func TestGetDeviceInfo_HTTPError(t *testing.T) { // Create mock server that returns 404 - server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) { w.WriteHeader(http.StatusNotFound) _, _ = w.Write([]byte("Not Found")) })) @@ -181,7 +181,7 @@ func TestGetDeviceInfo_HTTPError(t *testing.T) { func TestGetDeviceInfo_InvalidXML(t *testing.T) { // Create mock server that returns invalid XML - server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) { w.Header().Set("Content-Type", "application/xml") w.WriteHeader(http.StatusOK) _, _ = w.Write([]byte("invalid xml content")) @@ -204,7 +204,7 @@ func TestGetDeviceInfo_InvalidXML(t *testing.T) { func TestGetDeviceInfo_APIError(t *testing.T) { // Create mock server that returns API error - server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) { w.Header().Set("Content-Type", "application/xml") w.WriteHeader(http.StatusOK) _, _ = w.Write([]byte(`Device not found`)) @@ -228,7 +228,7 @@ func TestGetDeviceInfo_APIError(t *testing.T) { func TestPing_Success(t *testing.T) { testData := loadTestData(t, "info_response.xml") - server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) { w.Header().Set("Content-Type", "application/xml") w.WriteHeader(http.StatusOK) _, _ = w.Write([]byte(testData)) @@ -244,7 +244,7 @@ func TestPing_Success(t *testing.T) { } func TestPing_Failure(t *testing.T) { - server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) { w.WriteHeader(http.StatusInternalServerError) })) defer server.Close() @@ -268,7 +268,7 @@ func TestBaseURL(t *testing.T) { func TestClientTimeout(t *testing.T) { // Create a server that delays response - server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) { time.Sleep(200 * time.Millisecond) w.WriteHeader(http.StatusOK) _, _ = w.Write([]byte(``)) @@ -419,7 +419,7 @@ func TestClient_GetNowPlaying(t *testing.T) { } func TestClient_GetNowPlaying_ServerError(t *testing.T) { - server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) { w.WriteHeader(http.StatusInternalServerError) _, _ = w.Write([]byte("Internal Server Error")) })) @@ -465,7 +465,7 @@ func TestClient_GetNowPlaying_NetworkError(t *testing.T) { } func TestClient_GetNowPlaying_InvalidXML(t *testing.T) { - server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) { w.Header().Set("Content-Type", "application/xml") w.WriteHeader(http.StatusOK) _, _ = w.Write([]byte("")) @@ -608,7 +608,7 @@ func TestClient_GetSources(t *testing.T) { } func TestClient_GetSources_ServerError(t *testing.T) { - server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) { w.WriteHeader(http.StatusInternalServerError) _, _ = w.Write([]byte("Internal Server Error")) })) @@ -654,7 +654,7 @@ func TestClient_GetSources_NetworkError(t *testing.T) { } func TestClient_GetSources_InvalidXML(t *testing.T) { - server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) { w.Header().Set("Content-Type", "application/xml") w.WriteHeader(http.StatusOK) _, _ = w.Write([]byte("")) @@ -959,7 +959,7 @@ func TestClient_GetPresets(t *testing.T) { } func TestClient_GetName_ServerError(t *testing.T) { - server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) { w.WriteHeader(http.StatusInternalServerError) _, _ = w.Write([]byte("Internal Server Error")) })) diff --git a/pkg/client/source_selection_test.go b/pkg/client/source_selection_test.go index 3baf0c0..ff6dd62 100644 --- a/pkg/client/source_selection_test.go +++ b/pkg/client/source_selection_test.go @@ -396,7 +396,7 @@ func TestClient_SelectSource_ErrorHandling(t *testing.T) { }{ { name: "Server returns 404", - serverResponse: func(w http.ResponseWriter, r *http.Request) { + serverResponse: func(w http.ResponseWriter, _ *http.Request) { w.WriteHeader(http.StatusNotFound) _, _ = w.Write([]byte("Not Found")) }, @@ -405,7 +405,7 @@ func TestClient_SelectSource_ErrorHandling(t *testing.T) { }, { name: "Server returns 500", - serverResponse: func(w http.ResponseWriter, r *http.Request) { + serverResponse: func(w http.ResponseWriter, _ *http.Request) { w.WriteHeader(http.StatusInternalServerError) _, _ = w.Write([]byte("Internal Server Error")) }, @@ -414,7 +414,7 @@ func TestClient_SelectSource_ErrorHandling(t *testing.T) { }, { name: "Server returns API error", - serverResponse: func(w http.ResponseWriter, r *http.Request) { + serverResponse: func(w http.ResponseWriter, _ *http.Request) { w.WriteHeader(http.StatusBadRequest) apiError := models.APIError{ diff --git a/pkg/client/system_test.go b/pkg/client/system_test.go index 951a469..1ccffbf 100644 --- a/pkg/client/system_test.go +++ b/pkg/client/system_test.go @@ -351,7 +351,7 @@ func TestClient_EnableDisableClockDisplay(t *testing.T) { for _, tt := range tests { t.Run(tt.name, func(t *testing.T) { - server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) { w.WriteHeader(http.StatusOK) })) defer server.Close() diff --git a/pkg/client/websocket_test.go b/pkg/client/websocket_test.go index 6759da8..14e80ac 100644 --- a/pkg/client/websocket_test.go +++ b/pkg/client/websocket_test.go @@ -494,7 +494,7 @@ func BenchmarkWebSocketClient_SetHandlers(b *testing.B) { handlers := &models.WebSocketEventHandlers{ OnNowPlaying: func(_ *models.NowPlayingUpdatedEvent) {}, - OnVolumeUpdated: func(event *models.VolumeUpdatedEvent) {}, + OnVolumeUpdated: func(_ *models.VolumeUpdatedEvent) {}, } b.ResetTimer()