Skip to content

Commit 24fb94a

Browse files
committed
Fix tests
1 parent 49a98ad commit 24fb94a

4 files changed

Lines changed: 17 additions & 4 deletions

File tree

x-pack/solutions/security/packages/kbn-scout-security/src/playwright/fixtures/test/page_objects/detections_attack_discovery.ts

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ export class DetectionsAttackDiscoveryPage {
5353
public attacksPageAssigneeFilter: Locator;
5454
public attacksPageConnectorFilter: Locator;
5555
public attacksKpisSection: Locator;
56+
public kpisSectionToggleButton: Locator;
5657
public attacksSummaryView: Locator;
5758
public attacksListPanel: Locator;
5859
public attacksVolumePanel: Locator;
@@ -86,6 +87,9 @@ export class DetectionsAttackDiscoveryPage {
8687
ATTACKS_PAGE_CONNECTOR_FILTER_TEST_ID
8788
);
8889
this.attacksKpisSection = this.page.testSubj.locator(ATTACKS_KPIS_SECTION_TEST_ID);
90+
this.kpisSectionToggleButton = this.attacksKpisSection.locator(
91+
'[data-test-subj="query-toggle-header"]'
92+
);
8993
this.attacksSummaryView = this.page.testSubj.locator(ATTACKS_SUMMARY_VIEW_TEST_ID);
9094
this.attacksListPanel = this.page.testSubj.locator(ATTACKS_LIST_PANEL_TEST_ID);
9195
this.attacksVolumePanel = this.page.testSubj.locator(ATTACKS_VOLUME_PANEL_TEST_ID);
@@ -167,6 +171,13 @@ export class DetectionsAttackDiscoveryPage {
167171
await this.detectionsNavItem.click();
168172
}
169173

174+
async collapseKpisSection() {
175+
if (await this.attacksSummaryView.isVisible()) {
176+
await this.kpisSectionToggleButton.click();
177+
await this.attacksSummaryView.waitFor({ state: 'hidden' });
178+
}
179+
}
180+
170181
async openScheduleFlyout() {
171182
await this.scheduleButton.click();
172183
await this.settingsFlyout.waitFor({ state: 'visible' });
@@ -180,7 +191,6 @@ export class DetectionsAttackDiscoveryPage {
180191
throw new Error('No attack details expand button found');
181192
}
182193

183-
await firstExpandAttackButton.scrollIntoViewIfNeeded();
184194
await firstExpandAttackButton.click();
185195
await this.attackDetailsFlyoutBody.waitFor({ state: 'visible' });
186196
}
@@ -192,7 +202,6 @@ export class DetectionsAttackDiscoveryPage {
192202
throw new Error('No schedule button found');
193203
}
194204

195-
await firstScheduleButton.scrollIntoViewIfNeeded();
196205
await firstScheduleButton.click();
197206
await this.scheduleDetailsFlyout.waitFor({ state: 'visible' });
198207
}

x-pack/solutions/security/plugins/security_solution/test/scout_security_attacks_alignment/ui/parallel_tests/detections/attacks/attack_flyout.spec.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ spaceTest.describe(
2929
await browserAuth.loginAsAdmin();
3030

3131
await detectionsAttackDiscoveryPage.navigateToAttacksPage();
32+
await detectionsAttackDiscoveryPage.collapseKpisSection();
3233
await expect(detectionsAttackDiscoveryPage.attacksTableSection).toBeAttached();
3334
await detectionsAttackDiscoveryPage.attacksTableSection.scrollIntoViewIfNeeded();
3435
await expect(detectionsAttackDiscoveryPage.attacksTableSection).toBeVisible();

x-pack/solutions/security/plugins/security_solution/test/scout_security_attacks_alignment/ui/parallel_tests/detections/attacks/attacks_page_smoke.spec.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,8 @@ spaceTest.describe(
1515
'Attacks page smoke',
1616
{ tag: [...tags.stateful.classic, ...tags.serverless.security.complete] },
1717
() => {
18-
spaceTest.beforeAll(async ({ apiServices }) => {
18+
spaceTest.beforeAll(async ({ apiServices, scoutSpace }) => {
19+
await scoutSpace.savedObjects.cleanStandardList();
1920
await apiServices.attackDiscovery.seedAttackData();
2021
});
2122

x-pack/solutions/security/plugins/security_solution/test/scout_security_attacks_alignment/ui/parallel_tests/detections/attacks/attacks_schedule.spec.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,8 @@ spaceTest.describe(
1515
'Attacks schedule flyout',
1616
{ tag: [...tags.stateful.classic, ...tags.serverless.security.complete] },
1717
() => {
18-
spaceTest.beforeAll(async ({ apiServices }) => {
18+
spaceTest.beforeAll(async ({ apiServices, scoutSpace }) => {
19+
await scoutSpace.savedObjects.cleanStandardList();
1920
await apiServices.attackDiscovery.seedAttackData();
2021
await apiServices.attackDiscovery.seedAttackSchedule();
2122
});
@@ -53,6 +54,7 @@ spaceTest.describe(
5354
const { detectionsAttackDiscoveryPage } = pageObjects;
5455

5556
await detectionsAttackDiscoveryPage.navigateToAttacksPage();
57+
await detectionsAttackDiscoveryPage.collapseKpisSection();
5658

5759
await expect(detectionsAttackDiscoveryPage.attacksTableSection).toBeVisible();
5860
await expect(detectionsAttackDiscoveryPage.tableScheduleButtons).toHaveCount(1);

0 commit comments

Comments
 (0)