Create, update and delete hooks.

Author: Costin Bleotu

gogs_client/_implementation/http_utils.py

@@ -36,13 +36,13 @@ def options(self, relative_path, params=None, **kwargs):
         return self.session.options(self.absolute_url(relative_path), params=params, **kwargs)
 
     def patch(self, relative_path, data=None, **kwargs):
-        return self.session.patch(self.absolute_url(relative_path), data=data, **kwargs)
+        return self.session.patch(self.absolute_url(relative_path), json=data, **kwargs)
 
     def post(self, relative_path, data=None, **kwargs):
-        return self.session.post(self.absolute_url(relative_path), data=data, **kwargs)
+        return self.session.post(self.absolute_url(relative_path), json=data, **kwargs)
 
     def put(self, relative_path, params=None, data=None, **kwargs):
-        return self.session.put(self.absolute_url(relative_path), params=params, data=data, **kwargs)
+        return self.session.put(self.absolute_url(relative_path), params=params, json=data, **kwargs)
 
 
 def append_url(base_url, path):

gogs_client/entities.py

@@ -260,6 +260,16 @@ def from_json(parsed_json):
             return GogsRepo.Hook(hook_id=hook_id, hook_type=hook_type, events=events, active=active, 
                             config=config)
 
+        def as_dict(self):
+            fields = {
+                "id": self._id,
+                "type": self._type,
+                "events": self._events,
+                "config": self._config,
+                "active": self._active,
+            }
+            return {k: v for (k, v) in fields.items() if v is not None}
+
         @property  # named hook_id to avoid conflict with built-in id
         def hook_id(self):
             """

gogs_client/interface.py

@@ -334,10 +334,25 @@ def create_hook(self, auth, repo_name, hook_type, config, events=["push"], organ
 
         url = "/repos/{o}/{r}/hooks".format(o=organization, r=repo_name) if organization else "/repos/{r}/hooks".format(r=repo_name)
         response = self._post(url, auth=auth, data=data)
-        print response.text
         self._check_ok(response)
         return GogsRepo.Hook.from_json(response.json())
 
+    def update_hook(self, auth, repo_name, hook_id, update, organization=None):
+        """
+        Updates hook  with id ``hook_id`` according to ``update``.
+
+        :param auth.Authentication auth: authentication object, must be admin-level
+        :param str repo_name: repo of the hook to update
+        :param GogsHookUpdate update: a ``GogsHookUpdate`` object describing the requested update
+        :return: the updated hook
+        :rtype: GogsRepo.Hook
+        :raises NetworkFailure: if there is an error communicating with the server
+        :raises ApiFailure: if the request cannot be serviced
+        """
+        path = "/repos/{o}/{r}/hooks/{i}".format(o=organization, r=repo_name, i=hook_id) if organization else "/repos/{r}/hooks/{i}".format(r=repo_name, i=hook_id)
+        response = self._check_ok(self._patch(path, auth=auth, data=update.as_dict()))
+        return GogsRepo.Hook.from_json(response.json())
+
     def delete_hook(self, auth, username, repo_name, hook_id):
         """
         Deletes the hook with id ``hook_id`` for repo with name ``repo_name``

gogs_client/updates.py

@@ -190,22 +190,19 @@ class GogsHookUpdate(object):
 
     Instances should be created using the :class:`~GogsHookUpdate.Builder` class.
     """
-    def __init__(self, hook_type, events, config, active):
+    def __init__(self, events, config, active):
         """
-        :param hook_type:
         :param events:
         :param config:
         :param active:
         """
 
-        self._type = hook_type
         self._events = events
         self._config = config
         self._active = active
 
     def as_dict(self):
         fields = {
-            "type": self._type,
             "events": self._events,
             "config": self._config,
             "active": self._active,
@@ -218,12 +215,10 @@ def __init__(self):
             :param str login_name: login name for authentication source
             :param str email: email address of user to update
             """
-            self._type = None
             self._events = None
             self._config = None
             self._active = None
 
-
         def set_events(self, events):
             """
             :param list events:
@@ -257,7 +252,6 @@ def build(self):
             :rtype: GogsHookUpdate
             """
             return GogsHookUpdate(
-                hook_type=self._type,
                 events=self._events,
                 config=self._config,
                 active=self._active)