Closed
Bug 1090148
Opened 10 years ago
Closed 10 years ago
[mozlog] Remove test class from HTML formatter
Categories
(Testing :: Mozbase, defect)
Testing
Mozbase
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla36
People
(Reporter: davehunt, Assigned: davehunt)
References
Details
Attachments
(1 file)
(deleted),
patch
|
chmanchester
:
review+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #1065406 +++
We just need a way of identifying the test in the HTML report, so there's no need to split it into class/method.
Assignee | ||
Comment 1•10 years ago
|
||
This also include a small fix for the CSS to display the correct colours in the report.
Attachment #8512555 -
Flags: review?(james)
Assignee | ||
Comment 2•10 years ago
|
||
Comment on attachment 8512555 [details] [diff] [review]
Remove test class from HTML formatter. v1.0
Chris has said he can take this review.
Attachment #8512555 -
Flags: review?(james) → review?(cmanchester)
Comment 3•10 years ago
|
||
Comment on attachment 8512555 [details] [diff] [review]
Remove test class from HTML formatter. v1.0
Review of attachment 8512555 [details] [diff] [review]:
-----------------------------------------------------------------
I'm not sure of the intent of the original code, so I'd be a little curious to know why the empty column ended up here in the first place. I'm also not sure who else might be using this for reports or how.
But the change is reasonable and the report it produces looks good! Worst case scenario we can roll back if anything unexpected comes up.
::: testing/mozbase/mozlog/mozlog/structured/formatters/html/html.py
@@ +146,5 @@
> additional_html.append(log)
>
> self.result_rows.append(
> html.tr([html.td(status_name, class_='col-result'),
> + html.td(unicode(data["test"]), class_='col-name'),
"test" fields are always coerced to unicode before reaching the formatter now, you can leave out the coercion here if you like.
Attachment #8512555 -
Flags: review?(cmanchester) → review+
Assignee | ||
Comment 4•10 years ago
|
||
Landed in mozilla-inbound:
http://hg.mozilla.org/integration/mozilla-inbound/rev/c891f30c5a5f
Comment 5•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in
before you can comment on or make changes to this bug.
Description
•