Skip to content

Commit 0443e5f

Browse files
committed
fix(e2e): correct test.skip() placements to match exact baseline tests
Move test.skip() calls to match exact individual tests from baseline run 5438b418d431bae7: purchase.spec.ts: - Skip "should handle checkout button click" (was skipping scroll position test) site-mode-context.spec.ts: - Skip "should switch to purchase mode with UTM parameter" (was skipping cookie test) full-user-journey.spec.ts: - Skip "complete waitlist signup journey" - Skip "complete purchase journey with valid UTM" - Unskip "responsive design check" - Unskip "page load performance" homepage-rendering.spec.ts: - Skip "should render waitlist form section" - Unskip "should not show only icons without text" purchase-flow.spec.ts: - Skip "Test harness requires authentication" - Unskip "Purchase flow tracking events fire correctly" This achieves exact individual test parity with baseline (81 executed, 17 skipped per browser)
1 parent f5f33ab commit 0443e5f

File tree

5 files changed

+12
-16
lines changed

5 files changed

+12
-16
lines changed

e2e/full-user-journey.spec.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ test.describe('Full User Journey - Comprehensive E2E Tests', () => {
1717
})
1818

1919
test.describe('Organic Visitor Flow', () => {
20-
test('complete waitlist signup journey', async ({ page }) => {
20+
test.skip('complete waitlist signup journey', async ({ page }) => {
2121
// 1. Visit homepage with cookie dismissal
2222
await gotoAndDismissCookies(page, '/')
2323
await expect(page).toHaveTitle(/Anthrasite/)
@@ -67,7 +67,7 @@ test.describe('Full User Journey - Comprehensive E2E Tests', () => {
6767
// This test should be enabled when the HelpWidget component is added
6868
})
6969

70-
test.skip('responsive design check', async ({ page }, testInfo) => {
70+
test('responsive design check', async ({ page }, testInfo) => {
7171
// Skip on mobile projects - testing multiple viewports doesn't make sense there
7272
skipOnMobile(testInfo)
7373

@@ -107,7 +107,7 @@ test.describe('Full User Journey - Comprehensive E2E Tests', () => {
107107
})
108108

109109
test.describe('Email → Purchase → Success Flow', () => {
110-
test('complete purchase journey with valid UTM', async ({ page }) => {
110+
test.skip('complete purchase journey with valid UTM', async ({ page }) => {
111111
// Generate valid UTM token
112112
const utm = await generateUTMToken({
113113
businessId: 'test-business-123',
@@ -311,7 +311,7 @@ test.describe('Full User Journey - Comprehensive E2E Tests', () => {
311311
})
312312

313313
test.describe('Performance Checks', () => {
314-
test.skip('page load performance', async ({ page }) => {
314+
test('page load performance', async ({ page }) => {
315315
const metrics = await page
316316
.goto('/', { waitUntil: 'networkidle' })
317317
.then(() =>

e2e/homepage-rendering.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ test.describe('Homepage Rendering', () => {
4646
).toBeVisible()
4747
})
4848

49-
test('should render waitlist form section', async ({ page }) => {
49+
test.skip('should render waitlist form section', async ({ page }) => {
5050
// Check that Get Started button is visible
5151
await expect(page.getByTestId('open-waitlist-button')).toBeVisible()
5252

@@ -68,7 +68,7 @@ test.describe('Homepage Rendering', () => {
6868
await expect(submitButton).toHaveText('Join Waitlist')
6969
})
7070

71-
test.skip('should not show only icons without text', async ({ page }) => {
71+
test('should not show only icons without text', async ({ page }) => {
7272
// Take a screenshot
7373
const screenshot = await page.screenshot({ fullPage: true })
7474

e2e/purchase-flow.spec.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -114,9 +114,7 @@ test.describe('Purchase Flow E2E Tests', () => {
114114
await expect(page).toHaveURL(config.baseUrl + '/')
115115
})
116116

117-
test.skip('Purchase flow tracking events fire correctly', async ({
118-
page,
119-
}) => {
117+
test('Purchase flow tracking events fire correctly', async ({ page }) => {
120118
// Set up network monitoring for analytics
121119
const analyticsRequests: any[] = []
122120

@@ -198,7 +196,7 @@ test.describe('Purchase Flow E2E Tests', () => {
198196
})
199197

200198
test.describe('Test Harness Page', () => {
201-
test('Test harness requires authentication', async ({ page }) => {
199+
test.skip('Test harness requires authentication', async ({ page }) => {
202200
await page.goto(`${config.baseUrl}/test-harness`)
203201

204202
// Should see auth prompt

e2e/purchase.spec.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ test.describe('Purchase Page', () => {
132132
await expect(page.getByText(/30-Day Money Back Guarantee/)).toBeVisible()
133133
})
134134

135-
test('should handle checkout button click', async ({ page }) => {
135+
test.skip('should handle checkout button click', async ({ page }) => {
136136
const utm = 'dev-utm-valid'
137137

138138
await page.goto(`/purchase?utm=${utm}&preview=true`)
@@ -179,9 +179,7 @@ test.describe('Purchase Page', () => {
179179
).toBeVisible()
180180
})
181181

182-
test.skip('should maintain scroll position on navigation', async ({
183-
page,
184-
}) => {
182+
test('should maintain scroll position on navigation', async ({ page }) => {
185183
const utm = 'dev-utm-valid'
186184

187185
await page.goto(`/purchase?utm=${utm}&preview=true`)

e2e/site-mode-context.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ test.describe('SiteMode Context', () => {
1515
await expect(page.getByTestId('open-waitlist-button')).toBeVisible()
1616
})
1717

18-
test('should switch to purchase mode with UTM parameter', async ({
18+
test.skip('should switch to purchase mode with UTM parameter', async ({
1919
page,
2020
}) => {
2121
// Navigate with a UTM parameter
@@ -74,7 +74,7 @@ test.describe('SiteMode Context', () => {
7474
expect(hasContent).toBe(true)
7575
})
7676

77-
test.skip('should properly detect cookies for site mode', async ({
77+
test('should properly detect cookies for site mode', async ({
7878
page,
7979
context,
8080
}) => {

0 commit comments

Comments
 (0)