chore: review suggestions

qdrant/collections.go

@@ -63,15 +63,12 @@ func (c *Client) DeleteCollection(ctx context.Context, collectionName string) er
 	res, err := c.GetCollectionsClient().Delete(ctx, &DeleteCollection{
 		CollectionName: collectionName,
 	})
-
 	if err != nil {
 		return newQdrantErr(err, "DeleteCollection", collectionName)
 	}
-
 	if !res.GetResult() {
 		return newQdrantErr(errors.New("failed to delete collection"), "DeleteCollection", collectionName)
 	}
-
 	return nil
 }
 

qdrant/error.go

@@ -5,7 +5,7 @@ import (
 	"strings"
 )
 
-//nolint:revive // The linter says qdrant.QdrantErr stutters, but it's an apt name.
+//nolint:revive // The linter says qdrant.QdrantError stutters, but it's an apt name.
 type QdrantError struct {
 	operationName string
 	context       string

qdrant_test/collections_test.go

@@ -15,7 +15,7 @@ func TestCollectionsClient(t *testing.T) {
 	apiKey := "<COLLECTIONS_TEST>"
 
 	ctx, cancel := context.WithCancel(context.Background())
-	t.Cleanup(cancel)
+	defer cancel()
 
 	container, err := distributedQdrant(ctx, apiKey)
 	require.NoError(t, err)

qdrant_test/points_test.go

@@ -15,7 +15,7 @@ func TestPointsClient(t *testing.T) {
 	apiKey := "<POINTS_TEST>"
 
 	ctx, cancel := context.WithCancel(context.Background())
-	t.Cleanup(cancel)
+	defer cancel()
 
 	container, err := standaloneQdrant(ctx, apiKey)
 	require.NoError(t, err)

qdrant_test/qdrant_test.go

@@ -12,7 +12,7 @@ func TestHealthCheck(t *testing.T) {
 	apiKey := "<HEALTHCHECK_TEST>"
 
 	ctx, cancel := context.WithCancel(context.Background())
-	t.Cleanup(cancel)
+	defer cancel()
 
 	container, err := standaloneQdrant(ctx, apiKey)
 	require.NoError(t, err)

qdrant_test/snapshots_test.go

@@ -13,7 +13,7 @@ func TestSnapshotsClient(t *testing.T) {
 	apiKey := "<SNAPSHOTS_TEST>"
 
 	ctx, cancel := context.WithCancel(context.Background())
-	t.Cleanup(cancel)
+	defer cancel()
 
 	container, err := standaloneQdrant(ctx, apiKey)
 	require.NoError(t, err)