[GitHub] metron pull request #857: METRON-1340: Improve e2e tests for metron alerts

2017-12-06 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/857#discussion_r155261159 --- Diff: metron-interface/metron-alerts/src/app/utils/constants.ts --- @@ -38,5 +38,6 @@ export let TREE_SUB_GROUP_SIZE = 5; export let

[GitHub] metron pull request #857: METRON-1340: Improve e2e tests for metron alerts

2017-12-06 Thread iraghumitra
Github user iraghumitra commented on a diff in the pull request: https://github.com/apache/metron/pull/857#discussion_r155219994 --- Diff: metron-interface/metron-alerts/src/app/utils/constants.ts --- @@ -38,5 +38,6 @@ export let TREE_SUB_GROUP_SIZE = 5; export let

[GitHub] metron pull request #857: METRON-1340: Improve e2e tests for metron alerts

2017-12-05 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/857#discussion_r155067123 --- Diff: metron-interface/metron-alerts/src/app/utils/constants.ts --- @@ -38,5 +38,6 @@ export let TREE_SUB_GROUP_SIZE = 5; export let

[GitHub] metron pull request #857: METRON-1340: Improve e2e tests for metron alerts

2017-12-03 Thread iraghumitra
GitHub user iraghumitra opened a pull request: https://github.com/apache/metron/pull/857 METRON-1340: Improve e2e tests for metron alerts ## Contributor Comments This PR aims to improve the existing e2e tests of metro-alerts in the areas mentioned below. The PR doesn't