[rfc][icedtea-web] fixed
Jiri Vanek
jvanek at redhat.com
Thu Dec 19 06:19:43 PST 2013
Those two tests stared to fail recently, and it appeared they wer enot testing what they were
pretending to test.
They were pretending to test /tmp/cache_file, but in fact were testing the real cache file.
This patch is fixing it, by exposing a bit two variables with cached
JNLPRuntime.getConfiguration().getProperty(DeploymentConfiguration.KEY_USER_CACHE_DIR); value.
As one of the cached variables is opened properties file, then I'm not for fixing it better == do
not cache the value.
Also the test is now cleaning a biut after itself.
J.
and of course the unlucky
@@ -195,8 +195,6 @@
}
private String getIdForCacheFolder(String folder) {
- System.out.println(folder);
- System.out.println(cacheDir);
int len = cacheDir.length();
int index = folder.indexOf(File.separatorChar, len + 1);
return folder.substring(len + 1, index);
so not belongs here as those two lines from previous "push" it will not be pusheed in final push.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fixedLruCacheUnitTest.diff
Type: text/x-patch
Size: 6205 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20131219/97fa06fe/fixedLruCacheUnitTest.diff
More information about the distro-pkg-dev
mailing list