From ac7a7cec20644aae96255b39e66a34e3b6205186 Mon Sep 17 00:00:00 2001 From: Harshavardhana Date: Mon, 2 May 2016 16:58:10 -0700 Subject: [PATCH] bucket-policy: Delete policy should remove policy properly. (#1456) --- bucket-policy.go | 3 +++ server_test.go | 9 +++++++++ 2 files changed, 12 insertions(+) diff --git a/bucket-policy.go b/bucket-policy.go index d9186878b..6a7376e1d 100644 --- a/bucket-policy.go +++ b/bucket-policy.go @@ -101,6 +101,9 @@ func removeBucketPolicy(bucket string) error { } return err } + if err := os.Remove(bucketPolicyFile); err != nil { + return err + } return nil } diff --git a/server_test.go b/server_test.go index 587352a87..4f58bdbd0 100644 --- a/server_test.go +++ b/server_test.go @@ -350,6 +350,15 @@ func (s *MyAPISuite) TestBucketPolicy(c *C) { response, err = client.Do(request) c.Assert(err, IsNil) c.Assert(response.StatusCode, Equals, http.StatusNoContent) + + // Verify if the policy was indeed deleted. + request, err = s.newRequest("GET", testAPIFSCacheServer.URL+"/policybucket?policy", 0, nil) + c.Assert(err, IsNil) + + client = http.Client{} + response, err = client.Do(request) + c.Assert(err, IsNil) + c.Assert(response.StatusCode, Equals, http.StatusNotFound) } func (s *MyAPISuite) TestDeleteBucket(c *C) {