[GitHub] metron issue #1085: METRON-1636: Fix broken unit test setup in metron-alerts

2018-07-02 Thread tiborm
Github user tiborm commented on the issue: https://github.com/apache/metron/pull/1085 Reopening to trigger new travis build. ---

[GitHub] metron issue #1085: METRON-1636: Fix broken unit test setup in metron-alerts

2018-07-02 Thread tiborm
Github user tiborm commented on the issue: https://github.com/apache/metron/pull/1085 Conflict resolved. ---

[GitHub] metron issue #1085: METRON-1636: Fix broken unit test setup in metron-alerts

2018-06-29 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/1085 I have not seen that before. Fun. I'll take a look-see. ---

[GitHub] metron issue #1085: METRON-1636: Fix broken unit test setup in metron-alerts

2018-06-29 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/1085 @nickwallen these tests are failing because of: ``` Results : Failed tests: ProfileBuilderBoltTest.testFlushExpiredProfiles:205->getProfileMeasurements:265 outputCollect

[GitHub] metron issue #1085: METRON-1636: Fix broken unit test setup in metron-alerts

2018-06-29 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/1085 It looks like this is a conflict and the unit tests aren't quite running, past that I'm +1 ---

[GitHub] metron issue #1085: METRON-1636: Fix broken unit test setup in metron-alerts

2018-06-29 Thread sardell
Github user sardell commented on the issue: https://github.com/apache/metron/pull/1085 I tested this in full-dev and the unit tests now pass when run. I'm really happy to see this done so we can start to build upon these base tests. +1. ---

[GitHub] metron issue #1085: METRON-1636: Fix broken unit test setup in metron-alerts

2018-06-28 Thread tiborm
Github user tiborm commented on the issue: https://github.com/apache/metron/pull/1085 Fixed the title. Checking why travis check fails... ---

[GitHub] metron issue #1085: Metron 1636

2018-06-27 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/1085 Hey, thanks for the contribution! Can you make the title METRON-1636: Fix broken unit test setup in metron-alerts? Also, if you could mark the boxes in the pull request checklist to let us know t