History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: SEL-383
Type: Bug Bug
Status: Open Open
Priority: Major Major
Assignee: Unassigned
Reporter: Jean-Charles Meyrignac
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Selenium

<table> missing in the final report

Created: 23/Nov/06 10:00 AM   Updated: 24/Nov/06 04:50 AM
Component/s: TestRunner GUI/Suite Loader
Affects Version/s: 0.8.1
Fix Version/s: None

Original Estimate: Unknown Remaining Estimate: Unknown Time Spent: Unknown
Environment: Firefox 2.0 + XP SP2


 Description  « Hide
Here is an extract of a test report I just ran with Selenium RC 0.9
...
<table id="suiteTable" ...
</table><tr>

A <table> is missing just before the <tr> !

I found this issue when trying to load a report file into an XML editor.
Once <table> is added, the Firefox HTML report file is considered as correct XML, but the same IE report file is not valid, due to several incorrect attributes.

Here is a list of the errors I encountered:
- <TABLE class=selenium id=suiteTable cellSpacing=1 cellPadding=1 border=1>
All attributes should have quotes:
<TABLE class="selenium" id="suiteTable" cellSpacing="1" cellPadding="1" border="1">
(I guess this problem is related to IE)
- <TR class=" status_done"
Why 2 spaces before status_done ? (problem only on IE)
- <TD originalHTML>
IE tends to generate empty attributes.


 All   Comments   Work Log   Change History      Sort Order:
Jean-Charles Meyrignac - 24/Nov/06 04:50 AM
The <table> bug can be fixed by modifying HTMLTestResults.java as follows:
In public void write(Writer out):
   
out.write("<table>"); /* was missing !!! */

for (int i = 0; i < testTables.size(); i++) {
            String table = testTables.get(i).replace("\u00a0", "&nbsp;");
            out.write(MessageFormat.format(SUITE_HTML, i, suite.getHref(i), table));
        }
        out.write("</table></body></html>");

All the other problems are related to selenium-testrunner.js.