fix: remove custom delete method for labels

JohnVillalovos · JohnVillalovos · commit 3870af8ec4ce · 2022-02-01T08:51:05.000-08:00
The usage of deleting was incorrect according to the current API. Add tests to show it works with labels needing to be encoded. Also enable the test_group_labels() test function. Previously it was disabled. Closes: #1867
diff --git a/gitlab/v4/objects/labels.py b/gitlab/v4/objects/labels.py
@@ -1,4 +1,4 @@
-from typing import Any, cast, Dict, Optional, TYPE_CHECKING, Union
+from typing import Any, cast, Dict, Optional, Union
 
 from gitlab import exceptions as exc
 from gitlab.base import RequiredOptional, RESTManager, RESTObject
@@ -78,25 +78,6 @@ def update(  # type: ignore
             new_data["name"] = name
         return super().update(id=None, new_data=new_data, **kwargs)
 
-    # Delete without ID.
-    @exc.on_http_error(exc.GitlabDeleteError)
-    # NOTE(jlvillal): Signature doesn't match DeleteMixin.delete() so ignore
-    # type error
-    def delete(self, name: str, **kwargs: Any) -> None:  # type: ignore
-        """Delete a Label on the server.
-
-        Args:
-            name: The name of the label
-            **kwargs: Extra options to send to the server (e.g. sudo)
-
-        Raises:
-            GitlabAuthenticationError: If authentication is not correct
-            GitlabDeleteError: If the server cannot perform the request
-        """
-        if TYPE_CHECKING:
-            assert self.path is not None
-        self.gitlab.http_delete(self.path, query_data={"name": name}, **kwargs)
-
 
 class ProjectLabel(
     PromoteMixin, SubscribableMixin, SaveMixin, ObjectDeleteMixin, RESTObject
@@ -162,22 +143,3 @@ def update(  # type: ignore
         if name:
             new_data["name"] = name
         return super().update(id=None, new_data=new_data, **kwargs)
-
-    # Delete without ID.
-    @exc.on_http_error(exc.GitlabDeleteError)
-    # NOTE(jlvillal): Signature doesn't match DeleteMixin.delete() so ignore
-    # type error
-    def delete(self, name: str, **kwargs: Any) -> None:  # type: ignore
-        """Delete a Label on the server.
-
-        Args:
-            name: The name of the label
-            **kwargs: Extra options to send to the server (e.g. sudo)
-
-        Raises:
-            GitlabAuthenticationError: If authentication is not correct
-            GitlabDeleteError: If the server cannot perform the request
-        """
-        if TYPE_CHECKING:
-            assert self.path is not None
-        self.gitlab.http_delete(self.path, query_data={"name": name}, **kwargs)
diff --git a/tests/functional/api/test_groups.py b/tests/functional/api/test_groups.py
@@ -104,7 +104,6 @@ def test_groups(gl):
     group2.members.delete(gl.user.id)
 
 
-@pytest.mark.skip(reason="Commented out in legacy test")
 def test_group_labels(group):
     group.labels.create({"name": "foo", "description": "bar", "color": "#112233"})
     label = group.labels.get("foo")
@@ -116,6 +115,12 @@ def test_group_labels(group):
     assert label.description == "baz"
     assert len(group.labels.list()) == 1
 
+    label.new_name = "comp:Flat Export"
+    label.save()
+    assert label.name == "comp:Flat Export"
+    label = group.labels.get("comp:Flat Export")
+    assert label.name == "comp:Flat Export"
+
     label.delete()
     assert len(group.labels.list()) == 0
 
diff --git a/tests/functional/api/test_projects.py b/tests/functional/api/test_projects.py
@@ -146,9 +146,11 @@ def test_project_labels(project):
     label = project.labels.get("label")
     assert label == labels[0]
 
-    label.new_name = "labelupdated"
+    label.new_name = "comp:Flat Export"
     label.save()
-    assert label.name == "labelupdated"
+    assert label.name == "comp:Flat Export"
+    label = project.labels.get("comp:Flat Export")
+    assert label.name == "comp:Flat Export"
 
     label.subscribe()
     assert label.subscribed is True
