[icedtea-web] RFC: Unit tests for JNLPProxySelector

Jiri Vanek jvanek at redhat.com
Mon Sep 23 08:58:49 PDT 2013


On 09/19/2013 07:34 PM, Omair Majid wrote:
> Hi,
>
> The attached patch adds a bunch of unit tests for JNLPProxySelector.
> Most functionality aside from making use of a PAC file should be covered
> by tests now.
>
> I tried to avoid making any functional changes, but it felt wrong to let
> this one mistake go by:
> - String host = uri.getSchemeSpecificPart().split(":")[0];
> + String host = uri.getHost();
>
> Given the URI of "socket://example.org", the first line evaluates to
> "//example.org", while the second one (correctly) evaluates to
> "example.org".
>
> Okay to push?
>
> Thanks,
> Omair
>
looks ok to me.


More information about the distro-pkg-dev mailing list