test(qdoc): Add ui tests for qdoc create readme flow #5479
Conversation
|
/runUiTests |
| assertTrue(result.contains("Test Successful")) | ||
| assertFalse(result.contains("Error: Test Failed")) | ||
| if (result.contains("Error: Test Failed")) { | ||
| println("result: $result") |
There was a problem hiding this comment.
interesting why logger is not configured for those tests like it is configured for ui-tests/tst
There was a problem hiding this comment.
the console.log statements in the puppeteer scripts doesn't log and is diverted to the result variable, just adding a precautionary logging in case the test case failed, the test scripts have a lot more detailed logging as the test progresses
There was a problem hiding this comment.
Can you please try looking at the detailed logs from the plugin to see if you can see these errors, then we can remove the print statements
There was a problem hiding this comment.
Yeah I looked at all logs for a failure and success test case and the puppeteer script logs were not present in any file, so keeping these in to ensure we can debug test failures. Could be something wrong with the driver if it's supposed to be logging all the logs from the puppeteer script execution?
...-243+/software/aws/toolkits/jetbrains/uitests/docTests/createReadmeTests/CreateReadmeTest.kt
Show resolved
Hide resolved
...-243+/software/aws/toolkits/jetbrains/uitests/docTests/createReadmeTests/CreateReadmeTest.kt
Show resolved
Hide resolved
| val testCase = TestCase( | ||
| IdeProductProvider.IC, | ||
| LocalProjectInfo( | ||
| Paths.get("tstData", "qdoc", "createFlow") |
There was a problem hiding this comment.
What is the difference other than the readme file between the 2 projects? Is there a way we can reuse the existing project?
There was a problem hiding this comment.
We have a test case (in progress) that tests if we are able to generate a readme in a multi-root workspace so reusing doesn't help, we would still need atleast two different dummy repos for testing
...-243+/software/aws/toolkits/jetbrains/uitests/docTests/createReadmeTests/CreateReadmeTest.kt
Show resolved
Hide resolved
| assertTrue(result.contains("Test Successful")) | ||
| assertFalse(result.contains("Error: Test Failed")) | ||
| if (result.contains("Error: Test Failed")) { | ||
| println("result: $result") |
There was a problem hiding this comment.
Can you please try looking at the detailed logs from the plugin to see if you can see these errors, then we can remove the print statements
gandhi-21
force-pushed
the
createReadmeTests
branch
from
ef3c4ef to
a85591e
Compare
|
/runUiTests |
4 participants
Types of changes
Description
Followups from #5435:
findAndClickButtonto a common Utils.kt file in the tests directoryAdd UI tests to validate the
Create a READMEflowCreated a sample repository from scratch,
There are 5 test cases which validate the following
Checklist
License
I confirm that my contribution is made under the terms of the Apache 2.0 license.