Closed
Bug 793429
Opened 13 years ago
Closed 13 years ago
add test that net/url works with newlines
Categories
(Add-on SDK Graveyard :: General, defect, P2)
Add-on SDK Graveyard
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: evold, Unassigned)
Details
No description provided.
| Reporter | ||
Updated•13 years ago
|
Summary: add test that api-utils/url/io works with newlines characters → add test that api-utils/url/io works with newlines
Priority: -- → P2
Comment 1•13 years ago
|
||
The test was explicitly removed because added unnecessary complexity to the unit test itself, due Windows differences in newline character.
The net/url module itself has nothing to do with newlines – there is no method exposed neither internally used related to newlines – so adding such test is not going to cover anything of the module's code.
I'm -1 to add this test on net/url because it's out of the scope of the module. If there are any modules that use or relies on newlines, than we should add that test there not here.
Summary: add test that api-utils/url/io works with newlines → add test that net/url works with newlines
| Reporter | ||
Updated•13 years ago
|
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•