check-missing-unittests: use regex and simplify code

Change-Id: Ica37de1adad0e97d1efd5f3f699d570af97ca90e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/105188
Tested-by: Xisco Fauli <xiscofauli@libreoffice.org>
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>

1 files changed, 4 insertions, 11 deletions

diff --git a/bin/check-missing-unittests.py b/bin/check-missing-unittests.py
index cab6509fda93..901aa40cc82f 100755
--- a/bin/check-missing-unittests.py
+++ b/bin/check-missing-unittests.py
@@ -10,6 +10,7 @@ import os
 import datetime
 import subprocess
 import sys
+import re
 
 def main(ignoredBugs):
     results = {
@@ -50,7 +51,9 @@ def main(ignoredBugs):
         summary = commitInfo[0].strip('"').lower()
 
         #Check summary has a bug id
-        if 'tdf#' in summary or 'fdo#' in summary:
+        reBugId = re.search(r'(?<=tdf#|fdo#)\d{5,6}\b', summary)
+        if reBugId:
+            bugId = reBugId.group()
 
             isIgnored = False
             for i in ignoredBugs:
@@ -59,16 +62,6 @@ def main(ignoredBugs):
             if isIgnored:
                 continue
 
-            if 'tdf#' in summary:
-                if not summary.split('tdf#')[1][0].isdigit():
-                    continue
-                bugId = ''.join(filter(str.isdigit, summary.split('tdf#')[1].split(' ')[0]))
-            elif 'fdo#' in summary:
-                if not summary.split('fdo#')[1][0].isdigit():
-                    continue
-                bugId = ''.join(filter(str.isdigit, summary.split('fdo#')[1].split(' ')[0]))
-
-
             if bugId in hasTestSet:
                 continue