Skip to content
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

feat(web): Improve web e2e tests and move them to their directory #16842

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 13 commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
0f0c704
feat: Move web e2e tests to web app directory
svanaeinars Nov 13, 2024
f626089
Merge branch 'main' into feat/move-and-improve-e2e-tests-web
svanaeinars Nov 14, 2024
9ee7e96
Merge branch 'main' into feat/move-and-improve-e2e-tests-web
svanaeinars Nov 14, 2024
05e489f
Remove e2e from jest and improve tests
svanaeinars Nov 14, 2024
af54dcc
Minor tweaks
svanaeinars Nov 15, 2024
1226759
Small test improvements
svanaeinars Nov 15, 2024
dc24b29
Docs update for web
svanaeinars Nov 15, 2024
c3f0dbf
Minor tweak
svanaeinars Nov 15, 2024
3e2cbf3
Workflow test
svanaeinars Nov 18, 2024
0d9addf
Change web start command e2e playwright
svanaeinars Nov 18, 2024
394c853
Remove Playwright projects and add Tags
svanaeinars Nov 20, 2024
03cdcb7
Merge branch 'main' into feat/move-and-improve-e2e-tests-web
svanaeinars Nov 20, 2024
b946139
Revert workflow test
svanaeinars Nov 20, 2024
0783e04
Minor test updates
svanaeinars Nov 21, 2024
50b5a4d
Merge branch 'main' into feat/move-and-improve-e2e-tests-web
svanaeinars Nov 21, 2024
d2393a5
Docs update for web
svanaeinars Nov 21, 2024
a35543c
Test
svanaeinars Nov 21, 2024
9251134
Make web proxy URL dynamic
svanaeinars Nov 21, 2024
3e5405b
chore: charts update dirty files
andes-it Nov 21, 2024
14c8be2
Fix
svanaeinars Nov 21, 2024
0964e55
chore: charts update dirty files
andes-it Nov 21, 2024
1a4eea6
Merge branch 'main' into feat/move-and-improve-e2e-tests-web
svanaeinars Nov 22, 2024
9cf35bf
Elastisearch docker-compose for offline tests
svanaeinars Nov 22, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 36 additions & 0 deletions apps/web/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -90,6 +90,42 @@ yarn nx extract-strings web

Currently, in this project, only the `Custom Page` content type utilizes the `Translation Namespace` content type for translations

## E2E Testing

### Quick Start

To run the E2E tests for the `web` app:

1. **Install dependencies** (if not already installed):

```bash
yarn install && yarn playwright install
```

2. **Run the tests**:

```bash
yarn e2e web
```

### Test Organization

The E2E tests for the `web` app are located in the `e2e` folder.

Tests are tagged based on their execution time or other criteria. Use `@fast` for quick tests and `@slow` for longer-running tests. Example:

```typescript
import { test, expect } from '@playwright/test'

test('should get paid', { tag: '@slow' }, () => {
// ...
})
```

### More Resources

Refer to the [E2E Testing Library README](../../libs/testing/e2e/README.md) for common helper functions and utilities.
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved

## Further Documentation

[Subpages](./docs/subpages.md) - Information on Layouts and Components used when
Expand Down
212 changes: 212 additions & 0 deletions apps/web/e2e/homepage.spec.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,212 @@
import {
type BrowserContext,
createPageAndNavigate,
expect,
session,
test,
urls,
} from '@island.is/testing/e2e'

test.use({ baseURL: urls.islandisBaseUrl })

