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
