From e17f9e9be0c2419a82ac46d2149965970b2f6f91 Mon Sep 17 00:00:00 2001
From: Sigurd Spieckermann <sigurd.spieckermann@gmail.com>
Date: Thu, 20 Nov 2025 12:58:50 +0100
Subject: [PATCH] feat(registry-protection): add support for registry
 protection rule deletion

---
 .../registry_protection_repository_rules.py       | 10 +++++++++-
 tests/functional/api/test_registry.py             |  5 +++++
 .../objects/test_registry_protection_rules.py     | 15 +++++++++++++++
 3 files changed, 29 insertions(+), 1 deletion(-)

diff --git a/gitlab/v4/objects/registry_protection_repository_rules.py b/gitlab/v4/objects/registry_protection_repository_rules.py
index 19d4bdf59..2a457a024 100644
--- a/gitlab/v4/objects/registry_protection_repository_rules.py
+++ b/gitlab/v4/objects/registry_protection_repository_rules.py
@@ -1,5 +1,12 @@
 from gitlab.base import RESTObject
-from gitlab.mixins import CreateMixin, ListMixin, SaveMixin, UpdateMethod, UpdateMixin
+from gitlab.mixins import (
+    CreateMixin,
+    DeleteMixin,
+    ListMixin,
+    SaveMixin,
+    UpdateMethod,
+    UpdateMixin,
+)
 from gitlab.types import RequiredOptional
 
 __all__ = [
@@ -16,6 +23,7 @@ class ProjectRegistryRepositoryProtectionRuleManager(
     ListMixin[ProjectRegistryRepositoryProtectionRule],
     CreateMixin[ProjectRegistryRepositoryProtectionRule],
     UpdateMixin[ProjectRegistryRepositoryProtectionRule],
+    DeleteMixin[ProjectRegistryRepositoryProtectionRule],
 ):
     _path = "/projects/{project_id}/registry/protection/repository/rules"
     _obj_cls = ProjectRegistryRepositoryProtectionRule
diff --git a/tests/functional/api/test_registry.py b/tests/functional/api/test_registry.py
index 91fdceacc..d234128ca 100644
--- a/tests/functional/api/test_registry.py
+++ b/tests/functional/api/test_registry.py
@@ -26,3 +26,8 @@ def test_project_protected_registry(project: Project):
     protected_registry.minimum_access_level_for_push = "owner"
     protected_registry.save()
     assert protected_registry.minimum_access_level_for_push == "owner"
+
+    protected_registry.delete()
+
+    rules = project.registry_protection_repository_rules.list()
+    assert rules == []
diff --git a/tests/unit/objects/test_registry_protection_rules.py b/tests/unit/objects/test_registry_protection_rules.py
index 3078278f5..3e9db414a 100644
--- a/tests/unit/objects/test_registry_protection_rules.py
+++ b/tests/unit/objects/test_registry_protection_rules.py
@@ -58,6 +58,17 @@ def resp_update_protected_registry():
         yield rsps
 
 
+@pytest.fixture
+def resp_delete_protected_registry():
+    with responses.RequestsMock() as rsps:
+        rsps.add(
+            method=responses.DELETE,
+            url="http://localhost/api/v4/projects/1/registry/protection/repository/rules/1",
+            status=204,
+        )
+        yield rsps
+
+
 def test_list_project_protected_registries(project, resp_list_protected_registries):
     protected_registry = project.registry_protection_repository_rules.list()[0]
     assert isinstance(protected_registry, ProjectRegistryRepositoryProtectionRule)
@@ -80,3 +91,7 @@ def test_update_project_protected_registry(project, resp_update_protected_regist
         1, {"repository_path_pattern": "abc*"}
     )
     assert updated["repository_path_pattern"] == "abc*"
+
+
+def test_delete_project_protected_registry(project, resp_delete_protected_registry):
+    project.registry_protection_repository_rules.delete(1)