Added comment pin/unpin

Signed-off-by: Graham Weldon <graham.weldon@rakuten.com>

Author: Graham Weldon

atlassian/jira.py

@@ -1971,6 +1971,28 @@ def issue_edit_comment(
         params: dict = {"notifyUsers": "true" if notify_users else "false"}
         return self.put(url, data=data, params=params)
 
+    def issue_pin_comment(self, issue_key: str, comment_id: T_id) -> T_resp_json:
+        """
+        Pin a comment on a Jira issue
+        :param issue_key: str
+        :param comment_id: int or str
+        :return:
+        """
+        base_url = self.resource_url("issue")
+        url = f"{base_url}/{issue_key}/comment/{comment_id}/pin"
+        return self.put(url, data=True)
+
+    def issue_unpin_comment(self, issue_key: str, comment_id: T_id) -> T_resp_json:
+        """
+        Unpin a comment on a Jira issue
+        :param issue_key: str
+        :param comment_id: int or str
+        :return:
+        """
+        base_url = self.resource_url("issue")
+        url = f"{base_url}/{issue_key}/comment/{comment_id}/pin"
+        return self.put(url, data=False)
+
     def scrap_regex_from_issue(self, issue: str, regex: str):
         """
         This function scrapes the output of the given regex matches from the issue's description and comments.

atlassian/rest_client.py

@@ -470,7 +470,7 @@ def request(
             url += ("&" if params or params_already_in_url else "") + "&".join(flags or [])
         json_dump = None
         if files is None:
-            data = None if not data else dumps(data)
+            data = None if data is None else dumps(data)
             json_dump = None if not json else dumps(json)
 
         headers = headers or self.default_headers

tests/responses/jira/rest/api/2/issue/FOO-123/comment/10000/pin/PUT

@@ -0,0 +1,2 @@
+responses['true'] = {}
+responses['false'] = {}

tests/test_jira.py

@@ -46,6 +46,14 @@ def test_get_issue_comment_not_found(self):
         with self.assertRaises(HTTPError):
             self.jira.epic_issues("BAR-11")
 
+    def test_pin_issue_comment(self):
+        """Can pin a comment on an issue"""
+        self.jira.issue_pin_comment("FOO-123", 10000)
+
+    def test_unpin_issue_comment(self):
+        """Can unpin a comment on an issue"""
+        self.jira.issue_unpin_comment("FOO-123", 10000)
+
     def test_post_issue_with_invalid_request(self):
         """Post an issue but receive a 400 error response"""
         with self.assertRaises(HTTPError):