Move code into audit_events, add tests

Clayton Walker · Clayton Walker · commit 1f5c5bf157d6 · 2021-02-19T13:43:15.000-07:00
diff --git a/gitlab/tests/objects/test_audit_events.py b/gitlab/tests/objects/test_audit_events.py
@@ -0,0 +1,54 @@
+"""
+GitLab API:
+https://docs.gitlab.com/ee/api/audit_events.html#project-audit-events
+"""
+
+import re
+
+import pytest
+import responses
+
+from gitlab.v4.objects.audit_events import ProjectAudit
+
+audit_events_url = re.compile(
+    r"http://localhost/api/v4/(((groups|projects)/1)|(admin/ci))/audit_events"
+)
+
+audit_events_content = {
+    "id": 5,
+    "author_id": 1,
+    "entity_id": 7,
+    "entity_type": "Project",
+    "details": {
+        "change": "prevent merge request approval from reviewers",
+        "from": "",
+        "to": "true",
+        "author_name": "Administrator",
+        "target_id": 7,
+        "target_type": "Project",
+        "target_details": "twitter/typeahead-js",
+        "ip_address": "127.0.0.1",
+        "entity_path": "twitter/typeahead-js"
+    },
+    "created_at": "2020-05-26T22:55:04.230Z"
+}
+
+
+@pytest.fixture
+def resp_list_audit_events():
+    with responses.RequestsMock() as rsps:
+        rsps.add(
+            method=responses.GET,
+            url=audit_events_url,
+            json=[audit_events_content],
+            content_type="application/json",
+            status=200,
+        )
+        yield rsps
+
+
+def test_list_project_audit_events(project, resp_list_audit_events):
+    audit_events = project.audit_events.list()
+    assert isinstance(audit_events, list)
+    assert isinstance(audit_events[0], ProjectAudit)
+    assert audit_events[0].id == 5
diff --git a/gitlab/v4/objects/audit_events.py b/gitlab/v4/objects/audit_events.py
@@ -0,0 +1,23 @@
+"""
+GitLab API:
+https://docs.gitlab.com/ee/api/audit_events.html#project-audit-events
+"""
+
+from gitlab.base import *  # noqa
+from gitlab.mixins import *  # noqa
+
+__all__ = [
+    "ProjectAudit",
+    "ProjectAuditManager",
+]
+
+
+class ProjectAudit(RESTObject):
+    _id_attr = "id"
+
+
+class ProjectAuditManager(RetrieveMixin, RESTManager):
+    _path = "/projects/%(project_id)s/audit_events"
+    _obj_cls = ProjectAudit
+    _from_parent_attrs = {"project_id": "id"}
+    _list_filters = ("created_after", "created_before")
diff --git a/gitlab/v4/objects/projects.py b/gitlab/v4/objects/projects.py
@@ -16,6 +16,7 @@
 from .deployments import ProjectDeploymentManager
 from .environments import ProjectEnvironmentManager
 from .events import ProjectEventManager
+from .audit_events import ProjectAuditManager
 from .export_import import ProjectExportManager, ProjectImportManager
 from .files import ProjectFileManager
 from .hooks import ProjectHookManager
@@ -51,8 +52,6 @@
     "GroupProjectManager",
     "Project",
     "ProjectManager",
-    "ProjectAudit",
-    "ProjectAuditManager",
     "ProjectFork",
     "ProjectForkManager",
     "ProjectRemoteMirror",
@@ -87,17 +86,6 @@ class GroupProjectManager(ListMixin, RESTManager):
     )
 
 
-class ProjectAudit(RESTObject):
-    _id_attr = "id"
-
-
-class ProjectAuditManager(RetrieveMixin, RESTManager):
-    _path = "/projects/%(project_id)s/audit_events"
-    _obj_cls = ProjectAudit
-    _from_parent_attrs = {"project_id": "id"}
-    _list_filters = ("created_after", "created_before")
-
-
 class Project(SaveMixin, ObjectDeleteMixin, RESTObject):
     _short_print_attr = "path"
     _managers = (
