[rfc][icedtea-web] Add unit tests for CacheUtil class

Jie Kang jkang at redhat.com
Thu Jan 22 14:56:47 UTC 2015



----- Original Message -----
> On 01/15/2015 05:32 PM, Jie Kang wrote:
> > Hello,
> >
> > The patch adds a set of basic unit tests to CacheUtil to make sure they
> > work as intended. They don't cover many edge cases at the moment and are
> > mainly for developer's to read and gain an understanding of the CacheUtil
> > class quicker. Edge cases will be added in the future.
> >
> > As well, there is one KnownToFail test for CacheUtil.cleanCache : This is
> > due to an implementation error that causes entries marked for deletion to
> > never be deleted in certain circumstances. This will be fixed in a
> > subsequent patch.
> >
> > Okay to push?
> 
> 
> yes.

Hello,

I realized this patch relies on 'Allow cache directory to be changed during runtime' http://mail.openjdk.java.net/pipermail/distro-pkg-dev/2015-January/030291.html

I will wait for that to be resolved before pushing this or submitting a revised version of this for review.


Regards,

> 

-- 

Jie Kang

OpenJDK Team - Software Engineering Intern


More information about the distro-pkg-dev mailing list