fix s3 acl test and logging in assert_s3_acl (#598)

nspcc-dev · Sep 4, 2023 · 1b8a0b3 · 1b8a0b3
2 parents e95c173 + 6852478
commit 1b8a0b3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/pytest_tests/helpers/s3_helper.py b/pytest_tests/helpers/s3_helper.py
@@ -157,4 +157,4 @@ def assert_s3_acl(acl_grants: list, permitted_users: str):
                 permission = acl_grant.get("Permission")
                 assert permission == "FULL_CONTROL", "Only CanonicalUser should have FULL_CONTROL"
             else:
-                logger.error("FULL_CONTROL is given to All Users")
+                raise AssertionError("FULL_CONTROL is given to All Users")
diff --git a/pytest_tests/testsuites/services/s3_gate/test_s3_object.py b/pytest_tests/testsuites/services/s3_gate/test_s3_object.py
@@ -133,7 +133,7 @@ def test_s3_copy_acl(self, bucket, simple_object_size):
 
         with allure.step("Copy object and check acl attribute"):
             copy_obj_path = s3_gate_object.copy_object_s3(
-                self.s3_client, bucket, obj_key, ACL="public-read-write"
+                self.s3_client, bucket, obj_key, ACL="private"
             )
             obj_acl = s3_gate_object.get_object_acl_s3(self.s3_client, bucket, copy_obj_path)
             assert_s3_acl(acl_grants=obj_acl, permitted_users="CanonicalUser")