If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

mozTXTToHTMLConv: Tracking and testcases

RESOLVED INCOMPLETE

Status

()

Core
Networking
RESOLVED INCOMPLETE
16 years ago
2 years ago

People

(Reporter: BenB, Unassigned)

Tracking

(Depends on: 6 bugs, {meta})

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

16 years ago
I'll attach some of my testcases for the mozTXTToHTMLConv class here, so other
developers / QA can test it to avoid regressions.

This bug may also track bugs in / for that class.
(Reporter)

Comment 1

16 years ago
Created attachment 62739 [details]
Testcases: converter

Both this attachment and the next one are "mbox" mailboxes. Mozilla can load
them natively as folders, if you place them in the right dir. Same with the
UW-IMAP server.

This is my complete testcase folder for that converter class. It contains a set
of messages with a wild collection of testcases.
Some of the stuff there is supposed to work, some is supposed not to work, some
happenes not to work. Unfortunately, there is no hint at which is which, so
please just compare the results of a build with your changes with the results
of a build which is known to work. Make sure that both are identical unless the
old one is *obviously* wrong (e.g. <mailto:&$&%> being linked). Please compare
carefully: E.g.
- in <http://www.mozilla.org>, the brackets are part of the link (but they
  are not in <www.mozilla.org>)
- "hello :-), how are you?" should trigger a graphical smily, while
  "hello :-),how are you?" and "hello :-)| how are you?" should not.- make sure
that the converter doesn't "eat" any chars, i.e. that the comma in
  the example above is retained.
BTW: The "(no subject)" smilies don't work, because some of the whitespace are
not spaces (char 32), IIRC.
(Reporter)

Comment 2

16 years ago
Created attachment 62740 [details]
Testcases: I18N

I just saw that you can view the mboxes right in the browser by clicking on the
attachment. I don't think that's garanteed to work correctly, though. But nice
for a quick check...

These are some I18N testcases which I got from Rich, who got it from the i18n
group. I added a few of my own ones. There are /no/ special (I18N) testcases
for the recognition features of the converter. I attach it here, so you can
test the i18n-safetiness of the converter - all the text passes through it, it
is potentioanlly modified and should detect URLs moderately well even in
non-latin text (but using the lating rules, e.g. spaces around them).
(Reporter)

Comment 3

16 years ago
> I just saw that you can view the mboxes right in the browser by clicking on the
> attachment. I don't think that's garanteed to work correctly, though.

In fact, this method breaks for the I18N testcases, because it is regarded as
one message and thus the charset is not sat correctly.
(Reporter)

Updated

16 years ago
Depends on: 115160
(Reporter)

Updated

16 years ago
Keywords: meta

Comment 4

16 years ago
Afte thrashing around for a while trying to figure out if I've broken anything,
I've found out that converting html to plain text is broken in mozilla, and has
nothing to do with my changes. If you use the html compose window to make some
text bold, and send it as plain text, it can get sent incorrectly, or displayed
incorrectly (not sure which - if you make " test " bold, it will get sent and
displayed as "* test *", which is broken. Perhaps the struct code doesn't handle
this case correctly, because of the leading and trailing spaces. Is there a bug
on this? If that's the bug, it seems like this code is pretty seriously flawed.
(Reporter)

Comment 5

16 years ago
Filed bug 117009.
Depends on: 117009

Updated

16 years ago
Depends on: 106028

Updated

16 years ago
Depends on: 128138

Updated

16 years ago
Depends on: 68044

Updated

16 years ago
No longer depends on: 68044

Updated

16 years ago
Depends on: 147458
(Reporter)

Updated

16 years ago
Depends on: 20212, 23327, 39058, 76732, 126701
(Reporter)

Updated

16 years ago
Depends on: 140928
Depends on: 437495

Updated

9 years ago
QA Contact: benc → networking
Depends on: 540606
Depends on: 548982
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.