[Libreoffice-commits] dev-tools.git: qa/bugzillaChecker.py

[Xisco Fauli]

 qa/bugzillaChecker.py |   16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

New commits:
commit 915ad1f4f9b40d1427b07ce1bcc4b31197015c43
Author: Xisco Fauli <xiscofauli at libreoffice.org>
Date:   Wed May 9 15:32:46 2018 +0200

    QA: find bugs being confirmed without any metabug

diff --git a/qa/bugzillaChecker.py b/qa/bugzillaChecker.py
index 27f58c2..5f025b7 100755
--- a/qa/bugzillaChecker.py
+++ b/qa/bugzillaChecker.py
@@ -199,12 +199,7 @@ def analyze_bugzilla_checkers(statList, bugzillaData, cfg):
                                     row['assigned_to'] == 'libreoffice-bugs at lists.freedesktop.org':
                                 util_add_to_result(lResults, 'add_assignee', resultValue)
 
-                            if addedStatus == 'NEW' and rowStatus == 'NEW' and row['product'] == 'LibreOffice' and \
-                                    row['severity'] != 'enhancement' and \
-                                    ('regression' not in rowKeywords and 'bisected' not in rowKeywords and \
-                                    'easyHack' not in rowKeywords) and row['component'] != 'Documentation' and \
-                                    actionMail not in cfg['configQA']['ignore']['confirmer'] and \
-                                    (rowVersion.startswith(versionsToCheck) or rowVersion == 'unspecified'):
+                            if addedStatus == 'NEW' and rowStatus == 'NEW' and row['product'] == 'LibreOffice':
                                 movedToNew = True
                                 movedToNewValue = resultValue
 
@@ -322,7 +317,14 @@ def analyze_bugzilla_checkers(statList, bugzillaData, cfg):
             if movedToNeedInfo and everConfirmed:
                 util_add_to_result(lResults, 'moved_to_needinfo', movedToNeedInfoValue)
 
-            if not versionChanged and movedToNew and not autoConfirmed:
+            if movedToNew and not row['blocks']:
+                util_add_to_result(lResults, 'metabug_not_added', movedToNewValue)
+
+            if not versionChanged and movedToNew and not autoConfirmed and row['severity'] != 'enhancement' and \
+                    ('regression' not in rowKeywords and 'bisected' not in rowKeywords and \
+                    'easyHack' not in rowKeywords) and row['component'] != 'Documentation' and \
+                    movedToNewValue[2] not in cfg['configQA']['ignore']['confirmer'] and \
+                    (rowVersion.startswith(versionsToCheck) or rowVersion == 'unspecified'):
                 util_add_to_result(lResults, 'version_not_changed', movedToNewValue)
 
             #Check bugs where: