[icedtea-web] RFC: Unit tests for JNLPProxySelector

Omair Majid omajid at redhat.com
Thu Sep 19 10:34:09 PDT 2013


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

-- 
PGP Key: 66484681 (http://pgp.mit.edu/)
Fingerprint = F072 555B 0A17 3957 4E95  0056 F286 F14F 6648 4681
-------------- next part --------------
A non-text attachment was scrubbed...
Name: jnlpproxyselector-tests-01.patch
Type: text/x-patch
Size: 21580 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20130919/714a7d95/jnlpproxyselector-tests-01.patch 


More information about the distro-pkg-dev mailing list