Merge pull request #767 from python-gitlab/fix/744/delete_artifacts

max-wittig · web-flow · commit 4e1dd27d6400 · 2019-06-17T17:56:38.000+02:00
feature: Implement artifacts deletion
diff --git a/docs/gl_objects/builds.rst b/docs/gl_objects/builds.rst
@@ -319,6 +319,10 @@ Mark a job artifact as kept when expiration is set::
 
     build_or_job.keep_artifacts()
 
+Delete the artifacts of a job::
+
+    build_or_job.delete_artifacts()
+
 Get a job trace::
 
     build_or_job.trace()
diff --git a/gitlab/v4/objects.py b/gitlab/v4/objects.py
@@ -1584,6 +1584,21 @@ def keep_artifacts(self, **kwargs):
         path = "%s/%s/artifacts/keep" % (self.manager.path, self.get_id())
         self.manager.gitlab.http_post(path)
 
+    @cli.register_custom_action("ProjectJob")
+    @exc.on_http_error(exc.GitlabCreateError)
+    def delete_artifacts(self, **kwargs):
+        """Delete artifacts of a job.
+
+        Args:
+            **kwargs: Extra options to send to the server (e.g. sudo)
+
+        Raises:
+            GitlabAuthenticationError: If authentication is not correct
+            GitlabDeleteError: If the request could not be performed
+        """
+        path = "%s/%s/artifacts" % (self.manager.path, self.get_id())
+        self.manager.gitlab.http_delete(path)
+
     @cli.register_custom_action("ProjectJob")
     @exc.on_http_error(exc.GitlabGetError)
     def artifacts(self, streamed=False, action=None, chunk_size=1024, **kwargs):
diff --git a/tools/python_test_v4.py b/tools/python_test_v4.py
@@ -421,8 +421,9 @@
 
 # commit status
 commit = admin_project.commits.list()[0]
+size = len(commit.statuses.list())
 status = commit.statuses.create({"state": "success", "sha": commit.id})
-assert len(commit.statuses.list()) == 1
+assert len(commit.statuses.list()) == size + 1
 
 assert commit.refs()
 assert commit.merge_requests() is not None
