implemented CommentProperty

[zerwes]

handle comment properties analogue to the issue properties

example

is_internal_comment = False
try:
    p = jira.comment_property(425201, 'sd.public.comment')
    is_internal_comment = p.value.internal.lower() == 'true'
except JIRAError:
    pass

[adehad]

I think we can add a test similar to your example in the description in a file like
tests/resources/test_comment.py

[adehad]

Excellent PR, let's get the pipeline passing and a test in

[zerwes]

Hello @adehad

I think we can add a test similar to your example in the description in a file like tests/resources/test_comment.py

do you mean something like

diff --git a/tests/resources/test_comment.py b/tests/resources/test_comment.py
index 1dafc84..515c768 100644
--- a/tests/resources/test_comment.py
+++ b/tests/resources/test_comment.py
@@ -9,9 +9,10 @@ class CommentTests(JiraTestCase):
         self.issue_1_key = self.test_manager.project_b_issue1
         self.issue_2_key = self.test_manager.project_b_issue2
         self.issue_3_key = self.test_manager.project_b_issue3
+        self.issue_4_key = self.test_manager.project_b_issue4
 
     def tearDown(self) -> None:
-        for issue in [self.issue_1_key, self.issue_2_key, self.issue_3_key]:
+        for issue in [self.issue_1_key, self.issue_2_key, self.issue_3_key, self.issue_4_key]:
             for comment in self.jira.comments(issue):
                 comment.delete()
 
@@ -80,3 +81,10 @@ class CommentTests(JiraTestCase):
         comment.update(body="updated! without notification", notify=False)
         assert comment.body == "updated! without notification"
         comment.delete()
+
+    def test_comment_property(self):
+        comment = self.jira.add_comment(self.issue_4_key, "comment for property test")
+        comment_prop_data = {'internal': 'true'}
+        self.jira.add_comment_property(comment, 'sd.public.comment', comment_prop_data)
+        assert self.jira.comment_property(comment, 'sd.public.comment').value.internal.lower() == 'true'
+        comment.delete()

[adehad]

@zerwes Yes exactly something similar to this. I also suggest to add an additional assertion with an isinstance check

You don't need to create the issue 4 at the test setup level, reusing an existing issue should be fine.
Or also within the test function itself making a new comment is also perfect.

[zerwes]

here we go: 37d7247
I preferred to use a own fct. w/ issue and comment to keep things clearly separated ... if you think this is to heavy, let me know, I can integrate it in another comment test.

[adehad]

Excellent test, minor comment regarding the issue used.

We can ignore the docs build failures that's unrelated.