diff --git a/dax/controller/client/client.go b/dax/controller/client/client.go index ef55c68fd..b4e2a27af 100644 --- a/dax/controller/client/client.go +++ b/dax/controller/client/client.go @@ -120,7 +120,6 @@ func (c *Client) DatabaseByID(ctx context.Context, qdbid dax.QualifiedDatabaseID defer resp.Body.Close() if resp.StatusCode != http.StatusOK { - // b, _ := io.ReadAll(resp.Body) return nil, errors.Wrapf(errors.UnmarshalJSON(resp.Body), "status code: %d", resp.StatusCode) } diff --git a/dax/test/dax/dax_test.go b/dax/test/dax/dax_test.go index 666f20380..b4e757f71 100644 --- a/dax/test/dax/dax_test.go +++ b/dax/test/dax/dax_test.go @@ -710,6 +710,9 @@ func TestDAXIntegration(t *testing.T) { } }) + // These tests are to test the traversal of CodedErrors across HTTP + // The client is also wrapped to maintain the integrity of the interface to prevent any + // additional methods added to the client without the appropriate tests for CodedErrors t.Run("HTTPError", func(t *testing.T) { mc := test.MustRunManagedCommand(t) defer mc.Close()