vrakeshreddy / mb-unit

Automatically exported from code.google.com/p/mb-unit
0 stars 0 forks source link

Test report contains a large number of <wbr></wbr> statements - this bloats the size of the report #652

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Steps to reproduce:
1. Execute any test (MbUnit.Samples/SampleEquatableTest/EqualityTests) was
used to create attached file

Report is generated correctly - but report size is needlessly large.

What version of the product are you using? On what operating system?
MBUnit3.1, WindowsXPx32

I have searched the codebase for <wbr>, but the search returns no results.
Gallio-Report.css refers to <wbr>, but I cannot see where it is used
Gallio-Report.common.xsl inserts <wbr/>
HtmlTestStepRunFormatter.cs also inserts <wbr/>

We are running a very large number of tests, and we are using the log
features heavily for documentation purposes. This therefore becomes a big
problem for us.

Any feedback is appreciated

Original issue reported on code.google.com by espen.al...@gmail.com on 14 Apr 2010 at 12:17

Attachments:

GoogleCodeExporter commented 9 years ago
This is by design to work around the fact that FireFox and other browsers do not
provide CSS-based control over word breaks.  If we don't insert word breaks 
then a
single very long identifier (quite common in test names and stack traces) can 
cause
the report to bloat horizontally to the point it becomes unusable.

Original comment by jeff.br...@gmail.com on 14 Apr 2010 at 6:40

GoogleCodeExporter commented 9 years ago
OK - I understand the reasoning behind that. I still think it is a bug, because 
the
WBR is all over the place. A reasonable approach would be to insert WBR every 
'n'
characters, current code seems to have n<10.

Could you direct me to the code responsible for this feature? Searching for WBR 
in
the code did not yield any results...

Original comment by espen.al...@gmail.com on 22 Apr 2010 at 9:20

GoogleCodeExporter commented 9 years ago
It's in Gallio-Report.common.xsl, there's a comment on Ln 248 that explains why 
they
are there. If you can improve it, a patch would be great :) Thanks.

Original comment by grahamr...@gmail.com on 4 May 2010 at 8:40

GoogleCodeExporter commented 9 years ago
Working as intended, sorry.

Feel free to customize the XSLT and contribute a more lightweight report 
format.  It isn't just the <wbr>'s that bloat the report.  We could do better 
if we simplified the formatting.

Original comment by jeff.br...@gmail.com on 1 Sep 2010 at 7:52