<p dir="ltr">I don't have the full sources here, but assuming that warm will become pass, you can add my reviewed-by</p>
<div class="gmail_quote">On Jun 26, 2015 3:35 PM, "Ilia Mirkin" <<a href="mailto:imirkin@alum.mit.edu">imirkin@alum.mit.edu</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Reviewed-by: Ilia Mirkin <<a href="mailto:imirkin@alum.mit.edu">imirkin@alum.mit.edu</a>><br>
<br>
On Fri, Jun 26, 2015 at 6:35 PM, Mark Janes <<a href="mailto:mark.a.janes@intel.com">mark.a.janes@intel.com</a>> wrote:<br>
> JUnit has no concept of "warning". It supports the following<br>
> statuses:<br>
><br>
> - skip<br>
> - success<br>
> - fail<br>
> - error<br>
><br>
> A test which emits a warning is more accurately represented as<br>
> "success" in JUnit.<br>
><br>
> v2: Continue to report failure for "dmesg-warn", which is more serious<br>
> than "warn". (from Ilia Mirkin)<br>
> ---<br>
> framework/backends/junit.py | 3 +--<br>
> 1 file changed, 1 insertion(+), 2 deletions(-)<br>
><br>
> diff --git a/framework/backends/junit.py b/framework/backends/junit.py<br>
> index 632e516..8c8639e 100644<br>
> --- a/framework/backends/junit.py<br>
> +++ b/framework/backends/junit.py<br>
> @@ -142,8 +142,7 @@ class JUnitBackend(FileBackend):<br>
> if data['result'] == 'skip':<br>
> res = etree.SubElement(element, 'skipped')<br>
><br>
> - elif data['result'] in ['warn', 'fail', 'dmesg-warn',<br>
> - 'dmesg-fail']:<br>
> + elif data['result'] in ['fail', 'dmesg-warn', 'dmesg-fail']:<br>
> if expected_result == "failure":<br>
> err.text += "\n\nWARN: passing test as an expected failure"<br>
> res = etree.SubElement(element, 'skipped',<br>
> --<br>
> 2.1.4<br>
><br>
</blockquote></div>