From ddbae60fb31d6536059091308989b4f7c5974568 Mon Sep 17 00:00:00 2001 From: Mikalai Radchuk Date: Wed, 20 Sep 2023 11:05:04 +0100 Subject: [PATCH] Renames `NewClient` to `New` to avoid duplication So we can call `client.New` instead of `client.NewClient` Signed-off-by: Mikalai Radchuk --- internal/catalogmetadata/client/client.go | 2 +- internal/catalogmetadata/client/client_test.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/internal/catalogmetadata/client/client.go b/internal/catalogmetadata/client/client.go index b472d14ff..bf743a282 100644 --- a/internal/catalogmetadata/client/client.go +++ b/internal/catalogmetadata/client/client.go @@ -12,7 +12,7 @@ import ( "github.com/operator-framework/operator-controller/internal/catalogmetadata" ) -func NewClient(cl client.Client) *Client { +func New(cl client.Client) *Client { return &Client{cl: cl} } diff --git a/internal/catalogmetadata/client/client_test.go b/internal/catalogmetadata/client/client_test.go index 4b28dadd9..8ec64ca94 100644 --- a/internal/catalogmetadata/client/client_test.go +++ b/internal/catalogmetadata/client/client_test.go @@ -111,7 +111,7 @@ func TestClient(t *testing.T) { ctx := context.Background() objs, expectedBundles := tt.fakeCatalog() - fakeCatalogClient := catalogClient.NewClient( + fakeCatalogClient := catalogClient.New( fake.NewClientBuilder().WithScheme(scheme).WithObjects(objs...).Build(), )