[FIX] migration_issue_bot: Duplicated line for PRs

A new line was created because the regex was expecting something after the module name
OCA · Oct 10, 2022 · a721476 · a721476
1 parent d6b4161
commit a721476
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/oca_github_bot/tasks/migration_issue_bot.py b/src/oca_github_bot/tasks/migration_issue_bot.py
@@ -39,7 +39,7 @@ def _set_lines_issue(gh_pr_user_login, gh_pr_number, issue_body, module):
         if added:  # Bypass the checks for faster completion
             lines.append(line)
             continue
-        groups = re.match(rf"^- \[( |x)\] {module}( |\r)", line)
+        groups = re.match(rf"^- \[( |x)\] {module}", line)
         if groups:  # Line found
             # Get the Old PR value
             regex = r"\#(\d*)"

diff --git a/tests/test_migration_issue_bot.py b/tests/test_migration_issue_bot.py
@@ -53,6 +53,14 @@ def test_set_lines_issue(gh):
             f"- [x] {module} - By @{gh_pr_user_login} - #{gh_pr_number}\n"
             f"- [ ] z_module_1 - By @pedrobaeza - #2",
         ),
+        (
+            f"Issue with list containing the module with no PR\n"
+            f"- [x] {module}\n"
+            f"- [ ] z_module_1 - By @pedrobaeza - #2",
+            f"Issue with list containing the module with no PR\n"
+            f"- [x] {module} - By @{gh_pr_user_login} - #{gh_pr_number}\n"
+            f"- [ ] z_module_1 - By @pedrobaeza - #2",
+        ),
         (
             "Issue with no list",
             f"Issue with no list\n"