GitHub user abellina reopened a pull request:
https://github.com/apache/storm/pull/1114
STORM-1255: port storm_utils.clj to java and split Time tests into its
own test file
Added a few extra unit tests. It is hard to find what needs to be tested
most, so if you have a suggestion, I am happy to add more tests for Utils
(perhaps a different pr).
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/abellina/storm
STORM-1255_port_utils_test_to_java
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/1114.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 #1114
----
commit a2a656ed3fdbf76fddb730bced5bfe7f2b18df72
Author: Alessandro Bellina <[email protected]>
Date: 2016-02-15T19:30:10Z
STORM-1255: port storm_utils.clj to java and split Time tests into its
own test file
commit 20851f8b2d47bc74fa8ac36c78c43e038d56ed81
Author: Alessandro Bellina <[email protected]>
Date: 2016-02-17T18:26:02Z
STORM-1255: remove extra import, adjust formatting, shorten function names
by grouping assertions
commit 3fe11ecc652790684010b5fdd36c53844e89ce42
Author: Alessandro Bellina <[email protected]>
Date: 2016-02-17T18:31:58Z
STORM-1255: combine two tests to make things clearer
commit a8edd512c903d44bf26a57cace8f848cc4660738
Author: Alessandro Bellina <[email protected]>
Date: 2016-02-17T18:36:32Z
STORM-1255: fix spacing in test
----
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
