-
Notifications
You must be signed in to change notification settings - Fork 0
[DRAFT] tests: initial e2e config and basic testing #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
Conversation
| import { AUCTION_TOKEN_ADDRESS } from "../../../../app-config"; | ||
| import { COMPONENTS } from "../constants"; | ||
|
|
||
| const BASE_URL = "http://localhost:5173"; |
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 this could be the BASE_URL from constants
apps/dapp/cypress/constants.ts
Outdated
| const LAUNCH_ID = | ||
| Cypress.env("VITE_TESTNET") === "true" ? blastSepoliaChainId : baseChainId; | ||
|
|
||
| const APP_BASE_URL = "http://localhost:5173/#"; |
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.
Isn't the BASE_URL on line 1 this value?
|
|
||
| describe("Auction List Page", () => { | ||
| it("Should load the root page", () => { | ||
| cy.visit(BASE_URL); |
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.
This will pass even if the home page fails and an error page is shown.
I think we can omit this top level test from here as its covered in the smoke tests:
smoke.cy.ts
it("renders home page", () => {
cy.visit(URLS.HOME);
cy.shouldRenderPageWithId("__AXIS_HOME_PAGE__");
});
edmulraney
left a comment
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.
🔥
No description provided.