-
Notifications
You must be signed in to change notification settings - Fork 243
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
test: add Amazon Q /dev e2e tests #5434
Conversation
...-starter/tst-243+/software/aws/toolkits/jetbrains/uitests/featureDevTests/TestCaseScripts.kt
Show resolved
Hide resolved
...-starter/tst-243+/software/aws/toolkits/jetbrains/uitests/featureDevTests/TestCaseScripts.kt
Outdated
Show resolved
Hide resolved
868f90d
to
c10dd0b
Compare
/runUiTests |
All Failed test case Retry one more time |
/runUiTests |
All /dev E2E tests passed. Failed test case Retry one more time |
/runUiTests |
5 similar comments
/runUiTests |
/runUiTests |
/runUiTests |
/runUiTests |
/runUiTests |
...-starter/tst-243+/software/aws/toolkits/jetbrains/uitests/featureDevTests/TestCaseScripts.kt
Outdated
Show resolved
Hide resolved
|
||
// Validate if /dev ends the conversation successfully | ||
await page.waitForSelector('p ::-p-text(Okay, I\'ve ended this chat session. You can open a new tab to chat or start another workflow.)', {timeout: 3000}); | ||
console.log('Success: /dev ends the conversation successfully.'); |
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.
Do we want to catch some of these errors and log them I know the last line does that, but adding it here will provide some more granularity
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.
I think it is not necessary to add error log here as they would mostly just element not found
and can be captured in executePuppeteerScript
.
/runUiTests |
/runUiTests |
1 similar comment
/runUiTests |
/runUiTests |
Types of changes
Description
Add Amazon Q /dev e2e tests
Checklist
License
I confirm that my contribution is made under the terms of the Apache 2.0 license.