Hello community,
here is the log from the commit of package python-openqa_review for
openSUSE:Factory checked in at 2018-05-29 16:47:36
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/python-openqa_review (Old)
and /work/SRC/openSUSE:Factory/.python-openqa_review.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "python-openqa_review"
Tue May 29 16:47:36 2018 rev:22 rq:605874 version:1.10.1
Changes:
--------
---
/work/SRC/openSUSE:Factory/python-openqa_review/python-openqa_review.changes
2018-05-04 11:29:56.854780360 +0200
+++
/work/SRC/openSUSE:Factory/.python-openqa_review.new/python-openqa_review.changes
2018-05-29 16:47:37.830072274 +0200
@@ -1,0 +2,7 @@
+Wed May 09 18:45:40 UTC 2018 - [email protected]
+
+- Update to version 1.10.1:
+ * Fix parsing of failedmodules for bootstrap4
+ * Align travis python test versions to tox.ini
+
+-------------------------------------------------------------------
Old:
----
python-openqa_review-1.10.0.tar.gz
New:
----
python-openqa_review-1.10.1.tar.gz
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ python-openqa_review.spec ++++++
--- /var/tmp/diff_new_pack.QpbIsf/_old 2018-05-29 16:47:38.438049900 +0200
+++ /var/tmp/diff_new_pack.QpbIsf/_new 2018-05-29 16:47:38.438049900 +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.10.0
+Version: 1.10.1
Release: 0
Summary: A review helper script for openQA
License: MIT
++++++ python-openqa_review-1.10.0.tar.gz -> python-openqa_review-1.10.1.tar.gz
++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.10.0/.travis.yml
new/openqa_review-1.10.1/.travis.yml
--- old/openqa_review-1.10.0/.travis.yml 2018-02-09 14:34:50.000000000
+0100
+++ new/openqa_review-1.10.1/.travis.yml 2018-04-27 12:01:41.000000000
+0200
@@ -3,7 +3,7 @@
python:
- "2.7"
- "3.4"
- - "3.5"
+ - "3.6"
install: pip install tox-travis
script: tox
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.10.0/openqa_review/openqa_review.py
new/openqa_review-1.10.1/openqa_review/openqa_review.py
--- old/openqa_review-1.10.0/openqa_review/openqa_review.py 2018-02-09
14:34:50.000000000 +0100
+++ new/openqa_review-1.10.1/openqa_review/openqa_review.py 2018-04-27
12:01:41.000000000 +0200
@@ -246,8 +246,16 @@
def get_test_details(entry):
failedmodules = entry.find_all(class_='failedmodule')
+
+ def find_module_href(m):
+ # optional fallback to old openQA code, 4.5, pre bootstrap4
+ try:
+ return m['href']
+ except KeyError:
+ return m.a['href']
+
return {'href': entry.a['href'],
- 'failedmodules': [{'href': m.a['href'], 'name': m.text.strip(),
'needles': get_failed_needles(m)} for m in failedmodules]
+ 'failedmodules': [{'href': find_module_href(m), 'name':
m.text.strip(), 'needles': get_failed_needles(m)} for m in failedmodules]
}
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openqa_review-1.10.0/tests/openqa_4.5_dashboard/:tests:overview%3Fdistri%3Dkubic%26version%3DTumbleweed%26build%3D20180423%26groupid%3D41
new/openqa_review-1.10.1/tests/openqa_4.5_dashboard/:tests:overview%3Fdistri%3Dkubic%26version%3DTumbleweed%26build%3D20180423%26groupid%3D41
---
old/openqa_review-1.10.0/tests/openqa_4.5_dashboard/:tests:overview%3Fdistri%3Dkubic%26version%3DTumbleweed%26build%3D20180423%26groupid%3D41
1970-01-01 01:00:00.000000000 +0100
+++
new/openqa_review-1.10.1/tests/openqa_4.5_dashboard/:tests:overview%3Fdistri%3Dkubic%26version%3DTumbleweed%26build%3D20180423%26groupid%3D41
2018-04-27 12:01:41.000000000 +0200
@@ -0,0 +1,70 @@
+<!DOCTYPE html>
+<html lang="en">
+ <body>
+ <div class="container" id="content">
+
+
+
+<div>
+ <h2>Test result overview</h2>
+ <div id="summary" class="card border-danger">
+ <div class="card-header">
+ Overall Summary of
+ <b><a href="/group_overview/41">Development Kubic</a></b>
+ build 20180423
+ </div>
+ <div class="card-body">
+ Passed: <span class="badge badge-success">0</span>
+
+ Failed: <span class="badge badge-danger">1</span>
+
+ </div>
+ </div>
+ <h3>Flavor: DVD</h3>
+ <table id="results_DVD" class="overview table table-striped
table-hover">
+ <thead>
+ <tr>
+ <th>Test</th>
+ <th id="flavor_DVD_arch_x86_64">x86_64</th>
+ </tr>
+ </thead>
+ <tbody>
+ <tr>
+ <td class="name">
+ <span
title="MicroOS-autoyast-minimal">MicroOS-autoyast-minimal</span>
+ </td>
+
+
+ <td id="res_DVD_x86_64_MicroOS-autoyast-minimal">
+
+ <span id="res-661850">
+ <a href="/tests/661850">
+ <i class="status fa fa-circle result_failed" title="Done:
failed"></i>
+ </a>
+ </span>
+
+
+ <a
+ data-toggle="tooltip"
+ data-placement="top"
+ data-container="#res_DVD_x86_64_MicroOS-autoyast-minimal"
+ data-async="/tests/661850/modules/installation/fails"
+ title="<i class='fa fa-sync fa-spin fa-2x fa-fw'></i><span
class='sr-only'>Loading...</span>"
+ class="failedmodule"
+ href="/tests/661850#step/installation/1"
+ >
+ <span title="installation">installation</span>
+ </a>
+
+
+</td>
+
+ </tr>
+ </tbody>
+</table>
+
+</div>
+
+ </div>
+ </body>
+</html>
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openqa_review-1.10.0/tests/openqa_4.5_dashboard/:tests:overview%3Fdistri%3Dkubic%26version%3DTumbleweed%26build%3D20180424%26groupid%3D41
new/openqa_review-1.10.1/tests/openqa_4.5_dashboard/:tests:overview%3Fdistri%3Dkubic%26version%3DTumbleweed%26build%3D20180424%26groupid%3D41
---
old/openqa_review-1.10.0/tests/openqa_4.5_dashboard/:tests:overview%3Fdistri%3Dkubic%26version%3DTumbleweed%26build%3D20180424%26groupid%3D41
1970-01-01 01:00:00.000000000 +0100
+++
new/openqa_review-1.10.1/tests/openqa_4.5_dashboard/:tests:overview%3Fdistri%3Dkubic%26version%3DTumbleweed%26build%3D20180424%26groupid%3D41
2018-04-27 12:01:41.000000000 +0200
@@ -0,0 +1,70 @@
+<!DOCTYPE html>
+<html lang="en">
+ <body>
+ <div class="container" id="content">
+
+
+
+<div>
+ <h2>Test result overview</h2>
+ <div id="summary" class="card border-danger">
+ <div class="card-header">
+ Overall Summary of
+ <b><a href="/group_overview/41">Development Kubic</a></b>
+ build 20180424
+ </div>
+ <div class="card-body">
+ Passed: <span class="badge badge-success">0</span>
+
+ Failed: <span class="badge badge-danger">1</span>
+
+ </div>
+ </div>
+ <h3>Flavor: DVD</h3>
+ <table id="results_DVD" class="overview table table-striped
table-hover">
+ <thead>
+ <tr>
+ <th>Test</th>
+ <th id="flavor_DVD_arch_x86_64">x86_64</th>
+ </tr>
+ </thead>
+ <tbody>
+ <tr>
+ <td class="name">
+ <span
title="MicroOS-autoyast-minimal">MicroOS-autoyast-minimal</span>
+ </td>
+
+
+ <td id="res_DVD_x86_64_MicroOS-autoyast-minimal">
+
+ <span id="res-662477">
+ <a href="/tests/662477">
+ <i class="status fa fa-circle result_failed" title="Done:
failed"></i>
+ </a>
+ </span>
+
+
+ <a
+ data-toggle="tooltip"
+ data-placement="top"
+ data-container="#res_DVD_x86_64_MicroOS-autoyast-minimal"
+ data-async="/tests/662477/modules/installation/fails"
+ title="<i class='fa fa-sync fa-spin fa-2x fa-fw'></i><span
class='sr-only'>Loading...</span>"
+ class="failedmodule"
+ href="/tests/662477#step/installation/1"
+ >
+ <span title="installation">installation</span>
+ </a>
+
+
+</td>
+
+ </tr>
+ </tbody>
+</table>
+
+</div>
+
+ </div>
+ </body>
+</html>
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openqa_review-1.10.0/tests/openqa_4.5_dashboard/https%3A::openqa.opensuse.org:group_overview:41.json
new/openqa_review-1.10.1/tests/openqa_4.5_dashboard/https%3A::openqa.opensuse.org:group_overview:41.json
---
old/openqa_review-1.10.0/tests/openqa_4.5_dashboard/https%3A::openqa.opensuse.org:group_overview:41.json
1970-01-01 01:00:00.000000000 +0100
+++
new/openqa_review-1.10.1/tests/openqa_4.5_dashboard/https%3A::openqa.opensuse.org:group_overview:41.json
2018-04-27 12:01:41.000000000 +0200
@@ -0,0 +1 @@
+{"build_results": [{"all_passed": "", "build": "20180425", "distris":
{"kubic": 1}, "escaped_build": "20180425", "escaped_id": "Tumbleweed-20180425",
"escaped_version": "Tumbleweed", "failed": 0, "key": "Tumbleweed-20180425",
"labeled": 0, "oldest": "2018-04-26T11:07:48", "passed": 0, "reviewed": 1,
"skipped": 0, "softfailed": 0, "total": 1, "unfinished": 1, "version":
"Tumbleweed"}, {"all_passed": "", "build": "20180424", "distris": {"kubic": 1},
"escaped_build": "20180424", "escaped_id": "Tumbleweed-20180424",
"escaped_version": "Tumbleweed", "failed": 1, "key": "Tumbleweed-20180424",
"labeled": 0, "oldest": "2018-04-25T04:05:17", "passed": 0, "reviewed": "",
"skipped": 0, "softfailed": 0, "total": 1, "unfinished": 0, "version":
"Tumbleweed"}, {"all_passed": "", "build": "20180423", "distris": {"kubic": 1},
"escaped_build": "20180423", "escaped_id": "Tumbleweed-20180423",
"escaped_version": "Tumbleweed", "failed": 1, "key": "Tumbleweed-20180423",
"labeled": 0, "oldest": "2018-04-24T12:17:47", "passed": 0, "reviewed": "",
"skipped": 0, "softfailed": 0, "total": 1, "unfinished": 0, "version":
"Tumbleweed"}], "comments": [], "description": "", "group": {"id": 41, "name":
"Development Kubic", "parent_id": 6, "parent_name": "Development"}, "max_jobs":
1, "pinned_comments": []}
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.10.0/tests/test_openqa_review.py
new/openqa_review-1.10.1/tests/test_openqa_review.py
--- old/openqa_review-1.10.0/tests/test_openqa_review.py 2018-02-09
14:34:50.000000000 +0100
+++ new/openqa_review-1.10.1/tests/test_openqa_review.py 2018-04-27
12:01:41.000000000 +0200
@@ -237,6 +237,15 @@
assert 'Green' in report
+def test_openqa_45_bootstrap_4_can_parse_failed_modules():
+ args = cache_test_args_factory()
+ args.load_dir = os.path.join(os.path.dirname(os.path.realpath(__file__)),
'openqa_4.5_dashboard')
+ args.job_group_urls = 'https://openqa.opensuse.org/group_overview/41'
+ report = str(openqa_review.generate_report(args))
+ assert '20180424' in report
+ assert 'installation' in report
+
+
def test_get_build_urls_to_compare_finds_last_reviewed_if_selected():
args = cache_test_args_factory()
browser = browser_factory(args)
