Start of consolidating tests for exceptions

Author: adamtheturtle

tests/test_add_target.py

@@ -107,26 +107,6 @@ def test_given(
             application_metadata=b'a',
         )
 
-    def test_too_large(
-        self,
-        client: VWS,
-        high_quality_image: io.BytesIO,
-    ) -> None:
-        """
-        A ``MetadataTooLarge`` exception is raised if the metadata given is too
-        large.
-        """
-        with pytest.raises(MetadataTooLarge) as exc:
-            client.add_target(
-                name='x',
-                width=1,
-                image=high_quality_image,
-                application_metadata=b'a' * 1024 * 1024,
-            )
-
-        assert exc.value.response.status_code == codes.UNPROCESSABLE_ENTITY
-
-
 class TestActiveFlag:
     """
     Tests for the ``active_flag`` parameter to ``add_target``.

tests/test_delete_target.py

@@ -16,26 +16,6 @@ class TestDelete:
     Test for deleting a target.
     """
 
-    def test_target_processing(
-        self,
-        client: VWS,
-        high_quality_image: io.BytesIO,
-    ) -> None:
-        """
-        A ``TargetStatusProcessing`` exception is raised if trying to delete a
-        target which is processing.
-        """
-        target_id = client.add_target(
-            name='x',
-            width=1,
-            image=high_quality_image,
-        )
-
-        with pytest.raises(TargetStatusProcessing) as exc:
-            client.delete_target(target_id=target_id)
-
-        assert exc.value.response.status_code == codes.FORBIDDEN
-
     def test_delete_target(
         self,
         client: VWS,

tests/test_exceptions.py

@@ -16,6 +16,8 @@
     Fail,
     ImageTooLarge,
     TargetNameExist,
+    TargetStatusProcessing,
+    MetadataTooLarge,
     ProjectInactive,
     UnknownTarget,
 )
@@ -146,9 +148,50 @@ def test_project_inactive(client: VWS, high_quality_image: io.BytesIO) -> None:
             server_secret_key=mock.server_secret_key,
         )
 
-        with pytest.raises(ProjectInactive):
+        with pytest.raises(ProjectInactive) as exc:
             client.add_target(
                 name='x',
                 width=1,
                 image=high_quality_image,
             )
+
+    assert exc.value.response.status_code == codes.FORBIDDEN
+
+
+def test_target_status_processing(
+    client: VWS,
+    high_quality_image: io.BytesIO,
+) -> None:
+    """
+    A ``TargetStatusProcessing`` exception is raised if trying to delete a
+    target which is processing.
+    """
+    target_id = client.add_target(
+        name='x',
+        width=1,
+        image=high_quality_image,
+    )
+
+    with pytest.raises(TargetStatusProcessing) as exc:
+        client.delete_target(target_id=target_id)
+
+    assert exc.value.response.status_code == codes.FORBIDDEN
+
+
+def test_metadata_too_large(
+    client: VWS,
+    high_quality_image: io.BytesIO,
+) -> None:
+    """
+    A ``MetadataTooLarge`` exception is raised if the metadata given is too
+    large.
+    """
+    with pytest.raises(MetadataTooLarge) as exc:
+        client.add_target(
+            name='x',
+            width=1,
+            image=high_quality_image,
+            application_metadata=b'a' * 1024 * 1024,
+        )
+
+    assert exc.value.response.status_code == codes.UNPROCESSABLE_ENTITY