From a51efc9923c976259b329468510c8e5a0b259820 Mon Sep 17 00:00:00 2001 From: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> Date: Tue, 9 Nov 2021 11:29:01 -0500 Subject: [PATCH] [Uptime] Ping redirects - add retry logic (#117363) (#118017) * add retry logic and focus test * unfocus tests Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> Co-authored-by: Dominique Clarke --- x-pack/test/functional/apps/uptime/ping_redirects.ts | 3 +-- x-pack/test/functional/services/uptime/navigation.ts | 4 +++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/x-pack/test/functional/apps/uptime/ping_redirects.ts b/x-pack/test/functional/apps/uptime/ping_redirects.ts index 748163cb5ec7..03185ac9f146 100644 --- a/x-pack/test/functional/apps/uptime/ping_redirects.ts +++ b/x-pack/test/functional/apps/uptime/ping_redirects.ts @@ -19,8 +19,7 @@ export default ({ getPageObjects, getService }: FtrProviderContext) => { const monitor = () => uptime.monitor; - // FLAKY: https://github.com/elastic/kibana/issues/84992 - describe.skip('Ping redirects', () => { + describe('Ping redirects', () => { const start = '~ 15 minutes ago'; const end = 'now'; diff --git a/x-pack/test/functional/services/uptime/navigation.ts b/x-pack/test/functional/services/uptime/navigation.ts index 51806d1006ab..43d62ef74bf3 100644 --- a/x-pack/test/functional/services/uptime/navigation.ts +++ b/x-pack/test/functional/services/uptime/navigation.ts @@ -61,7 +61,9 @@ export function UptimeNavigationProvider({ getService, getPageObjects }: FtrProv goToMonitor: async (monitorId: string) => { // only go to monitor page if not already there if (!(await testSubjects.exists('uptimeMonitorPage', { timeout: 0 }))) { - await testSubjects.click(`monitor-page-link-${monitorId}`); + return retry.try(async () => { + await testSubjects.click(`monitor-page-link-${monitorId}`); + }); await testSubjects.existOrFail('uptimeMonitorPage', { timeout: 30000, });