From aca34ca81f2481216dce1597fc9d01a56f48d7d3 Mon Sep 17 00:00:00 2001 From: Matt Kime Date: Mon, 22 Apr 2024 12:35:22 -0500 Subject: [PATCH] place retry logic into setFieldType fn --- test/functional/page_objects/settings_page.ts | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/test/functional/page_objects/settings_page.ts b/test/functional/page_objects/settings_page.ts index 822abbc0ba008..e3e68613d8873 100644 --- a/test/functional/page_objects/settings_page.ts +++ b/test/functional/page_objects/settings_page.ts @@ -787,12 +787,7 @@ export class SettingsPageObject extends FtrService { ) { await this.clickAddField(); await this.setFieldName(name); - - await this.retry.try(async () => { - await this.setFieldType('Composite'); - const comboBox = await this.testSubjects.find('typeField'); - expect(await this.comboBox.isOptionSelected(comboBox, 'Composite')).to.be(true); - }); + await this.setFieldType('Composite'); await this.setCompositeScript(script); if (subfieldCount > 0) { await this.testSubjects.find(`typeField_${subfieldCount - 1}`); @@ -879,8 +874,13 @@ export class SettingsPageObject extends FtrService { } async setFieldType(type: string) { + const typeFieldDataTestSubj = 'typeField'; this.log.debug('set type = ' + type); - await this.comboBox.set('typeField', type); + await this.retry.try(async () => { + await this.comboBox.set(typeFieldDataTestSubj, type); + const comboBox = await this.testSubjects.find(typeFieldDataTestSubj); + expect(await this.comboBox.isOptionSelected(comboBox, type)).to.be(true); + }); } async setFieldScript(script: string) {