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 - [email protected]
+
+- 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
