Add more tests and start measuring coverage (#2486)

Author: iarspider

.github/workflows/test-process-pr.yaml

@@ -26,7 +26,9 @@ jobs:
           sparse-checkout: tests/Framework.py
           sparse-checkout-cone-mode: false
       - run: mv tmp/tests/Framework.py tests/
-      - run: sed -i -e 's/pool_size=self.pool_size/per_page=100/g' tests/Framework.py
+      - run: |
+          cd tests
+          patch -p0 < Framework.patch
       - name: Set up Python ${{ matrix.python-version }}
         uses: actions/setup-python@v4
         with:

.gitignore

@@ -2,5 +2,10 @@
 .idea
 *.properties
 tests/Framework.py
-venv
+venv*
 GithubCredentials.py
+.coverage
+htmlcov
+/tests/ReplayData/TestProcessPr.test_mark_rejected.txt
+/tests/ReplayData/TestProcessPr.test_mark_passed.txt
+/tests/*.log

modify_comment.py

@@ -7,10 +7,39 @@
 from optparse import OptionParser
 import sys
 from socket import setdefaulttimeout
+import re
 
 setdefaulttimeout(120)
 SCRIPT_DIR = dirname(abspath(sys.argv[0]))
 
+
+def find_last_comment(issue, user, match):
+    last_comment = None
+    for comment in issue.get_comments():
+        if (user != comment.user.login) or (not comment.body):
+            continue
+        if not re.match(
+            match, comment.body.encode("ascii", "ignore").decode().strip("\n\t\r "), re.MULTILINE
+        ):
+            continue
+        last_comment = comment
+        print("Matched comment from %s with comment id %s" % (comment.user.login, comment.id))
+    return last_comment
+
+
+def modify_comment(comment, match, replace, dryRun):
+    comment_msg = comment.body.encode("ascii", "ignore").decode() if comment.body else ""
+    if match:
+        new_comment_msg = re.sub(match, replace, comment_msg)
+    else:
+        new_comment_msg = comment_msg + "\n" + replace
+    if new_comment_msg != comment_msg:
+        if not dryRun:
+            comment.edit(new_comment_msg)
+            print("Message updated")
+    return 0
+
+
 valid_types = {}
 valid_types["JENKINS_TEST_URL"] = ["", None]
 valid_types["JENKINS_STYLE_URL"] = ["", None]
@@ -68,7 +97,6 @@
     if exists(join(repo_dir, "repo_config.py")):
         sys.path.insert(0, repo_dir)
     import repo_config
-    from process_pr import modify_comment, find_last_comment
     from process_pr import TRIGERING_TESTS_MSG, TRIGERING_STYLE_TEST_MSG
 
     valid_types["JENKINS_TEST_URL"] = ["^\\s*" + TRIGERING_TESTS_MSG + ".*$", None]