[
https://issues.apache.org/jira/browse/METRON-1489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16456777#comment-16456777
]
ASF GitHub Bot commented on METRON-1489:
----------------------------------------
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/metron/pull/1004#discussion_r184750091
--- Diff:
metron-interface/metron-alerts/e2e/alerts-list/alert-status/alerts-list-status.e2e-spec.ts
---
@@ -44,84 +45,84 @@ describe('metron-alerts alert status', function() {
jasmine.addMatchers(customMatchers);
});
- it('should change alert status for multiple alerts to OPEN', () => {
- page.navigateTo();
- page.toggleAlertInList(0);
- page.toggleAlertInList(1);
- page.toggleAlertInList(2);
- page.clickActionDropdownOption('Open');
- expect(page.getAlertStatus(0, 'NEW')).toEqual('OPEN');
- expect(page.getAlertStatus(1, 'NEW')).toEqual('OPEN');
- expect(page.getAlertStatus(2, 'NEW')).toEqual('OPEN');
+ it('should change alert status for multiple alerts to OPEN', async
function() : Promise<any> {
+ await page.navigateTo();
+ await page.toggleAlertInList(0);
+ await page.toggleAlertInList(1);
+ await page.toggleAlertInList(2);
+ await page.clickActionDropdownOption('Open');
+ // expect(await page.getAlertStatus(0, 'NEW')).toEqual('OPEN');
+ // expect(await page.getAlertStatus(1, 'NEW')).toEqual('OPEN');
+ // expect(await page.getAlertStatus(2, 'NEW')).toEqual('OPEN');
});
- it('should change alert status for multiple alerts to DISMISS', () => {
- page.toggleAlertInList(3);
- page.toggleAlertInList(4);
- page.toggleAlertInList(5);
- page.clickActionDropdownOption('Dismiss');
- expect(page.getAlertStatus(3, 'NEW')).toEqual('DISMISS');
- expect(page.getAlertStatus(4, 'NEW')).toEqual('DISMISS');
- expect(page.getAlertStatus(5, 'NEW')).toEqual('DISMISS');
+ it('should change alert status for multiple alerts to DISMISS', async
function() : Promise<any> {
+ await page.toggleAlertInList(3);
+ await page.toggleAlertInList(4);
+ await page.toggleAlertInList(5);
+ await page.clickActionDropdownOption('Dismiss');
+ // expect(await page.getAlertStatus(3, 'NEW')).toEqual('DISMISS');
+ // expect(await page.getAlertStatus(4, 'NEW')).toEqual('DISMISS');
+ // expect(await page.getAlertStatus(5, 'NEW')).toEqual('DISMISS');
});
- it('should change alert status for multiple alerts to ESCALATE', () => {
- page.toggleAlertInList(6);
- page.toggleAlertInList(7);
- page.toggleAlertInList(8);
- page.clickActionDropdownOption('Escalate');
- expect(page.getAlertStatus(6, 'NEW')).toEqual('ESCALATE');
- expect(page.getAlertStatus(7, 'NEW')).toEqual('ESCALATE');
- expect(page.getAlertStatus(8, 'NEW')).toEqual('ESCALATE');
+ it('should change alert status for multiple alerts to ESCALATE', async
function() : Promise<any> {
+ await page.toggleAlertInList(6);
+ await page.toggleAlertInList(7);
+ await page.toggleAlertInList(8);
+ await page.clickActionDropdownOption('Escalate');
+ // expect(await page.getAlertStatus(6, 'NEW')).toEqual('ESCALATE');
--- End diff --
Do we need these commented out bits?
> Retrofit UI tests to run reliably during nightly QE runs
> --------------------------------------------------------
>
> Key: METRON-1489
> URL: https://issues.apache.org/jira/browse/METRON-1489
> Project: Metron
> Issue Type: Improvement
> Reporter: Daniel Toth
> Assignee: Daniel Toth
> Priority: Major
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
