fix: Removing GetClient from ChromaContainer

Author: tazarov

docs/modules/chroma.md

@@ -59,14 +59,6 @@ This method returns the REST endpoint of the Chroma container, using the default
 [Get REST endpoint](../../modules/chroma/chroma_test.go) inside_block:restEndpoint
 <!--/codeinclude-->
 
-#### GetClient
-
-This method returns a `chroma-go` client that can be used to interact with the Chroma container.
-
-<!--codeinclude-->
-[GetClient](../../modules/chroma/chroma_test.go) inside_block:getClient
-<!--/codeinclude-->
-
 ## Examples
 
 ### Getting a Chroma client

modules/chroma/chroma.go

@@ -4,8 +4,6 @@ import (
 	"context"
 	"fmt"
 
-	chromago "github.com/amikos-tech/chroma-go"
-
 	"github.com/testcontainers/testcontainers-go"
 	"github.com/testcontainers/testcontainers-go/wait"
 )
@@ -60,11 +58,3 @@ func (c *ChromaContainer) RESTEndpoint(ctx context.Context) (string, error) {
 
 	return fmt.Sprintf("http://%s:%s", host, containerPort.Port()), nil
 }
-
-func (c *ChromaContainer) GetClient(opt ...chromago.ClientOption) (*chromago.Client, error) {
-	endpoint, err := c.RESTEndpoint(context.Background())
-	if err != nil {
-		return nil, err
-	}
-	return chromago.NewClient(endpoint, opt...)
-}

modules/chroma/chroma_test.go

@@ -49,10 +49,14 @@ func TestChroma(t *testing.T) {
 
 	t.Run("GetClient", func(tt *testing.T) {
 		// restEndpoint {
-		chromaClient, err := container.GetClient(chromago.WithDebug(true))
+		endpoint, err := container.RESTEndpoint(context.Background())
+		if err != nil {
+			tt.Fatalf("failed to get REST endpoint: %s", err) // nolint:gocritic
+		}
+		chromaClient, err := chromago.NewClient(endpoint)
 		// }
 		if err != nil {
-			tt.Fatalf("failed to get REST endpoint: %s", err)
+			tt.Fatalf("failed to create client: %s", err)
 		}
 
 		hb, err := chromaClient.Heartbeat(context.TODO())

modules/chroma/examples_test.go

@@ -56,7 +56,11 @@ func ExampleChromaContainer_connectWithClient() {
 		}
 	}()
 
-	chromaClient, err := chromaContainer.GetClient()
+	endpoint, err := chromaContainer.RESTEndpoint(context.Background())
+	if err != nil {
+		log.Fatalf("failed to get REST endpoint: %s", err) // nolint:gocritic
+	}
+	chromaClient, err := chromago.NewClient(endpoint)
 	if err != nil {
 		log.Fatalf("failed to get client: %s", err) // nolint:gocritic
 	}
@@ -90,7 +94,11 @@ func ExampleChromaContainer_collections() {
 
 	// getClient {
 	// create the client connection and confirm that we can access the server with it
-	chromaClient, err := chromaContainer.GetClient(chromago.WithDebug(true))
+	endpoint, err := chromaContainer.RESTEndpoint(context.Background())
+	if err != nil {
+		log.Fatalf("failed to get REST endpoint: %s", err) // nolint:gocritic
+	}
+	chromaClient, err := chromago.NewClient(endpoint)
 	// }
 	if err != nil {
 		log.Fatalf("failed to get client: %s", err) // nolint:gocritic