Hello community, here is the log from the commit of package python-openqa_review for openSUSE:Factory checked in at 2017-08-28 15:17:46 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/python-openqa_review (Old) and /work/SRC/openSUSE:Factory/.python-openqa_review.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Package is "python-openqa_review" Mon Aug 28 15:17:46 2017 rev:15 rq:518876 version:1.7.4 Changes: -------- --- /work/SRC/openSUSE:Factory/python-openqa_review/python-openqa_review.changes 2017-07-25 11:41:23.607895027 +0200 +++ /work/SRC/openSUSE:Factory/.python-openqa_review.new/python-openqa_review.changes 2017-08-28 15:19:09.875868826 +0200 @@ -1,0 +2,9 @@ +Sat Aug 26 08:53:36 UTC 2017 - okurz@suse.com + +- Update to version 1.7.4: + * Fix error 401 on redmine after server side ngingx update + * Ignore cscope.out + * Fix whitespace as reported by flake8 + * Fix handling of dotted build numbers parsing from comments + +------------------------------------------------------------------- Old: ---- python-openqa_review-1.7.2.tar.gz New: ---- python-openqa_review-1.7.4.tar.gz ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ python-openqa_review.spec ++++++ --- /var/tmp/diff_new_pack.iYob47/_old 2017-08-28 15:19:11.747605796 +0200 +++ /var/tmp/diff_new_pack.iYob47/_new 2017-08-28 15:19:11.751605234 +0200 @@ -18,7 +18,7 @@ %define short_name openqa_review Name: python-%{short_name} -Version: 1.7.2 +Version: 1.7.4 Release: 0 Summary: A review helper script for openQA License: MIT ++++++ _update.sh ++++++ --- /var/tmp/diff_new_pack.iYob47/_old 2017-08-28 15:19:11.887586125 +0200 +++ /var/tmp/diff_new_pack.iYob47/_new 2017-08-28 15:19:11.887586125 +0200 @@ -8,3 +8,4 @@ osc service localrun download_files osc add *.tar.gz osc ci -m "Update to $version" +osc sr -m "Update to $version" ++++++ python-openqa_review-1.7.2.tar.gz -> python-openqa_review-1.7.4.tar.gz ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/openqa_review-1.7.2/.gitignore new/openqa_review-1.7.4/.gitignore --- old/openqa_review-1.7.2/.gitignore 2017-07-24 17:06:59.000000000 +0200 +++ new/openqa_review-1.7.4/.gitignore 2017-08-26 10:51:56.000000000 +0200 @@ -8,3 +8,4 @@ version.py coverage_html/ .*.swp +cscope.out diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/openqa_review-1.7.2/openqa_review/browser.py new/openqa_review-1.7.4/openqa_review/browser.py --- old/openqa_review-1.7.2/openqa_review/browser.py 2017-07-24 17:06:59.000000000 +0200 +++ new/openqa_review-1.7.4/openqa_review/browser.py 2017-08-26 10:51:56.000000000 +0200 @@ -154,7 +154,7 @@ else: # pragma: no cover absolute_url = url if not url.startswith('/') else urljoin(str(self.root_url), str(url)) data = json.dumps(data) - r = requests.request(method, absolute_url, data=data, auth=self.auth, headers={'content-type': 'application/json'}) + r = requests.request(method, absolute_url, data=data, headers={'X-Redmine-API-Key': self.auth[0], 'content-type': 'application/json'}) r.raise_for_status() return r.json() if r.text else None diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/openqa_review-1.7.2/openqa_review/openqa_review.py new/openqa_review-1.7.4/openqa_review/openqa_review.py --- old/openqa_review-1.7.2/openqa_review/openqa_review.py 2017-07-24 17:06:59.000000000 +0200 +++ new/openqa_review-1.7.4/openqa_review/openqa_review.py 2017-08-26 10:51:56.000000000 +0200 @@ -229,6 +229,7 @@ 'poo': lambda i: 'https://progress.opensuse.org/issues/%s' % i, } + def status(entry): """Return test status from entry, e.g. 'result_passed'.""" return [s for s in entry.i['class'] if re.search('(state|result)_', s)][0] @@ -397,7 +398,7 @@ # Could also find previous one with a comment on the build status, # i.e. a reviewed finished build # The build number itself might be prefixed with a redundant 'Build' which we ignore - build_re = re.compile('[bB]uild:(\*\*)? *(Build)?([\w@]*)(.*reference.*)?(\*\*)?\r\n') + build_re = re.compile('[bB]uild:(\*\*)? *(Build)?([\w@.]*)(.*reference.*)?(\*\*)?\r\n') # Assuming the most recent with a build number also has the most recent review for c in reversed(comments): match = build_re.search(c['text']) @@ -807,7 +808,6 @@ assert bug_id, "No bug_id found for %s" % v v['bugref_href'] = issue_tracker[bugref](bug_id) - @property def total_issues(self): """Return Number of issue entries for this arch.""" diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/openqa_review-1.7.2/tests/:tests:overview%3Fdistri%3Dopensuse%26version%3D15%26build%3D162.2%26groupid%3D26 new/openqa_review-1.7.4/tests/:tests:overview%3Fdistri%3Dopensuse%26version%3D15%26build%3D162.2%26groupid%3D26 --- old/openqa_review-1.7.2/tests/:tests:overview%3Fdistri%3Dopensuse%26version%3D15%26build%3D162.2%26groupid%3D26 1970-01-01 01:00:00.000000000 +0100 +++ new/openqa_review-1.7.4/tests/:tests:overview%3Fdistri%3Dopensuse%26version%3D15%26build%3D162.2%26groupid%3D26 2017-08-28 15:19:12.467504631 +0200 @@ -0,0 +1 @@ +symbolic link to :tests:overview%3Fdistri%3Dopensuse%26version%3D42.1%26build%3D0311%26groupid%3D25 diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/openqa_review-1.7.2/tests/:tests:overview%3Fdistri%3Dopensuse%26version%3D15%26build%3D170.1%26groupid%3D26 new/openqa_review-1.7.4/tests/:tests:overview%3Fdistri%3Dopensuse%26version%3D15%26build%3D170.1%26groupid%3D26 --- old/openqa_review-1.7.2/tests/:tests:overview%3Fdistri%3Dopensuse%26version%3D15%26build%3D170.1%26groupid%3D26 1970-01-01 01:00:00.000000000 +0100 +++ new/openqa_review-1.7.4/tests/:tests:overview%3Fdistri%3Dopensuse%26version%3D15%26build%3D170.1%26groupid%3D26 2017-08-28 15:19:12.419511375 +0200 @@ -0,0 +1 @@ +symbolic link to :tests:overview%3Fdistri%3Dopensuse%26version%3D42.1%26build%3D0313%26groupid%3D25 diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/openqa_review-1.7.2/tests/https%3A::openqa.opensuse.org:group_overview:26.json new/openqa_review-1.7.4/tests/https%3A::openqa.opensuse.org:group_overview:26.json --- old/openqa_review-1.7.2/tests/https%3A::openqa.opensuse.org:group_overview:26.json 1970-01-01 01:00:00.000000000 +0100 +++ new/openqa_review-1.7.4/tests/https%3A::openqa.opensuse.org:group_overview:26.json 2017-08-26 10:51:56.000000000 +0200 @@ -0,0 +1,77 @@ +{ + "comments": [ + {"created": "2015-12-17T15:46:59", "user": "bar", "text": "**Date: 2015-12-17 - 16:45**\r\n**Build: 162.1**\r\n\r\n**Common issues:** baz\r\n<hr>"}, + {"created": "2015-12-18T13:28:20", "user": "foo", "text": "**Date: 2015-12-18 - 11:16**\r\n**Build: 162.2 (reference 162.1)**\r\n\r\n**Common issues:**\r\n"}, + {"created": "2015-12-19T13:28:20", "user": "baz", "text": "Some other comment"} + ], + "pinned_comments": [], + "description": "", + "max_jobs": 91, + "group": {"id": 26, "name": "openSUSE Leap"}, + "result": { + "162.1": { + "distri": "opensuse", + "version": "15", + "reviewed": "", + "reviewed_all_passed": "", + "escaped_build": "162_1", + "escaped_id": "15-162_1", + "labeled": 0, + "oldest": "2016-11-08T16:59:11", + "failed": 20, + "passed": 39, + "skipped": 0, + "softfailed": 0, + "unfinished": 0, + "total": 59 + }, + "162.2": { + "distri": "opensuse", + "version": "15", + "reviewed": "", + "reviewed_all_passed": "", + "escaped_build": "162_2", + "escaped_id": "15-162_2", + "labeled": 0, + "oldest": "2016-11-08T16:59:11", + "failed": 9, + "passed": 38, + "skipped": 0, + "softfailed": 0, + "unfinished": 0, + "total": 47 + }, + "170.1": { + "distri": "opensuse", + "version": "15", + "reviewed": "", + "reviewed_all_passed": "", + "escaped_build": "170_1", + "escaped_id": "15-170_1", + "labeled": 0, + "oldest": "2016-11-08T16:59:11", + "failed": 9, + "passed": 38, + "skipped": 0, + "softfailed": 0, + "unfinished": 0, + "total": 47 + }, + "170.4": { + "distri": "opensuse", + "version": "15", + "reviewed": "", + "reviewed_all_passed": "", + "escaped_build": "170_4", + "escaped_id": "15-170_4", + "labeled": 0, + "oldest": "2016-11-08T16:59:11", + "failed": 21, + "passed": 37, + "skipped": 0, + "softfailed": 0, + "unfinished": 33, + "total": 91 + } + } +} diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/openqa_review-1.7.2/tests/test_openqa_review.py new/openqa_review-1.7.4/tests/test_openqa_review.py --- old/openqa_review-1.7.2/tests/test_openqa_review.py 2017-07-24 17:06:59.000000000 +0200 +++ new/openqa_review-1.7.4/tests/test_openqa_review.py 2017-08-26 10:51:56.000000000 +0200 @@ -282,6 +282,15 @@ current, reviewed = openqa_review.get_build_urls_to_compare(browser, args.job_group_urls, against_reviewed='last') +def test_also_dotted_builds_can_be_specified_and_appear_in_report(): + args = cache_test_args_factory() + args.job_group_urls = args.host + '/group_overview/26' + browser = browser_factory(args) + current, reviewed = openqa_review.get_build_urls_to_compare(browser, args.job_group_urls, against_reviewed='last') + assert '=170.1' in current + assert '=162.2' in reviewed + + def test_generate_report_with_progress_notification_does_not_fail(): args = cache_test_args_factory() # Not easy to test automatically but at least we can call it and assume it also gives valid results