Correct integration test

Author: BryanFauble

tests/integration/synapseclient/models/async/test_permissions_async.py

@@ -736,17 +736,21 @@ async def test_delete_permissions_recursive(self, project_model: Project) -> Non
         await self._set_custom_permissions(file_3)
 
         # WHEN I delete permissions recursively but without container content
-        await top_level_folder.delete_permissions_async(
-            recursive=True, include_container_content=False
-        )
+        with pytest.raises(
+            ValueError,
+            match="When recursive=True, include_container_content must also be True",
+        ) as exc_info:
+            await top_level_folder.delete_permissions_async(
+                recursive=True, include_container_content=False
+            )
 
         # THEN the top_level_folder permissions should be deleted
         await self._verify_permissions_deleted(top_level_folder)
 
         # AND the folder_1 permissions should remain (because include_container_content=False)
         await self._verify_permissions_not_deleted(folder_1)
 
-        # BUT the file_3 permissions should remain (because include_container_content=False)
+        # AND the file_3 permissions should remain (because include_container_content=False)
         assert await self._verify_permissions_not_deleted(file_3)
 
     async def test_delete_permissions_recursive_with_container_content(

tests/integration/synapseclient/models/synchronous/test_permissions.py

@@ -727,18 +727,22 @@ async def test_delete_permissions_recursive(self, project_model: Project) -> Non
         await self._set_custom_permissions(folder_1)
         await self._set_custom_permissions(file_3)
 
-        # WHEN I delete permissions recursively but without container content
-        top_level_folder.delete_permissions(
-            recursive=True, include_container_content=False
-        )
+        with pytest.raises(
+            ValueError,
+            match="When recursive=True, include_container_content must also be True",
+        ) as exc_info:
+            # WHEN I delete permissions recursively but without container content
+            top_level_folder.delete_permissions(
+                recursive=True, include_container_content=False
+            )
 
         # THEN the top_level_folder permissions should be deleted
         await self._verify_permissions_deleted(top_level_folder)
 
         # AND the folder_1 permissions should remain (because include_container_content=False)
         await self._verify_permissions_not_deleted(folder_1)
 
-        # BUT the file_3 permissions should remain (because include_container_content=False)
+        # AND the file_3 permissions should remain (because include_container_content=False)
         assert await self._verify_permissions_not_deleted(file_3)
 
     async def test_delete_permissions_recursive_with_container_content(