Skip to content

Commit

Permalink
chore: Update success URLs for E2E tests
Browse files Browse the repository at this point in the history
  • Loading branch information
flozia committed Sep 3, 2024
2 parents 553f582 + 7e40cfd commit a540ed9
Show file tree
Hide file tree
Showing 4 changed files with 183 additions and 186 deletions.
2 changes: 1 addition & 1 deletion playwright.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ export default defineConfig({
retries: process.env.CI ? 1 : 0,

/* Use all of the available wokers in CI and use default locally. */
workers: process.env.CI ? "100%" : undefined,
workers: process.env.CI ? "75%" : undefined,

/* Reporter to use. See https://playwright.dev/docs/test-reporters */
reporter: process.env.CI ? [['github'], ['html']] : 'html',
Expand Down
2 changes: 1 addition & 1 deletion src/e2e/pages/dashBoardPage.ts
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ export class DashboardPage {
'//a[starts-with(@class, "Shell_mozillaLink")]',
);
this.allBreachesFooter = page.locator("footer a", {
hasText: "All Breaches",
hasText: "Recent data breaches",
});
this.faqsFooter = page.locator("footer a", { hasText: "FAQs" });
this.termsOfServiceFooter = page.locator("footer a", {
Expand Down
4 changes: 2 additions & 2 deletions src/e2e/specs/auth.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,8 @@ test.describe(`${process.env.E2E_TEST_ENV} - Authentication flow verification @s
await authPage.signUp(randomEmail, page);

// assert successful login
const successUrl = "/user/welcome";
expect(page.url()).toBe(`${process.env.E2E_TEST_BASE_URL}${successUrl}`);
const successUrl = `${process.env.E2E_TEST_BASE_URL}/user/welcome`;
expect(page.url()).toBe(successUrl);

Check failure on line 31 in src/e2e/specs/auth.spec.ts

View workflow job for this annotation

GitHub Actions / e2e-tests

[chromium] › auth.spec.ts:12:3 › local - Authentication flow verification @smoke › Verify sign up with new user

1) [chromium] › auth.spec.ts:12:3 › local - Authentication flow verification @smoke › Verify sign up with new user Error: expect(received).toBe(expected) // Object.is equality Expected: "***/user/welcome" Received: "***/user/dashboard" 29 | // assert successful login 30 | const successUrl = `${process.env.E2E_TEST_BASE_URL}/user/welcome`; > 31 | expect(page.url()).toBe(successUrl); | ^ 32 | 33 | await testInfo.attach( 34 | `${process.env.E2E_TEST_ENV}-signup-monitor-dashboard.png`, at /home/runner/work/blurts-server/blurts-server/src/e2e/specs/auth.spec.ts:31:24

await testInfo.attach(
`${process.env.E2E_TEST_ENV}-signup-monitor-dashboard.png`,
Expand Down
Loading

0 comments on commit a540ed9

Please sign in to comment.