test.describe('Front page', { tag: '@fast' }, () => {
let context: BrowserContext

test.beforeAll(async ({ browser }) => {
context = await session({
browser: browser,
storageState: 'homepage.json',
homeUrl: `${urls.islandisBaseUrl}/`,
phoneNumber: '0103019',
idsLoginOn: false,
})
})

test.afterAll(async () => {
await context.close()
})

test('has expected sections @lang:is', async () => {
const page = await createPageAndNavigate(context, '/')
await expect(
page.locator('text=Öll opinber þjónusta á einum stað'),
).toBeVisible()
await expect(page.locator('data-testid=home-banner')).toBeVisible()
await expect(page.locator('data-testid=home-heading')).toBeVisible()
await expect(page.locator('data-testid=home-news')).toBeVisible()
})

test(`should have life event @lang:is`, async () => {
test.slow()
const page = await createPageAndNavigate(context, '/')
const lifeEventsCards = page.locator('[data-testid="lifeevent-card"]')
await expect(lifeEventsCards.count()).resolves.toBeGreaterThan(3)
const lifeEventHandles = await lifeEventsCards.elementHandles()
const lifeEventUrls = await Promise.all(
lifeEventHandles.map((item) => item.getAttribute('href')),
)
const lifeEventPage = await context.newPage()
for (const url of lifeEventUrls) {
const result = await lifeEventPage.goto(url ?? '')
await expect(
lifeEventPage.getByRole('link', { name: 'island.is logo' }),
).toBeVisible()
expect(result?.status()).toBe(200)
}
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved
await lifeEventPage.close()
})

test(`should have life event @lang:en`, async () => {
test.slow()
const page = await createPageAndNavigate(context, '/en')
const lifeEventsCards = page.locator('[data-testid="lifeevent-card"]')
await expect(lifeEventsCards.count()).resolves.toBeGreaterThan(3)
const lifeEventHandles = await lifeEventsCards.elementHandles()
const lifeEventUrls = await Promise.all(
lifeEventHandles.map((item) => item.getAttribute('href')),
)
const lifeEventPage = await context.newPage()
for (const url of lifeEventUrls) {
const result = await lifeEventPage.goto(url ?? '')
await expect(
lifeEventPage.getByRole('link', { name: 'island.is logo' }),
).toBeVisible()
expect(result?.status()).toBe(200)
}
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved
await lifeEventPage.close()
})

test(`should navigate to featured link @lang:is`, async () => {
test.slow()
const page = await createPageAndNavigate(context, '/')
const featuredLinks = page.locator('[data-testid="featured-link"]')
await expect(featuredLinks.count()).resolves.toBeGreaterThan(3)
const featuredLinksHandles = await featuredLinks.elementHandles()
const featuresLinksUrls = await Promise.all(
featuredLinksHandles.map((item) => item.getAttribute('href')),
)
const featuredPage = await context.newPage()
for (const url of featuresLinksUrls) {
if (url) {
const result = await featuredPage.goto(url)
await expect(
featuredPage.getByRole('link', { name: 'island.is logo' }),
).toBeVisible()
if (result) {
expect(result.status()).toBe(200)
}
}
}
await featuredPage.close()
})

test(`should navigate to featured link @lang:en`, async () => {
test.slow()
const page = await createPageAndNavigate(context, '/en')
const featuredLinks = page.locator('[data-testid="featured-link"]')
await expect(featuredLinks.count()).resolves.toBeGreaterThan(3)
const featuredLinksHandles = await featuredLinks.elementHandles()
const featuresLinksUrls = await Promise.all(
featuredLinksHandles.map((item) => item.getAttribute('href')),
)
const featuredPage = await context.newPage()
for (const url of featuresLinksUrls) {
if (url) {
const result = await featuredPage.goto(url)
await expect(
featuredPage.getByRole('link', { name: 'island.is logo' }),
).toBeVisible()
if (result) {
expect(result.status()).toBe(200)
}
}
}
await featuredPage.close()
})

test(`should have link on life events pages to navigate back to the main page @lang:is`, async () => {
test.slow()
const page = await createPageAndNavigate(context, '/')
const lifeEventsCards = page.locator('[data-testid="lifeevent-card"]')
const lifeEventHandles = await lifeEventsCards.elementHandles()
const lifeEventUrls = await Promise.all(
lifeEventHandles.map((item) => item.getAttribute('href')),
)
const lifeEventPage = await context.newPage()
for (const url of lifeEventUrls) {
if (url) {
await lifeEventPage.goto(url)
await lifeEventPage.locator('[data-testid="link-back-home"]').click()
}
await expect(
lifeEventPage.locator('data-testid=home-heading'),
).toBeVisible()
await expect(lifeEventPage).toHaveURL('/')
}
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved
await lifeEventPage.close()
})

test(`should have link on life events pages to navigate back to the main page @lang:en`, async () => {
test.slow()
const page = await createPageAndNavigate(context, '/en')
const lifeEventsCards = page.locator('[data-testid="lifeevent-card"]')
const lifeEventHandles = await lifeEventsCards.elementHandles()
const lifeEventUrls = await Promise.all(
lifeEventHandles.map((item) => item.getAttribute('href')),
)
const lifeEventPage = await context.newPage()
for (const url of lifeEventUrls) {
if (url) {
await lifeEventPage.goto(url)
await lifeEventPage.locator('[data-testid="link-back-home"]').click()
}
await lifeEventPage.locator('[data-testid="link-back-home"]').click()
await expect(
lifeEventPage.locator('data-testid=home-heading'),
).toBeVisible()
await expect(lifeEventPage).toHaveURL('/en')
}
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved
await lifeEventPage.close()
})

test('should change welcome message on language toggle @lang:is', async () => {
const page = await createPageAndNavigate(context, '/')
const homeHeading = page.locator('h1[data-testid="home-heading"]')
const icelandicHeading = await homeHeading.textContent()
await page.locator('button[data-testid="language-toggler"]:visible').click()
if (icelandicHeading) {
await expect(homeHeading).not.toHaveText(icelandicHeading)
}
await expect(page).toHaveURL('/en')
})

test('should toggle mega-menu @lang:is', async () => {
const page = await createPageAndNavigate(context, '/')
await page
.locator('[data-testid="frontpage-burger-button"]:nth-child(2)')
.click()
await expect(
page.locator('[data-testid="mega-menu-link"] > a').count(),
).resolves.toBeGreaterThan(18)
})

test('burger menu should open and close', async () => {
const page = await createPageAndNavigate(context, '/')
await page.getByRole('button', { name: 'Valmynd' }).click()

await expect(page.getByRole('dialog', { name: 'Menu' })).toBeVisible()
await expect(
page.getByRole('paragraph').filter({ hasText: 'Þjónustuflokkar' }),
).toBeVisible()
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved
await expect(page.getByRole('dialog', { name: 'Menu' })).toBeVisible()
// Heading is "visible" behind menu
// await expect(page.getByTestId('home-heading')).not.toBeVisible()
await page
.getByRole('dialog', { name: 'Menu' })
.getByRole('button')
.getByTestId('icon-close')
.click()
await expect(page.getByTestId('home-heading')).toBeVisible()
await expect(page.getByRole('dialog', { name: 'Menu' })).not.toBeVisible()
})
})
57 changes: 57 additions & 0 deletions apps/web/e2e/search.spec.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,57 @@
import {
type BrowserContext,
expect,
session,
test,
urls,
} from '@island.is/testing/e2e'

