-
Notifications
You must be signed in to change notification settings - Fork 406
tests for improve eula screen feature #929
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
async getAnalyticsSwitcherValue(): Promise<string> { | ||
return await this.switchAnalyticsOption.getAttribute('aria-checked'); | ||
} | ||
|
||
async getNotificationsSwitcherValue(): Promise<string> { | ||
return await this.switchNovitifationsOption.getAttribute('aria-checked'); | ||
} | ||
|
||
async getEulaSwitcherValue(): Promise<string> { | ||
return await this.switchEulaOption.getAttribute('aria-checked'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add description, please
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
} | ||
} | ||
|
||
async getRecommendedSwitcherValue(): Promise<string> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
await t.click(addRedisDatabasePage.addDatabaseButton); | ||
await t.expect(addRedisDatabasePage.addDatabaseManually.exists).ok('User can add a database'); | ||
}); | ||
test('Verify that user should accept User Agreements to continue working with the application, the Welcome page is opened after user agrees, the encryption enabled by default and specific message', async t => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test name is very long
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
await t.click(addRedisDatabasePage.addDatabaseButton); | ||
await t.expect(addRedisDatabasePage.addDatabaseManually.exists).ok('User can add a database'); | ||
}); | ||
test('Verify that user should accept User Agreements to continue working with the application, the Welcome page is opened after user agrees, the encryption enabled by default and specific message', async t => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ViktarStarastsenka - This isn't e2e, it has to be covered in JEST as many tests cases
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not my test, but ok, I'll update it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
await t.expect(userAgreementPage.pluginSectionWithText.visible).ok('Plugin text is displayed'); | ||
//Verify that text that is displayed in window is 'While adding new visualization plugins, use files only from trusted authors to avoid automatic execution of malicious code.' | ||
const pluginText = userAgreementPage.pluginSectionWithText.innerText; | ||
await t.expect(pluginText).eql('To avoid automatic execution of malicious code, when adding new Workbench plugins, use files from trusted authors only.'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add error message
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
No description provided.