From a232d73cb174ff2c2383617313f5acb0a64b6a67 Mon Sep 17 00:00:00 2001
From: Cesar Celis Hernandez <celis.hernandez.cesar@gmail.com>
Date: Sat, 12 Feb 2022 00:55:26 -0500
Subject: [PATCH] Add test for Bucket Set Policy end point (#1562)

---
 integration/buckets_test.go | 104 ++++++++++++++++++++++++++++++++++++
 1 file changed, 104 insertions(+)

diff --git a/integration/buckets_test.go b/integration/buckets_test.go
index 18a6ca525..a80a41132 100644
--- a/integration/buckets_test.go
+++ b/integration/buckets_test.go
@@ -628,6 +628,41 @@ func RestoreObjectToASelectedVersion(bucketName string, prefix string, versionID
 	return response, err
 }
 
+func BucketSetPolicy(bucketName string, access string, definition string) (*http.Response, error) {
+	/*
+		Helper function to set policy on a bucket
+		Name: Bucket Set Policy
+		HTTP Verb: PUT
+		URL: {{baseUrl}}/buckets/:name/set-policy
+		Body:
+		{
+			"access": "PRIVATE",
+			"definition": "dolo"
+		}
+	*/
+	requestDataAdd := map[string]interface{}{
+		"access":     access,
+		"definition": definition,
+	}
+	requestDataJSON, _ := json.Marshal(requestDataAdd)
+	requestDataBody := bytes.NewReader(requestDataJSON)
+	request, err := http.NewRequest(
+		"PUT",
+		"http://localhost:9090/api/v1/buckets/"+bucketName+"/set-policy",
+		requestDataBody,
+	)
+	if err != nil {
+		log.Println(err)
+	}
+	request.Header.Add("Cookie", fmt.Sprintf("token=%s", token))
+	request.Header.Add("Content-Type", "application/json")
+	client := &http.Client{
+		Timeout: 2 * time.Second,
+	}
+	response, err := client.Do(request)
+	return response, err
+}
+
 func TestAddBucket(t *testing.T) {
 	assert := assert.New(t)
 	type args struct {
@@ -2047,3 +2082,72 @@ func TestRestoreObjectToASelectedVersion(t *testing.T) {
 		})
 	}
 }
+
+func TestBucketSetPolicy(t *testing.T) {
+
+	// Variables
+	assert := assert.New(t)
+	validBucketName := "testbucketsetpolicy"
+
+	// 1. Create bucket
+	response, err := AddBucket(validBucketName, true, true, nil, nil)
+	assert.Nil(err)
+	if err != nil {
+		log.Println(err)
+		assert.Fail("Error creating the bucket")
+		return
+	}
+	if response != nil {
+		assert.Equal(201, response.StatusCode, inspectHTTPResponse(response))
+	}
+
+	// 2. Set a bucket's policy using table driven tests
+	type args struct {
+		bucketName string
+	}
+	tests := []struct {
+		name           string
+		expectedStatus int
+		args           args
+	}{
+		{
+			name:           "Valid bucket when setting a policy",
+			expectedStatus: 200,
+			args: args{
+				bucketName: validBucketName,
+			},
+		},
+		{
+			name:           "Invalid bucket when setting a bucket",
+			expectedStatus: 500,
+			args: args{
+				bucketName: "wlkjsdkalsjdklajsdlkajsdlkajsdlkajsdklajsdkljaslkdjaslkdj",
+			},
+		},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+
+			// Set Policy
+			restResp, restErr := BucketSetPolicy(
+				tt.args.bucketName,
+				"PUBLIC",
+				"",
+			)
+			assert.Nil(restErr)
+			if restErr != nil {
+				log.Println(restErr)
+				return
+			}
+			finalResponse := inspectHTTPResponse(restResp)
+			if restResp != nil {
+				assert.Equal(
+					tt.expectedStatus,
+					restResp.StatusCode,
+					finalResponse,
+				)
+			}
+
+		})
+	}
+}