Hello community,
here is the log from the commit of package python-openqa_review for
openSUSE:Factory checked in at 2018-07-27 10:56:02
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/python-openqa_review (Old)
and /work/SRC/openSUSE:Factory/.python-openqa_review.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "python-openqa_review"
Fri Jul 27 10:56:02 2018 rev:24 rq:625324 version:1.12.1
Changes:
--------
---
/work/SRC/openSUSE:Factory/python-openqa_review/python-openqa_review.changes
2018-07-12 09:21:51.126644044 +0200
+++
/work/SRC/openSUSE:Factory/.python-openqa_review.new/python-openqa_review.changes
2018-07-27 10:56:04.405627200 +0200
@@ -1,0 +2,9 @@
+Wed Jul 25 07:03:33 UTC 2018 - [email protected]
+
+- Update to version 1.12.1:
+ * Fix problem introduced in 4189852 parsing the URL parameter
+ * Fix non-ascii characters in external references
+ * Fix report_link for pre-4.6
+ * Add backwards-compatible support for pre-4.6 openQA
+
+-------------------------------------------------------------------
Old:
----
python-openqa_review-1.11.0.obscpio
New:
----
python-openqa_review-1.12.1.obscpio
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ python-openqa_review.spec ++++++
--- /var/tmp/diff_new_pack.Tr3Jik/_old 2018-07-27 10:56:05.161628647 +0200
+++ /var/tmp/diff_new_pack.Tr3Jik/_new 2018-07-27 10:56:05.165628655 +0200
@@ -31,7 +31,7 @@
%define binaries openqa-review openqa-review-daily-email
openqa-review-sles-ha tumblesle-release
openqa-review-functional_autoyast_concise
%define oldpython python
Name: python-%{short_name}%{?name_ext}
-Version: 1.11.0
+Version: 1.12.1
Release: 0
Summary: A review helper script for openQA
License: MIT
++++++ _update.sh ++++++
--- /var/tmp/diff_new_pack.Tr3Jik/_old 2018-07-27 10:56:05.189628700 +0200
+++ /var/tmp/diff_new_pack.Tr3Jik/_new 2018-07-27 10:56:05.193628709 +0200
@@ -1,11 +1,9 @@
#!/bin/sh -e
osc up
+osc rm *.obscpio
osc service disabledrun
version=$(osc diff | sed -n 's/^.*Update to version \([0-9.]\+\):/\1/p')
sed -i -e "s/^\(Version:\s*\)[0-9.]\+/\1$version/g" python-openqa_review.spec
-rm *.tar.xz
-osc rm *.tar.gz
-osc service localrun download_files
-osc add *.tar.gz
+sc add *.obscpio
osc ci -m "Update to $version"
osc sr -m "Update to $version"
++++++ python-openqa_review-1.11.0.obscpio ->
python-openqa_review-1.12.1.obscpio ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/python-openqa_review-1.11.0/openqa_review/openqa_review.py
new/python-openqa_review-1.12.1/openqa_review/openqa_review.py
--- old/python-openqa_review-1.11.0/openqa_review/openqa_review.py
2018-07-11 07:55:36.000000000 +0200
+++ new/python-openqa_review-1.12.1/openqa_review/openqa_review.py
2018-07-25 07:59:04.000000000 +0200
@@ -89,7 +89,7 @@
# Python 2 and 3: easiest option
# see http://python-future.org/compatible_idioms.html
-from __future__ import absolute_import
+from __future__ import absolute_import, unicode_literals
from future.standard_library import install_aliases # isort:skip to keep
'install_aliases()'
install_aliases()
from future.utils import iteritems
@@ -101,6 +101,7 @@
import re
import sys
import json
+from builtins import str
from collections import defaultdict, OrderedDict
from configparser import ConfigParser, NoSectionError, NoOptionError #
isort:skip can not make isort happy here
from requests.exceptions import HTTPError
@@ -489,7 +490,7 @@
return name, url, details
-def issue_report_link(root_url, f, test_browser=None):
+def issue_report_link(root_url, f, test_browser=None): # noqa: C901 # too
complex, we might want to remove this function anyway as openQA has it already
"""Generate a bug reporting link for the current issue."""
# always select the first failed module.
# It might not be the fatal one but better be safe and assume the first
@@ -500,11 +501,20 @@
overview_params = parse_qs(current_build_overview[-1])
group = overview_params['groupid'][0]
build = overview_params['build'][0]
- scenario_div = test_details_page.find(class_='next_previous').div.div
- scenario = re.findall('Next & previous results for (.*) \(',
scenario_div.text)[0]
+ try:
+ scenario_div = test_details_page.find(class_='next_previous').div.div
+ scenario = re.findall('Next & previous results for (.*) \(',
scenario_div.text)[0]
+ except AttributeError: # pragma: no cover
+ # pre-4.6
+ scenario_div = test_details_page.find(class_='previous').div.div
+ scenario = re.findall('[Rr]esults for (.*) \(', scenario_div.text)[0]
latest_link = absolute_url(root_url, scenario_div.a)
module, url, details = get_failed_module_details_for_report(f)
- previous_results = test_details_page.find(id='job_next_previous_table',
class_='overview').find_all('tr')[1:]
+ try:
+ previous_results =
test_details_page.find(id='job_next_previous_table',
class_='overview').find_all('tr')[1:]
+ except AttributeError: # pragma: no cover
+ # pre-4.6
+ previous_results = test_details_page.find(id='previous_results',
class_='overview').find_all('tr')[1:]
previous_results_list = [(i.td['id'], {'status': status(i),
'details': get_test_details(i),
'build':
i.find(class_='build').text}) for i in previous_results]
@@ -1160,7 +1170,7 @@
if args.job_group_urls:
root_url = urljoin('/'.join(args.job_group_urls.split("/")[0:3]), '/')
else:
- root_url = urljoin(args.host, '/')
+ root_url = urljoin(str(args.host), '/')
browser = Browser(args, root_url)
job_groups = get_job_groups(browser, root_url, args)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/python-openqa_review-1.11.0/tests/tags_labels/https%3A::progress.opensuse.org:issues:11948.json
new/python-openqa_review-1.12.1/tests/tags_labels/https%3A::progress.opensuse.org:issues:11948.json
---
old/python-openqa_review-1.11.0/tests/tags_labels/https%3A::progress.opensuse.org:issues:11948.json
2018-07-11 07:55:36.000000000 +0200
+++
new/python-openqa_review-1.12.1/tests/tags_labels/https%3A::progress.opensuse.org:issues:11948.json
2018-07-25 07:59:04.000000000 +0200
@@ -1 +1 @@
-{"issue": {"done_ratio": 0, "created_on": "2016-05-12T12:49:00Z",
"description": "Something is wrong with created disk image", "assigned_to":
{"name": "user1", "id": 1}, "subject": "can not boot pre-installed image",
"tracker": {"name": "action", "id": 4}, "start_date": "2016-05-12", "category":
{"name": "Bugs in existing tests", "id": 152}, "author": {"name": "user2",
"id": 2}, "status": {"name": "New", "id": 1}, "id": 11948, "priority": {"name":
"Normal", "id": 4}, "project": {"name": "openQA Tests", "id": 36},
"updated_on": "2016-07-08T06:24:00Z"}}
+{"issue": {"done_ratio": 0, "created_on": "2016-05-12T12:49:00Z",
"description": "Something is wrong with created disk image", "assigned_to":
{"name": "user1", "id": 1}, "subject": "can not boot pre-installed image, test
non-ascii: \u201c0\u201d", "tracker": {"name": "action", "id": 4},
"start_date": "2016-05-12", "category": {"name": "Bugs in existing tests",
"id": 152}, "author": {"name": "user2", "id": 2}, "status": {"name": "New",
"id": 1}, "id": 11948, "priority": {"name": "Normal", "id": 4}, "project":
{"name": "openQA Tests", "id": 36}, "updated_on": "2016-07-08T06:24:00Z"}}
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/python-openqa_review-1.11.0/tests/tags_labels/report25_bugrefs_build1508.md
new/python-openqa_review-1.12.1/tests/tags_labels/report25_bugrefs_build1508.md
---
old/python-openqa_review-1.11.0/tests/tags_labels/report25_bugrefs_build1508.md
2018-07-11 07:55:36.000000000 +0200
+++
new/python-openqa_review-1.12.1/tests/tags_labels/report25_bugrefs_build1508.md
2018-07-25 07:59:04.000000000 +0200
@@ -23,7 +23,7 @@
**Existing openQA-issues:**
-* ssh-X@i586--l3, textmode ->
[poo#11948](https://progress.opensuse.org/issues/11948 "can not boot
pre-installed image") (Ticket status: New, prio/severity: Normal, assignee:
user1)
+* ssh-X@i586--l3, textmode ->
[poo#11948](https://progress.opensuse.org/issues/11948 "can not boot
pre-installed image, test non-ascii: “0”") (Ticket status: New, prio/severity:
Normal, assignee: user1)
* allpatterns@i586--l2 -> [poo#9772](https://progress.opensuse.org/issues/9772
"Nothing works") (Ticket status: New, prio/severity: High, assignee: None)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/python-openqa_review-1.11.0/tests/tags_labels/report25_bugrefs_query_issues.md
new/python-openqa_review-1.12.1/tests/tags_labels/report25_bugrefs_query_issues.md
---
old/python-openqa_review-1.11.0/tests/tags_labels/report25_bugrefs_query_issues.md
2018-07-11 07:55:36.000000000 +0200
+++
new/python-openqa_review-1.12.1/tests/tags_labels/report25_bugrefs_query_issues.md
2018-07-25 07:59:04.000000000 +0200
@@ -25,7 +25,7 @@
**Existing openQA-issues:**
-* textmode -> [poo#11948](https://progress.opensuse.org/issues/11948 "can not
boot pre-installed image") (Ticket status: New, prio/severity: Normal,
assignee: user1)
+* textmode -> [poo#11948](https://progress.opensuse.org/issues/11948 "can not
boot pre-installed image, test non-ascii: “0”") (Ticket status: New,
prio/severity: Normal, assignee: user1)
* allpatterns@i586--l2 -> [poo#9772](https://progress.opensuse.org/issues/9772
"Nothing works") (Ticket status: New, prio/severity: High, assignee: None)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/python-openqa_review-1.11.0/tests/test_openqa_review.py
new/python-openqa_review-1.12.1/tests/test_openqa_review.py
--- old/python-openqa_review-1.11.0/tests/test_openqa_review.py 2018-07-11
07:55:36.000000000 +0200
+++ new/python-openqa_review-1.12.1/tests/test_openqa_review.py 2018-07-25
07:59:04.000000000 +0200
@@ -7,10 +7,12 @@
"""
# see http://python-future.org/compatible_idioms.html
+from __future__ import unicode_literals
from future.standard_library import install_aliases # isort:skip to keep
'install_aliases()'
from future.utils import iteritems
install_aliases()
+import codecs
import contextlib
import os.path
import re
@@ -18,6 +20,7 @@
import sys
import tempfile
from argparse import Namespace
+from builtins import str
from openqa_review.browser import filename_to_url
from urllib.parse import urljoin, urlparse
from configparser import ConfigParser # isort:skip can not make isort happy
here
@@ -115,8 +118,8 @@
def compare_report(report, ref_report_path):
# If the reference report should be written initially or updated one can
just write the report as string to file
- # open(ref_report_path, 'w').write(str(report))
- ref_report = open(ref_report_path).read()
+ # codecs.open(ref_report_path, 'w', 'utf-8').write(report)
+ ref_report = codecs.open(ref_report_path, 'r', 'utf-8').read()
# for simpler display of the diff in case of differences it helps to have
# both reports in same encoding, i.e. casting to str
lines = str(report).splitlines()
@@ -465,7 +468,7 @@
# report generated when no todo items are left and some bugref is not
accessible
args.builds = '1508,1500'
args.include_softfails = False
- report = openqa_review.generate_report(args)
+ report = str(openqa_review.generate_report(args))
compare_report(report, os.path.join(args.load_dir,
'report25_bugrefs_build1508.md'))
@@ -493,12 +496,12 @@
# now, try filtering: unassigned
report = openqa_review.generate_report(args)
openqa_review.filter_report(report, openqa_review.ie_filters["unassigned"])
- compare_report(report, os.path.join(args.load_dir,
'report25_bugrefs_query_issues_filter_unassigned.md'))
+ compare_report(str(report), os.path.join(args.load_dir,
'report25_bugrefs_query_issues_filter_unassigned.md'))
# 2nd filter: closed
report = openqa_review.generate_report(args)
openqa_review.filter_report(report, openqa_review.ie_filters["closed"])
- compare_report(report, os.path.join(args.load_dir,
'report25_bugrefs_query_issues_filter_closed.md'))
+ compare_report(str(report), os.path.join(args.load_dir,
'report25_bugrefs_query_issues_filter_closed.md'))
def test_arch_distinguish():
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/python-openqa_review-1.11.0/tox.ini
new/python-openqa_review-1.12.1/tox.ini
--- old/python-openqa_review-1.11.0/tox.ini 2018-07-11 07:55:36.000000000
+0200
+++ new/python-openqa_review-1.12.1/tox.ini 2018-07-25 07:59:04.000000000
+0200
@@ -28,7 +28,7 @@
max-complexity = 10
# isort and documentation flags, minor importance
ignore = I002,D100,D103,D104,D203,D204,D211,E402
-exclude = .ropeproject/,.tox/
+exclude = .ropeproject/,.tox/,venv/
[testenv:doctests]
# only run doctest on main modules to prevent parsing all files including
++++++ python-openqa_review.obsinfo ++++++
--- /var/tmp/diff_new_pack.Tr3Jik/_old 2018-07-27 10:56:05.389629083 +0200
+++ /var/tmp/diff_new_pack.Tr3Jik/_new 2018-07-27 10:56:05.389629083 +0200
@@ -1,5 +1,5 @@
name: python-openqa_review
-version: 1.11.0
-mtime: 1531288536
-commit: c73bfc6bf6604a3e28002832da3dc00d2042ed0b
+version: 1.12.1
+mtime: 1532498344
+commit: 42a115d7fb88c0616a3f5940677b9ed361f88e91
