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)

defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: evold, Unassigned)

Details

No description provided.
Summary: add test that api-utils/url/io works with newlines characters → add test that api-utils/url/io works with newlines
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
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.