Renames NewClient to New to avoid duplication

So we can call `client.New` instead of `client.NewClient` Signed-off-by: Mikalai Radchuk <mradchuk@redhat.com>
operator-framework · Sep 20, 2023 · ddbae60 · ddbae60
1 parent 4b9cfd9
commit ddbae60
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git 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
@@ -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(),
 				)