-
Notifications
You must be signed in to change notification settings - Fork 7
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
URL changes in e2e tests to make it work locally #188
Conversation
Deploying wallet-app with Cloudflare Pages
|
Cloudflare deployment logs are available here |
0ac6128
to
14d454c
Compare
@@ -19,15 +19,15 @@ describe('Wallet App Test Cases', { execTimeout: DEFAULT_EXEC_TIMEOUT }, () => { | |||
}); | |||
}); | |||
it(`should open the web wallet URL successfully`, () => { | |||
cy.visit(`${userConfig.webwalletURL}`); | |||
cy.visit('/'); |
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.
Shouldn't we do it conditionally wheb AGORIC_NET is local?
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.
All networks should use localhost:3000
for testing. Previously, all networks used localhost:3000
except when AGORIC_NET
was set to local
, in which case https://wallet.agoric.app/
was used. You can see this in the webWalletURL
parameter here.
Looks like E2E tests are passing with Only the test case related to viewing bids from CLI is failing. It fails with most networks except |
After updating the
ses
version in #187, thea3p
chain is working fine with the wallet app. This PR updates the e2e tests to use thelocalhost
URL whenAGORIC_NET
islocal
.closes #184