Fix missing name test

Author: hovnatan

superannotate/db/annotation_classes.py

@@ -350,7 +350,8 @@ def fill_class_and_attribute_ids(annotation_json, annotation_classes_dict):
 
 def check_annotation_json(annotation_json):
     if "metadata" not in annotation_json or "width" not in annotation_json[
-        "metadata"] or "height" not in annotation_json["metadata"]:
+        "metadata"] or "height" not in annotation_json[
+            "metadata"] or "name" not in annotation_json["metadata"]:
         return False
     else:
         return True

superannotate/db/projects.py

@@ -1103,7 +1103,6 @@ def __upload_annotations_thread(
     annotation_classes_dict, thread_id, chunksize, missing_images,
     couldnt_upload, uploaded, from_s3_bucket
 ):
-    print("DEBUG")
     NUM_TO_SEND = 500
     len_anns = len(anns_filenames)
     start_index = thread_id * chunksize
@@ -1120,7 +1119,6 @@ def __upload_annotations_thread(
 
     for i in range(start_index, end_index, NUM_TO_SEND):
         data = {"project_id": project_id, "team_id": team_id, "names": []}
-        print("DEBUG", flush=True)
         for j in range(i, i + NUM_TO_SEND):
             if j >= end_index:
                 break

tests/sample_project_vector_for_checks/example_image_2.jpg___objects.json

@@ -1,6 +1,5 @@
 {
   "metadata": {
-    "name": "example_image_2.jpg",
     "width": 1885,
     "height": 1060,
     "status": "Completed",

tests/test_missing_annotation_upload.py

@@ -2,6 +2,7 @@
 
 import superannotate as sa
 
+
 def test_missing_annotation_upload(tmpdir):
     name = "Example Project test vector missing annotation upload"
     project_type = "Vector"
@@ -23,22 +24,14 @@ def test_missing_annotation_upload(tmpdir):
         project, from_folder
     )
     print(uploaded, couldnt_upload, missing_images)
-    assert len(uploaded) == 2
-    assert len(couldnt_upload) == 1
+    assert len(uploaded) == 1
+    assert len(couldnt_upload) == 2
     assert len(missing_images) == 1
 
-    assert uploaded[
-        0
-    ] == "tests/sample_project_vector_for_checks/example_image_1.jpg___objects.json"
-    assert uploaded[
-        1
-    ] == "tests/sample_project_vector_for_checks/example_image_2.jpg___objects.json"
-    assert couldnt_upload[
-        0
-    ] == "tests/sample_project_vector_for_checks/example_image_4.jpg___objects.json"
-    assert missing_images[
-        0
-    ] == "tests/sample_project_vector_for_checks/example_image_5.jpg___objects.json"
+    assert "tests/sample_project_vector_for_checks/example_image_1.jpg___objects.json" in uploaded
+    assert "tests/sample_project_vector_for_checks/example_image_2.jpg___objects.json" in couldnt_upload
+    assert "tests/sample_project_vector_for_checks/example_image_4.jpg___objects.json" in couldnt_upload
+    assert "tests/sample_project_vector_for_checks/example_image_5.jpg___objects.json" in missing_images
 
 
 def test_missing_preannotation_upload(tmpdir):
@@ -47,7 +40,6 @@ def test_missing_preannotation_upload(tmpdir):
     description = "test vector"
     from_folder = Path("./tests/sample_project_vector_for_checks")
 
-
     projects = sa.search_projects(name, return_metadata=True)
     for project in projects:
         sa.delete_project(project)
@@ -64,18 +56,10 @@ def test_missing_preannotation_upload(tmpdir):
         project, from_folder
     )
     print(uploaded, couldnt_upload)
-    assert len(uploaded) == 3
-    assert len(couldnt_upload) == 1
+    assert len(uploaded) == 2
+    assert len(couldnt_upload) == 2
 
-    assert uploaded[
-        0
-    ] == "tests/sample_project_vector_for_checks/example_image_1.jpg___objects.json"
-    assert uploaded[
-        1
-    ] == "tests/sample_project_vector_for_checks/example_image_2.jpg___objects.json"
-    assert uploaded[
-        2
-    ] == "tests/sample_project_vector_for_checks/example_image_5.jpg___objects.json"
-    assert couldnt_upload[
-        0
-    ] == "tests/sample_project_vector_for_checks/example_image_4.jpg___objects.json"
+    assert "tests/sample_project_vector_for_checks/example_image_1.jpg___objects.json" in uploaded
+    assert "tests/sample_project_vector_for_checks/example_image_5.jpg___objects.json" in uploaded
+    assert "tests/sample_project_vector_for_checks/example_image_2.jpg___objects.json" in couldnt_upload
+    assert "tests/sample_project_vector_for_checks/example_image_4.jpg___objects.json" in couldnt_upload