test.use({ baseURL: urls.islandisBaseUrl })

test.describe('Search feature', { tag: '@fast' }, () => {
let context: BrowserContext
test.beforeAll(async ({ browser }) => {
context = await session({
browser: browser,
storageState: 'homepage.json',
homeUrl: `${urls.islandisBaseUrl}/`,
phoneNumber: '0103019',
idsLoginOn: false,
})
})
test.afterAll(async () => {
await context.close()
})

test('has expected sections', async () => {
const testPhrase = 'umsókn'
const page = await context.newPage()
await page.goto('/', { waitUntil: 'networkidle' })
await page
.getByRole('textbox', { name: 'Leitaðu á Ísland.is' })
.fill(testPhrase)
await page.keyboard.press('Enter')
const testResults = page.locator('[data-testid="search-result"]')
await expect(testResults.count()).resolves.toBeGreaterThan(9)
const searchUrl = page.url()
await testResults.nth(0).click()
await page.waitForLoadState('networkidle')
await expect(page).not.toHaveURL(searchUrl)
})
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved

test('should have no search results for long bogus search words', async () => {
const page = await context.newPage()
await page.goto('/', { waitUntil: 'networkidle' })
await page
.getByRole('textbox', { name: 'Leitaðu á Ísland.is' })
.fill('abcdefhijklmnopqrstuvwxyz1234567890')
await page.keyboard.press('Enter')
await page.waitForLoadState('networkidle')
const testResults = page.locator('[data-testid="search-result"]')
await expect(testResults).toHaveCount(0)
})
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved

test.skip('should search in English', async () => {
return
})
})
Loading