Skip to content

Commit

Permalink
PMM-7 try fix for remote instances
Browse files Browse the repository at this point in the history
  • Loading branch information
yurkovychv committed Nov 15, 2024
1 parent 3433d50 commit e260301
Showing 1 changed file with 1 addition and 23 deletions.
24 changes: 1 addition & 23 deletions tests/pages/remoteInstancesPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -529,10 +529,9 @@ module.exports = {

async clickAddInstanceAndWaitForSuccess() {
I.waitForVisible(this.fields.addService, 30);
I.wait(1);

await I.usePlaywrightTo('click Add service and wait for response', async ({ page }) => {
const [response] = await Promise.all([
await Promise.all([
page.waitForResponse(
async (response) => {
if (response.url().includes('v1/management/services')) {
Expand All @@ -550,27 +549,6 @@ module.exports = {
page.click(this.fields.addService),
]);
});

// eslint-disable-next-line no-undef
// await retryTo((tryNum) => {
// I.click(this.fields.addService);
//
// I.waitForResponse(
// (response) => {
// if (response.url().includes('v1/management/services')) {
// if (response.status() === 200) {
// return true;
// }
//
// throw new Error(`Expected status 200 but received ${response.status()}`);
// }
//
// // Continue waiting if this isn't the target request
// return false;
// },
// 5,
// );
// }, 5);
},

openAddAzure() {
Expand Down

0 comments on commit e260301

Please sign in to comment.