test: e2e tests added for psm UI #125
Conversation
| }); | ||
|
|
||
| it('should swap tokens from IST to stable', () => { | ||
| let ISTbalance; |
There was a problem hiding this comment.
our style is camel case
| let ISTbalance; | |
| let istBalance; |
There was a problem hiding this comment.
Looks like we need to exclude tests/e2e from vitest, I think we can do this by modifying vitest.config.ts like https://vitest.dev/config/#exclude
@samsiegart updated accordingly |
vitest.config.ts
Outdated
| @@ -8,6 +8,10 @@ export default mergeConfig( | |||
| test: { | |||
| setupFiles: ['src/installSesLockdown.ts'], | |||
| environment: 'happy-dom', | |||
| exclude: [ | |||
| ...configDefaults.exclude, | |||
| 'tests/e2e/**' | |||
There was a problem hiding this comment.
alternately we could consider tests to be the purview of vitest and have synpress run in /integration-tests or /e2e-tests. I advise this to simplify the configurations but I defer to whatever @samsiegart prefers.
There was a problem hiding this comment.
if our goal is to simply config then just a head up: changing this path will require updating the synpress.config.js file since it uses tests/e2e/** as its default path
There was a problem hiding this comment.
Thanks for the heads up, I think we can consider separating the directories but not a blocker
This PR aims to add e2e tests to the repo using the @agoric/synpress library. CI/CD setup to run these tests will be handled in a future PR
How to run and verify tests:
Note: Once the tests have been run a few times, you will need to recreate the docker container for the chain, since there will not be enough tokens to run the test again