clean up comments

cloudflare · Aug 9, 2022 · 0fd9c64 · 0fd9c64
1 parent bf27665
commit 0fd9c64
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/r2_bucket.go b/r2_bucket.go
@@ -18,11 +18,14 @@ func (api *API) CreateR2Bucket(ctx context.Context, rc *ResourceContainer, bucke
 	if rc.Identifier == "" {
 		return ErrMissingAccountID
 	}
+
 	if bucketName == "" {
 		return ErrMissingBucketName
 	}
+
 	uri := fmt.Sprintf("/accounts/%s/r2/buckets/%s", rc.Identifier, bucketName)
 	_, err := api.makeRequestContext(ctx, http.MethodPost, uri, nil)
+
 	return err
 }
 
@@ -33,10 +36,13 @@ func (api *API) DeleteR2Bucket(ctx context.Context, rc *ResourceContainer, bucke
 	if rc.Identifier == "" {
 		return ErrMissingAccountID
 	}
+
 	if bucketName == "" {
 		return ErrMissingBucketName
 	}
+
 	uri := fmt.Sprintf("/accounts/%s/r2/buckets/%s", rc.Identifier, bucketName)
 	_, err := api.makeRequestContext(ctx, http.MethodDelete, uri, nil)
+
 	return err
 }
diff --git a/r2_bucket_test.go b/r2_bucket_test.go
@@ -3,9 +3,10 @@ package cloudflare
 import (
 	"context"
 	"fmt"
-	"github.com/stretchr/testify/assert"
 	"net/http"
 	"testing"
+
+	"github.com/stretchr/testify/assert"
 )
 
 const testBucketName = "example-bucket"
@@ -22,16 +23,14 @@ func TestR2_CreateBucket(t *testing.T) {
   "errors": [],
   "messages": [],
   "result": {}
-}`) //nolint
+}`)
 	})
 
-	// Make sure missing account ID is thrown
 	err := client.CreateR2Bucket(context.Background(), AccountIdentifier(""), "")
 	if assert.Error(t, err) {
 		assert.Equal(t, ErrMissingAccountID, err)
 	}
 
-	// Make sure missing bucket name is thrown
 	err = client.CreateR2Bucket(context.Background(), AccountIdentifier(testAccountID), "")
 	if assert.Error(t, err) {
 		assert.Equal(t, ErrMissingBucketName, err)
@@ -53,16 +52,14 @@ func TestR2_DeleteBucket(t *testing.T) {
   "errors": [],
   "messages": [],
   "result": {}
-}`) //nolint
+}`)
 	})
 
-	// Make sure missing account ID is thrown
 	err := client.DeleteR2Bucket(context.Background(), AccountIdentifier(""), "")
 	if assert.Error(t, err) {
 		assert.Equal(t, ErrMissingAccountID, err)
 	}
 
-	// Make sure missing bucket name is thrown
 	err = client.DeleteR2Bucket(context.Background(), AccountIdentifier(testAccountID), "")
 	if assert.Error(t, err) {
 		assert.Equal(t, ErrMissingBucketName, err)