Skip to content

Conversation

tanyatrayanava
Copy link
Contributor

No description provided.

}
}

async clickOnEditDBByName(databaseName: string): Promise<void>{

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a description on top (I see it didn't happen also on functions above)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

})
('Verify that user can connect to DB and see breadcrumbs at the top of the application in Browser page', async t => {
//Verify that user can see breadcrumbs
await t.expect(browserPage.breadcrumbsContainer.visible).ok('User can see breadcrumbs in Browser page', { timeout: 20000 });

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 line test? couldn't it be covered over lower levels (such as components)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

another verification is added in this test

await t.typeText(cliPage.cliCommandInput, command);
await t.pressKey('enter');
//Verify result
await t.expect(cliPage.cliOutputResponseSuccess.innerText).eql(`"${jsonValue.replace(/"/g, '\\"')}"`, 'The user can see JSON object with escaped quotes');

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix the eslint error

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@tanyatrayanava tanyatrayanava merged commit 005f1fe into main Mar 2, 2022
@tanyatrayanava tanyatrayanava deleted the feature/e2e-regression branch March 2, 2022 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants