expose IsObjectAction helper (#57)

Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
minio · Feb 21, 2023 · fe221d2 · fe221d2
1 parent 797e92e
commit fe221d2
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/bucket/policy/action.go b/bucket/policy/action.go
@@ -209,8 +209,8 @@ var supportedObjectActions = map[Action]struct{}{
 	ResetBucketReplicationStateAction:    {},
 }
 
-// isObjectAction - returns whether action is object type or not.
-func (action Action) isObjectAction() bool {
+// IsObjectAction - returns whether action is object type or not.
+func (action Action) IsObjectAction() bool {
 	_, ok := supportedObjectActions[action]
 	return ok
 }

diff --git a/bucket/policy/action_test.go b/bucket/policy/action_test.go
@@ -37,7 +37,7 @@ func TestActionIsObjectAction(t *testing.T) {
 	}
 
 	for i, testCase := range testCases {
-		result := testCase.action.isObjectAction()
+		result := testCase.action.IsObjectAction()
 
 		if testCase.expectedResult != result {
 			t.Fatalf("case %v: expected: %v, got: %v", i+1, testCase.expectedResult, result)

diff --git a/bucket/policy/statement.go b/bucket/policy/statement.go
@@ -103,7 +103,7 @@ func (statement Statement) isValid() error {
 	}
 
 	for action := range statement.Actions {
-		if action.isObjectAction() {
+		if action.IsObjectAction() {
 			if !statement.Resources.objectResourceExists() {
 				return Errorf("unsupported Resource found %v for action %v", statement.Resources, action)
 			}