Commit a636336a authored by Jérome Perrin's avatar Jérome Perrin

Zelenium: do not include (HTML) link in case of expected failure

89115b88 is great, but with "testFunctionalAnonymousSelection" the
result HTML is more than 40Mo.

/reviewed-on !772
parent a76b5815
...@@ -502,6 +502,8 @@ objectExtend(HtmlTestCaseRow.prototype, { ...@@ -502,6 +502,8 @@ objectExtend(HtmlTestCaseRow.prototype, {
// Store the test HTML as a dataURL to ease debugging from posted test results // Store the test HTML as a dataURL to ease debugging from posted test results
// This code should be synchronous (no Blob asDataURL method for example) // This code should be synchronous (no Blob asDataURL method for example)
// We don't do this with "expected failures" tests, not to create too big result pages
if (testFrame.currentTestCase.headerRow.trElement.textContent.indexOf("expected failure") === -1) {
var aElement = this.trElement.ownerDocument.createElement("a"); var aElement = this.trElement.ownerDocument.createElement("a");
aElement.textContent = ' (HTML)'; aElement.textContent = ' (HTML)';
function b64EncodeUnicode(str) { function b64EncodeUnicode(str) {
...@@ -511,6 +513,7 @@ objectExtend(HtmlTestCaseRow.prototype, { ...@@ -511,6 +513,7 @@ objectExtend(HtmlTestCaseRow.prototype, {
} }
aElement.href = 'data:text/html;charset=utf-8;base64,' + b64EncodeUnicode(sel$('selenium_myiframe').contentWindow.document.body.innerHTML); aElement.href = 'data:text/html;charset=utf-8;base64,' + b64EncodeUnicode(sel$('selenium_myiframe').contentWindow.document.body.innerHTML);
this.trElement.cells[2].appendChild(aElement); this.trElement.cells[2].appendChild(aElement);
}
}, },
setMessage: function(message) { setMessage: function(message) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment