[ 
https://issues.apache.org/jira/browse/OFBIZ-13311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18035530#comment-18035530
 ] 

Gaetan Chaboussie commented on OFBIZ-13311:
-------------------------------------------

I suggest the removal of the `useAllMemory()`method in `GenericTestCaseBase`.
This method serves as a time measure for a cache peremption test, but seems 
quite out of date, and very sensitive to the OS and or the hardware the test 
runs on, and this seems incompatible with the idea of unit testing for OFBiz.

Also, is the  GenericTestCaseBase used by projects ? 
It doesn't seem to be used in the framework elsewhere than in the 
UtilCacheTests.

> Rewriting the UtilCacheTests
> ----------------------------
>
>                 Key: OFBIZ-13311
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-13311
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework/base
>    Affects Versions: Upcoming Branch
>            Reporter: Gaetan Chaboussie
>            Priority: Major
>
> The UtilCacheTest class was written 15 years ago, and has barely been updated 
> since.
> It's very low level, so sensitive to modify, and may require attentive review.
> But these tests are quite hard to read, so i offer to, if not rewrite, at 
> least try to explicit make the tests more readable.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to