added new logicaly case in copy_items

Author: Narek Mkhitaryan

src/superannotate/lib/app/interface/sdk_interface.py

@@ -2927,7 +2927,7 @@ def copy_items(
         source: Union[NotEmptyStr, dict],
         destination: Union[NotEmptyStr, dict],
         items: Optional[List[NotEmptyStr]] = None,
-        include_annotations: Optional[bool] = True,
+        include_annotations: bool = True,
         duplicate_strategy: Literal[
             "skip", "replace", "replace_annotations_only"
         ] = "skip",
@@ -2961,6 +2961,12 @@ def copy_items(
         :rtype: list of strs
         """
 
+        if not include_annotations and duplicate_strategy != "skip":
+            duplicate_strategy = "skip"
+            logger.warning(
+                "Copy operation continuing without annotations and metadata due to include_annotations=False."
+            )
+
         project_name, source_folder = extract_project_folder(source)
         to_project_name, destination_folder = extract_project_folder(destination)
         if project_name != to_project_name:

src/superannotate/lib/infrastructure/controller.py

@@ -599,7 +599,7 @@ def move_multiple(
                 duplicate_strategy=duplicate_strategy,
                 include_annotations=True,
                 operation="move",
-                chunk_size=1000,
+                chunk_size=500,
             )
         return use_case.execute()
 

tests/integration/items/test_copy_items.py

@@ -18,6 +18,16 @@ class TestCopyItems(BaseTestCase):
     FOLDER_1 = "folder_1"
     FOLDER_2 = "folder_2"
     CSV_PATH = "data_set/attach_urls.csv"
+    Attachment = [
+        {
+            "url": "https://drive.google.com/uc?export=download&id=1vwfCpTzcjxoEA4hhDxqapPOVvLVeS7ZS",
+            "name": IMAGE_NAME,
+        },
+        {
+            "url": "https://drive.google.com/uc?export=download&id=1vwfCpTzcjxoEA4hhDxqapPOVvLVeS7Zw",
+            "name": IMAGE_NAME_2,
+        },
+    ]
 
     @property
     def scv_path(self):
@@ -69,12 +79,7 @@ def test_skipped_count(self):
     def test_copy_items_wrong_items_list(self):
         uploaded, _, _ = sa.attach_items(
             self.PROJECT_NAME,
-            [
-                {
-                    "url": "https://drive.google.com/uc?export=download&id=1vwfCpTzcjxoEA4hhDxqapPOVvLVeS7ZS",
-                    "name": self.IMAGE_NAME,
-                }
-            ],
+            self.Attachment,
         )
         sa.set_approval_statuses(self.PROJECT_NAME, "Approved", items=[self.IMAGE_NAME])
         sa.set_annotation_statuses(
@@ -92,3 +97,47 @@ def test_copy_items_wrong_items_list(self):
         assert items[0]["annotation_status"] == "Completed"
         assert items[0]["approval_status"] == "Approved"
         assert Counter(skipped_items) == Counter(["as", "asd"])
+
+    def test_copy_duplicated_items_without_data_with_replace_strategy(self):
+        sa.create_folder(self.PROJECT_NAME, self.FOLDER_1)
+        sa.create_folder(self.PROJECT_NAME, self.FOLDER_2)
+        uploaded, _, _ = sa.attach_items(
+            f"{self.PROJECT_NAME}/{self.FOLDER_1}", self.Attachment
+        )
+        assert len(uploaded) == 2
+        sa.set_approval_statuses(
+            f"{self.PROJECT_NAME}/{self.FOLDER_1}",
+            "Approved",
+            items=[self.IMAGE_NAME, self.IMAGE_NAME_2],
+        )
+        sa.set_annotation_statuses(
+            f"{self.PROJECT_NAME}/{self.FOLDER_1}",
+            "Completed",
+            items=[self.IMAGE_NAME, self.IMAGE_NAME_2],
+        )
+
+        uploaded_2, _, _ = sa.attach_items(
+            f"{self.PROJECT_NAME}/{self.FOLDER_2}", self.Attachment
+        )
+        assert len(uploaded_2) == 2
+
+        with self.assertLogs("sa", level="WARNING") as cm:
+            skipped_items = sa.copy_items(
+                f"{self.PROJECT_NAME}/{self.FOLDER_1}",
+                f"{self.PROJECT_NAME}/{self.FOLDER_2}",
+                include_annotations=False,
+                duplicate_strategy="replace",
+            )
+            assert (
+                "WARNING:sa:Copy operation continuing without annotations and metadata"
+                " due to include_annotations=False." == cm.output[0]
+            )
+        assert len(skipped_items) == 2
+        folder_1_items = sa.search_items(f"{self.PROJECT_NAME}/{self.FOLDER_1}")
+        folder_2_items = sa.search_items(f"{self.PROJECT_NAME}/{self.FOLDER_2}")
+        assert len(folder_1_items) == 2
+        assert len(folder_2_items) == 2
+
+        folder_2_items = sa.search_items(f"{self.PROJECT_NAME}/{self.FOLDER_2}")
+        assert folder_2_items[0]["annotation_status"] == "NotStarted"
+        assert not folder_2_items[0]["approval_status"]