Keith, I would recommend sending us a patch with your changes. This would be a good way of getting more acquainted with the development process.
On Nov 14, 2007 8:27 AM, Keith Pierce <[EMAIL PROTECTED]> wrote: > Saurabh, I just assigned issue 1491 (timezone problems with unit tests) to > myself. I've already found some easily-fixed bugs. Let me know how you'd > like to proceed: take over the issue, collaborate on it, or leave it to me > so you can work on higher-priority issues. I am pretty new to MifOS, and > this looked like a good place for me to start diving in. > > Keith Pierce > > > On Nov 14, 2007 3:27 AM, Saurabh Kumar <[EMAIL PROTECTED]> > wrote: > > > > > > > Hi Tom, > > > > > > > > I tried by changing the timezone to Pacific and the testsuite passed > > successfully. > > > > > > > > Please find the patch for moratorium module attached with the mail. > > > > > > > > As far as the issue regarding failing of testsuite because of different > > timezones is concerned, I am looking into that. Will update you as soon > > as I crack the problem. > > > > > > > > *Thanks & Regards,* > > > > *Saurabh Kumar* • Developer • SunGard • Offshore Services • Divyasree > > Chambers, Langford Road, Bangalore 560025 India > > Tel +91-80-2222-0501 • Mobile +91-9886945575 • Fax +91-80-2222-0511 • > > www.sungard.com > > > > -----Original Message----- > > *From:* [EMAIL PROTECTED] [mailto: > > [EMAIL PROTECTED] *On Behalf Of *Tom > > Bostelmann > > *Sent:* Tuesday, November 13, 2007 4:53 PM > > *To:* Developer > > *Subject:* Re: [Mifos-developer] Patch for moratorium module.... > > > > > > > > Another good option would be to fix the bug and submit a fix with the > > code :) > > > > You can add this to your patch of you would like or you can submit the > > fix as a separate patch. > > > > Aliya, is there an issue number associated with this? > > > > On Nov 13, 2007 8:50 AM, Aliya Walji <[EMAIL PROTECTED]> > > wrote: > > > > Hi Saurabh, > > > > > > > > Currently, if I'm not mistaken, we have a bug where the test suite fails > > when the time zone is not set to Pacific timezone. This may be what is > > causing the issue for you. Can you try changing your system timezone to > > Pacific Time and running the test suite again? Hopefully this will resolve > > your issue. > > > > > > > > Thanks, > > > > > > > > Aliya > > > > > > ------------------------------ > > > > *From:* [EMAIL PROTECTED] [mailto: > > [EMAIL PROTECTED] *On Behalf Of *Saurabh > > Kumar > > *Sent:* Monday, November 12, 2007 11:00 PM > > > > > > *To:* Developer > > *Subject:* Re: [Mifos-developer] Patch for moratorium module.... > > > > > > > > > > > > Hi Aliya, > > > > > > > > I have checked the code and I feel there might me some configuration > > issue with date. > > > > > > > > There is some problem with setting of FiscalWeekStartDay -- > > > > > > > > <testcase classname=" > > org.mifos.framework.components.configuration.business.TestConfiguration" > > name="testAreaOfficeConfiguration" time="0.0"> > > > > <failure message ="expected:<2> but was:<0>" type=" > > junit.framework.AssertionFailedError"> > > junit.framework.AssertionFailedError: expected:<2> but > > was:<0> > > > > at > > org.mifos.framework.components.configuration.business.TestConfiguration.assertForMeetingConfig > > (TestConfiguration.java:86) > > > > at > > org.mifos.framework.components.configuration.business.TestConfiguration.testAreaOfficeConfiguration > > (TestConfiguration.java:52) > > > > </failure> > > > > </testcase> > > > > > > > > In above failure of assertion it is trying to get FiscalWeekStartDay, > > whose value is 0 instead of two. > > > > > > > > <testcase classname=" > > org.mifos.application.accounts.savings.business.TestSavingsBO " > > name="testCalculateInterest_IntCalcFreqTenDays_minbal" time="0.888"> > > > > <failure message ="expected:< Sat Apr 01 00: 00:00 IST 2006> > > but was:< Mon Apr 03 00: 00:00 IST 2006>" type=" > > junit.framework.AssertionFailedError"> > > junit.framework.AssertionFailedError: expected:<Sat Apr 01 00 :00:00IST > > 2006> but was:<Mon > > Apr 03 00 :00:00 IST 2006> > > > > at > > org.mifos.application.accounts.savings.business.TestSavingsBO.testCalculateInterest_IntCalcFreqTenDays_minbal > > (TestSavingsBO.java:2390) > > > > </failure> > > > > </testcase> > > > > > > > > Any idea, what might be the problem?? > > > > > > > > For reference I m attaching the test suite report with mail. > > > > > > > > *Thanks & Regards,* > > > > *Saurabh Kumar* • Developer • SunGard • Offshore Services • Divyasree > > Chambers, Langford Road, Bangalore 560025 India > > Tel +91-80-2222-0501 • Mobile +91-9886945575 • Fax +91-80-2222-0511 • > > www.sungard.com > > > > -----Original Message----- > > *From:* [EMAIL PROTECTED] [mailto: > > [EMAIL PROTECTED] *On Behalf Of *Aliya > > Walji > > *Sent:* Tuesday, November 13, 2007 12:17 AM > > *To:* Developer > > *Subject:* Re: [Mifos-developer] Patch for moratorium module.... > > > > > > > > Hi Saurabh, > > > > > > > > When I look at the build server, revision 2183 does not seem to have any > > build issues or test failures. You may want to investigate things on your > > side again if you are still experiencing test failures in the latest code. > > > > > > > > Aliya > > > > > > ------------------------------ > > > > *From:* [EMAIL PROTECTED] [mailto: > > [EMAIL PROTECTED] *On Behalf Of *Saurabh > > Kumar > > *Sent:* Monday, November 12, 2007 2:48 AM > > *To:* Developer > > *Subject:* Re: [Mifos-developer] Patch for moratorium module.... > > > > > > > > > > > > Hi Tom, > > > > > > > > Thanks for reviewing the patch. > > > > > > > > I m sending new patch, it contains code which is integrated with > > revision 12183. > > > > > > > > Do let me know if there are still any issues. > > > > > > > > Please note that the test suite in revision 12183 is getting failed > > (without our code.) After integrating our code, I have checked thoroughly > > and there are no issues because of our code. > > > > *Thanks & Regards,* > > > > *Saurabh Kumar* • Developer • SunGard • Offshore Services • Divyasree > > Chambers, Langford Road, Bangalore 560025 India > > Tel +91-80-2222-0501 • Mobile +91-9886945575 • Fax +91-80-2222-0511 • > > www.sungard.com > > > > > > > > > > > > ------------------------------------------------------------------------- > > 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: 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: 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: 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/
