Better HTML output for System Testing Report

Bug #1433873 reported by Pierre Equoy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
PlainBox (Toolkit)
Fix Released
Medium
Sylvain Pineau

Bug Description

Plainbox 0.20 introduces the concept of certification blocker tests.

When a testing session ends, Plainbox generates an HTML document with the results. Starting with v0.20, the HTML output contains a "Certification Status" column in the "Tests Performed" section as well as a "Certification Blockers" section.

Two requests:

1. Move the "Certification Status" column next to the "Result" column, so it's easier to identify blocker issues that have failed. (see screenshot attached)
2. In the new "Certification Blockers" section, separate the tests by results:
    - Failed
    - Skipped
This way, it will be very easy to identify the failed cert-blockers.

Related branches

Revision history for this message
Pierre Equoy (pieq) wrote :
tags: added: ce-qa-concern
Revision history for this message
Pierre Equoy (pieq) wrote :

I guess it's just an XSL change away, in this file:
checkbox/plainbox/plainbox/data/report/checkbox.xsl

Revision history for this message
Zygmunt Krynicki (zyga) wrote : Re: [Bug 1433873] Re: Better HTML output for System Testing Report

Hi Pierre

Both requests make sense. If you want to do that with the current
stack then go for it. Otherwise I'd like to wait until we have the
jinja2 based reports. Hacking on xslt is pretty frustrating and time
consuming.

Best regards
ZK

On Thu, Mar 19, 2015 at 7:18 AM, Pierre Equoy
<email address hidden> wrote:
> I guess it's just an XSL change away, in this file:
> checkbox/plainbox/plainbox/data/report/checkbox.xsl
>
> --
> You received this bug notification because you are a member of Checkbox
> Bug Wranglers, which is subscribed to Checkbox.
> https://bugs.launchpad.net/bugs/1433873
>
> Title:
> Better HTML output for System Testing Report
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/checkbox/+bug/1433873/+subscriptions

Changed in checkbox:
status: New → In Progress
assignee: nobody → Sylvain Pineau (sylvain-pineau)
importance: Undecided → Medium
affects: checkbox → plainbox
Changed in plainbox:
milestone: none → 0.21
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

It think the first request makes sense, the second one will be hard to implement in the current format.

Revision history for this message
Pierre Equoy (pieq) wrote :

@Zygmunt: Jinja2!!! \o/

@Sylvain: If you want I can make a few tests tomorrow/next week and provide a branch for it.

Just a question: how do you test the Plainbox export feature if you already have an XML file available?

Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

Only 1. is done in the linked MR. 2. is nice to have but not critical. It will be provided by new versions of the html report (Jquery based)

Zygmunt Krynicki (zyga)
Changed in plainbox:
status: In Progress → Fix Committed
Revision history for this message
Pierre Equoy (pieq) wrote :

Tested point 1 successfully.

As noted by Sylvain, point 2 can be achieved at a later date with the new template feature, so it's been dismissed.

tags: added: cqa-verified
Changed in plainbox:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.