activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From xiaolongzuo <...@git.apache.org>
Subject [GitHub] activemq pull request: Fixed bug for cron test.
Date Sun, 25 Jan 2015 15:37:10 GMT
GitHub user xiaolongzuo opened a pull request:

    https://github.com/apache/activemq/pull/58

    Fixed bug for cron test.

    The test method(org.apache.activemq.broker.scheduler.CronParserTest.testgetNextTimeMinutes)
is failure on my machine,because the result of calender.setTimeInMillis(0) is 1970-01-01 08:30:00
not 1970-01-01 08:00:00.
    The result of expresssion(next - current) should be 60*40*1000 not 60*10*1000. 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/xiaolongzuo/activemq trunk

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/activemq/pull/58.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #58
    
----
commit c28b6ba415f222a2cd1762d48c6b6d40b127000d
Author: Xiaolong Zuo <150349407@qq.com>
Date:   2015-01-24T14:37:50Z

    Merge pull request #1 from apache/trunk
    
    merge

commit 0842ce6a7e8463a5e0b2bf194baf487c7cd148af
Author: Xiaolong Zuo <150349407@qq.com>
Date:   2015-01-25T08:02:19Z

    Fixed the test method named testgetNextTimeMinutes.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message