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

[Xisco Fauli]

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

New commits:
commit 038eadf60123a2c1f13fc4023fb0552dce03950b
Author: Xisco Fauli <xiscofauli at libreoffice.org>
Date:   Wed Mar 28 17:39:48 2018 +0200

    QA: Warn about bisected regressions that can be verified

diff --git a/qa/bugzillaChecker.py b/qa/bugzillaChecker.py
index 2d282b8..8e501fe 100755
--- a/qa/bugzillaChecker.py
+++ b/qa/bugzillaChecker.py
@@ -207,6 +207,10 @@ def analyze_bugzilla_checkers(statList, bugzillaData, cfg):
                                 movedToNew = True
                                 movedToNewValue = resultValue
 
+                            if addedStatus == 'RESOLVED_FIXED' and rowStatus == 'RESOLVED_FIXED' and 'regression' in rowKeywords \
+                                    and 'bisected' in rowKeywords:
+                                util_add_to_result(lResults, 'verify_fix', resultValue)
+
                     elif change['field_name'] == 'resolution':
                         if newStatus:
                             addedStatus = newStatus + "_" + change['added']
commit 14db751d175a5333323e6cb5efdc310bbdec4740
Author: Xisco Fauli <xiscofauli at libreoffice.org>
Date:   Wed Mar 28 14:00:25 2018 +0200

    QA: We only care about open bugs

diff --git a/qa/bugzillaChecker.py b/qa/bugzillaChecker.py
index 0a73fae..2d282b8 100755
--- a/qa/bugzillaChecker.py
+++ b/qa/bugzillaChecker.py
@@ -229,13 +229,14 @@ def analyze_bugzilla_checkers(statList, bugzillaData, cfg):
                                         if keyword == 'patch' and (common.isOpen(rowStatus) or rowStatus == 'UNCONFIRMED'):
                                             util_add_to_result(lResults, 'patch_added', resultValue)
 
-                                        if keyword == 'regression' and (common.isOpen(rowStatus) or rowStatus == 'UNCONFIRMED') and \
+                                        if keyword == 'regression' and row['status'] != 'RESOLVED' and \
                                                 'bibisectRequest' not in rowKeywords and 'bibisected' not in rowKeywords and \
                                                 'bisected' not in rowKeywords and 'preBibisect' not in rowKeywords and \
                                                 'bibisectNotNeeded' not in rowKeywords and 'notBibisectable' not in rowKeywords:
                                             util_add_to_result(lResults, 'regression_added', resultValue)
 
-                                        if keyword == 'possibleRegression' and 'possibleRegression' in rowKeywords:
+                                        if keyword == 'possibleRegression' and 'possibleRegression' in rowKeywords and \
+                                                row['status'] != 'RESOLVED':
                                             util_add_to_result(lResults, 'possibleregression_added', resultValue)
 
                     elif change['field_name'] == 'whiteboard':