Thanks Tom, will do so and apply it to the patch. On Dec 4, 2007 3:54 AM, Tom Bostelmann <[EMAIL PROTECTED]> wrote: > most likely that's just an uncaught, ignored exception from: > > > org.mifos.application.accounts.struts.action.TestApplyAdjustmentAction.testAdjustmentForZeroPmnt(TestApplyAdjustmentAction.java:309) > > to fix this, you should add a catch block for this exception and explicitly > ignore it so that it doesn't fall into stdout and confuse other developers. > > > > On Dec 4, 2007 3:05 AM, Dion Dodgen <[EMAIL PROTECTED]> wrote: > > Hi, > > > > I seem to be getting 1 error when running the test suite for revision > > 12211, but all tests seem to be passing. > > > > <system-out><![CDATA[ERROR, org.mifos, MifosExceptionHandler, > > logException , 72, No resource is associated with key > > "exception.accounts.ApplicationException.CannotAdjust"., 04 Dec 2007 > > 01:07:50 > > ERROR, org.mifos, MifosExceptionHandler, logException , 72, > > exception.accounts.ApplicationException.CannotAdjust, 04 Dec 2007 > > 01:07:50 > > org.mifos.application.accounts.exceptions.AccountException: > > exception.accounts.ApplicationException.CannotAdjust > > at > org.mifos.application.accounts.business.AccountBO.adjustLastPayment(AccountBO.java > :370) > > at > org.mifos.application.accounts.struts.action.ApplyAdjustment.applyAdjustment(ApplyAdjustment.java:144) > > at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) > > at sun.reflect.NativeMethodAccessorImpl.invoke > (NativeMethodAccessorImpl.java:39) > > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) > > at java.lang.reflect.Method.invoke(Method.java:597) > > at org.apache.struts.actions.DispatchAction.dispatchMethod > (DispatchAction.java:274) > > at > org.apache.struts.actions.DispatchAction.execute(DispatchAction.java:194) > > at > org.mifos.framework.struts.action.BaseAction.execute(BaseAction.java:72) > > at org.apache.struts.action.RequestProcessor.processActionPerform > (RequestProcessor.java:419) > > at > org.apache.struts.action.RequestProcessor.process(RequestProcessor.java:224) > > at > org.apache.struts.action.ActionServlet.process(ActionServlet.java:1194) > > at org.apache.struts.action.ActionServlet.doPost > (ActionServlet.java:432) > > at > servletunit.struts.MockStrutsTestCase.actionPerform(MockStrutsTestCase.java:290) > > at > org.mifos.application.accounts.struts.action.TestApplyAdjustmentAction.testAdjustmentForZeroPmnt > (TestApplyAdjustmentAction.java:309) > > at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) > > at > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) > > at sun.reflect.DelegatingMethodAccessorImpl.invoke > (DelegatingMethodAccessorImpl.java:25) > > at java.lang.reflect.Method.invoke(Method.java:597) > > at junit.framework.TestCase.runTest(TestCase.java:168) > > at junit.framework.TestCase.runBare(TestCase.java :134) > > at junit.framework.TestResult$1.protect(TestResult.java:110) > > at junit.framework.TestResult.runProtected(TestResult.java:128) > > at junit.framework.TestResult.run(TestResult.java:113) > > at junit.framework.TestCase.run(TestCase.java:124) > > at junit.framework.TestSuite.runTest(TestSuite.java:232) > > at junit.framework.TestSuite.run(TestSuite.java:227) > > at junit.framework.TestSuite.runTest (TestSuite.java:232) > > at junit.framework.TestSuite.run(TestSuite.java:227) > > at junit.framework.TestSuite.runTest(TestSuite.java:232) > > at junit.framework.TestSuite.run(TestSuite.java:227) > > at > org.apache.tools.ant.taskdefs.optional.junit.JUnitTestRunner.run(JUnitTestRunner.java:421) > > at > org.apache.tools.ant.taskdefs.optional.junit.JUnitTestRunner.launch(JUnitTestRunner.java:912) > > at > org.apache.tools.ant.taskdefs.optional.junit.JUnitTestRunner.main > (JUnitTestRunner.java:766) > > ERROR, org.mifos, MifosExceptionHandler, logException , 72, You do not > > have permissions to perform this activity. Contact your system > > administrator to grant you required permissions and try again., 04 Dec > > 2007 01:08:45 > > org.mifos.framework.exceptions.ServiceException: > error.activity.not.allowed > > at > org.mifos.application.accounts.business.service.AccountBusinessService.checkPermissionForStatusChange(AccountBusinessService.java > :332) > > at > org.mifos.application.accounts.struts.action.EditStatusAction.checkPermission(EditStatusAction.java:245) > > at > org.mifos.application.accounts.struts.action.EditStatusAction.update(EditStatusAction.java > :130) > > > > > > Does anybody else get this? > > > > Thanks, > > Dion > > > > > > > > > > > > On Dec 3, 2007 1:57 PM, Tom Bostelmann <[EMAIL PROTECTED] > wrote: > > > Thanks Dion, apologies for taking a while before getting back to you on > > > this. Almost there :) > > > > > > If you're adding a new test, please add it to a package test suite and > not > > > the main ApplicationTestSuite. If you end up having to create a new > package > > > test suite, then add that test suite to the parent package's test suite. > > > > > > Also as a side-note to other developers, make sure that all tests can > be > > > executed independently, they should not have a dependency on another > test. > > > Dion, your tests look fine here. I'm just adding this as a reminder to > the > > > other developers. > > > > > > > > > > > > On Nov 25, 2007 10:56 PM, Dion Dodgen <[EMAIL PROTECTED]> wrote: > > > > Hi Tom, > > > > > > > > Hope you had a good weekend :) I have integrated all resources into > > > > the single patch file. > > > > > > > > > > > > > > > > > > > > > > > > On 11/22/07, Tom Bostelmann < [EMAIL PROTECTED]> wrote: > > > > > Dion, the patch needs contain all of the contents of the patch in > one > > > file. > > > > > There are a couple files, namely 'OffsetTestSuite.java' and > > > > > 'TestOffsetAccountBO.java ' that should be incorporated in the > patch. > > > > > > > > > > Can you fix this and then submit it again? It'd make my life a > little > > > > > easier ;) > > > > > > > > > > > > > > > On Nov 17, 2007 11:17 PM, Dion Dodgen < [EMAIL PROTECTED] > > > > wrote: > > > > > > Great thanks Tom. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Nov 16, 2007 6:07 PM, Tom Bostelmann < [EMAIL PROTECTED]> > > > wrote: > > > > > > > Thanks Dion! I added it to the patch queue - probably going to > be > > > next > > > > > week > > > > > > > before I get back to you, though. Here's the queue wiki: > > > > > > > > > > > > > > http://mifos.org/developers/wiki/MifosPatchQueue > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Nov 14, 2007 4:58 AM, Dion Dodgen < > [EMAIL PROTECTED]> > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi Tom, all, > > > > > > > > > > > > > > > > Sorry for the delay, hope you well. > > > > > > > > I have reworked the initial patch. All tests pass using the > > > correct > > > > > > > > time zone and db use_lower_case_table_names=1. > > > > > > > > I would like to patch this dependency as soon as offsetting > > > reaches > > > > > > > maturity. > > > > > > > > > > > > > > > > This patch covers all the persistence needs for the Offsetting > > > module. > > > > > > > > > > > > > > > > I am now on suse 10.3 :) > > > > > > > > > > > > > > > > Thanks, > > > > > > > > Dion > > > > > > > > > > > > > > > > > > > > > > > > > ------------------------------------------------------------------------- > > > > > > > > This SF.net email is sponsored by: Splunk Inc. > > > > > > > > Still grepping through log files to find problems? Stop. > > > > > > > > Now Search log events and configuration files using AJAX and a > > > > > browser. > > > > > > > > Download your FREE copy of Splunk now >> > http://get.splunk.com/ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ------------------------------------------------------------------------- > > > > > > > This SF.net email is sponsored by: Microsoft > > > > > > > Defy all challenges. Microsoft(R) Visual Studio 2005. > > > > > > > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ > > > > > > > > > > > > > > > > > > > > > > > > > > > > ------------------------------------------------------------------------- > > > > > > This SF.net email is sponsored by: Microsoft > > > > > > Defy all challenges. Microsoft(R) Visual Studio 2005. > > > > > > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ > > > > > > > > > > > > > > > > > > > > > > > > > ------------------------------------------------------------------------- > > > > > This SF.net email is sponsored by: Microsoft > > > > > Defy all challenges. Microsoft(R) Visual Studio 2005. > > > > > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ > > > > > > > > > > > > > > ------------------------------------------------------------------------- > > > > This SF.net email is sponsored by: Microsoft > > > > Defy all challenges. Microsoft(R) Visual Studio 2005. > > > > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ > > > > > > > > > > > > > > ------------------------------------------------------------------------- > > > SF.Net email is sponsored by: The Future of Linux Business White Paper > > > from Novell. From the desktop to the data center, Linux is going > > > mainstream. Let it simplify your IT future. > > > http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4 > > > > > > > ------------------------------------------------------------------------- > > SF.Net email is sponsored by: The Future of Linux Business White Paper > > from Novell. From the desktop to the data center, Linux is going > > mainstream. Let it simplify your IT future. > > http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4 > > > > > ------------------------------------------------------------------------- > SF.Net email is sponsored by: The Future of Linux Business White Paper > from Novell. From the desktop to the data center, Linux is going > mainstream. Let it simplify your IT future. > http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4 >
------------------------------------------------------------------------- SF.Net email is sponsored by: The Future of Linux Business White Paper from Novell. From the desktop to the data center, Linux is going mainstream. Let it simplify your IT future. http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4
