Github user merrimanr commented on a diff in the pull request:

    https://github.com/apache/metron/pull/1095#discussion_r201060283
  
    --- Diff: 
metron-interface/metron-alerts/e2e/alerts-list/meta-alerts/meta-alert.e2e-spec.ts
 ---
    @@ -242,7 +244,8 @@ describe('Test spec for meta alerts workflow', 
function() {
         expect(guidValues).toEqual(alertsInMetaAlerts.sort());
         await tablePage.removeAlert(5);
         await treePage.clickYesForConfirmation();
    -    expect(await tablePage.getCellValue(0, 2, '(20')).toContain('(19)', 
'alert count should be decremented');
    +    // expect(await tablePage.getCellValue(0, 2, '(20')).toContain('(19)', 
'alert count should be decremented');
    --- End diff --
    
    should this be removed?


---

Reply via email to