Re: Failing tests due to bad commits

2017-06-08 Thread Jacques Le Roux
OK the 2nd test issue is fixed with r1798086 where I reverted my changes in ListFinder.java But as noted in the commit comment I have to check all is correct with current code. Is the created EntityListIterator always closed (in all cases) is still obscure to me. Maybe it's not a big deal b

Re: Failing tests due to bad commits

2017-06-08 Thread Jacques Le Roux
Hi Taher, This is my bad, let me explain why I missed it. I use to rely on Buildbot. Last trunk message was OK https://s.apache.org/Uz5M But I completely forgot that we still need to update our Buildbot config. And actually we still rely on https://ci.apache.org/builders/ofbiz-trunk-framework-

Failing tests due to bad commits

2017-06-08 Thread Taher Alkhateeb
Hey Folks, I have 2 failing tests in the system: - party-tests.testClearAddressMatchMap (Failure) - testPerformFind (Error) So whoever committed this code please look into fixing it. Also let's agree that at a minimum tests should pass with ./gradlew cleanAll loadDefault testIntegration before co