There used to be a unit test in the closed workspace for this case that was removed when the fix for 6500133 was introduced.

That test and its certificate will be restored.

Thanks,
Jason

On 01/14/2013 02:18 PM, Sean Mullan wrote:
On 01/14/2013 05:00 PM, Bernd Eckenfels wrote:
Am 14.01.2013, 22:52 Uhr, schrieb Sean Mullan <[email protected]>:

Looks good to me.

The patch removes all unit tests for this class it seems. There should be
one parseable URL and if you intent to reject illegal content you can use
the test vectors to test for the expected exception.

Good point. Jason, can you keep the test but readjust to test for an
expected exception?

Thanks,
Sean

Reply via email to