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 1 commit
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
146 changes: 146 additions & 0 deletions apps/web/e2e/smoke/homepage.spec.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,146 @@
import {
BrowserContext,
expect,
session,
test,
urls,
} from '@island.is/testing/e2e'

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

test.describe('Front page', () => {
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 context.newPage()
await page.goto('/')
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()
})

for (const { lang, home } of [
{ lang: 'is', home: '/' },
{ lang: 'en', home: '/en' },
]) {
test(`should have life event @lang:${lang}`, async () => {
test.slow()
const page = await context.newPage()
await page.goto(home)
const lifeEventsCards = page.locator('[data-testid="lifeevent-card"]')

await expect(lifeEventsCards).toHaveCountGreaterThan(3)
const lifeEventHandles = await lifeEventsCards.elementHandles()
const lifeEventUrls = await Promise.all(
lifeEventHandles.map((item) => item.getAttribute('href')),
)
for (const url of lifeEventUrls) {
const page = await context.newPage()
const result = await page.goto(url!)
await expect(
page.getByRole('link', { name: 'island.is logo' }),
).toBeVisible()
expect(result!.status()).toBe(200)
await page.close()
}
})
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved
test(`should navigate to featured link @lang:${lang}`, async () => {
test.slow()
const page = await context.newPage()
await page.goto(home)
const featuredLinks = page.locator('[data-testid="featured-link"]')
await expect(featuredLinks).toHaveCountGreaterThan(3)
const featuredLinksHandles = await featuredLinks.elementHandles()
const featuresLinksUrls = await Promise.all(
featuredLinksHandles.map((item) => item.getAttribute('href')),
)
for (const url of featuresLinksUrls) {
const page = await context.newPage()
const result = await page.goto(url!)
await expect(
page.getByRole('link', { name: 'island.is logo' }),
).toBeVisible()
expect(result!.status()).toBe(200)
await page.close()
}
})

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

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

test('should toggle mega-menu @lang:is', async () => {
const page = await context.newPage()
await page.goto('/')
await page
.locator('[data-testid="frontpage-burger-button"]:nth-child(2)')
.click()
await expect(
page.locator('[data-testid="mega-menu-link"] > a'),
).toHaveCountGreaterThan(18)
})
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved

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

// Act
await expect(page.getByRole('dialog', { name: 'Menu' })).toBeVisible()
await expect(
page.getByRole('paragraph').filter({ hasText: 'Þjónustuflokkar' }),
).toBeVisible()
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()
})
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved
})
58 changes: 58 additions & 0 deletions apps/web/e2e/smoke/search.spec.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,58 @@
import {
BrowserContext,
expect,
session,
test,
urls,
} from '@island.is/testing/e2e'

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

test.describe('Search feature', () => {
let context: BrowserContext
test.beforeAll(async ({ browser }) => {
context = await session({
browser: browser,
storageState: 'homepage.json',
homeUrl: `${urls.islandisBaseUrl}/`,
phoneNumber: '0103019',
idsLoginOn: false,
})
})
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved
test.afterAll(async () => {
await context.close()
})

test('has expected sections', async () => {
const testPhrase = 'umsókn'
const page = await context.newPage()
await page.goto('/')
await page
.getByRole('textbox', { name: 'Leitaðu á Ísland.is' })
.type(testPhrase, { delay: 100 })
await page.keyboard.press('Enter')
const testResults = page.locator('[data-testid="search-result"]')
await expect(testResults).toHaveCountGreaterThan(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('/')
await page.type(
'role=textbox[name="Leitaðu á Ísland.is"]',
'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 Enlish', async () => {
return
})
})
80 changes: 80 additions & 0 deletions apps/web/e2e/smoke/sites-of-institutions.spec.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,80 @@
import slugify from 'slugify'

import {
BrowserContext,
expect,
Page,
session,
test,
urls,
} from '@island.is/testing/e2e'

type GetByRole = Pick<Page, 'getByRole'>['getByRole']
type GetByRoleParameters = Parameters<GetByRole>

test.use({ baseURL: urls.islandisBaseUrl })
type Orgs = {
organisationName: string
organisationHome?: string
enabled?: boolean
target?: { role: GetByRoleParameters[0]; options?: GetByRoleParameters[1] }
}
const orgs: Orgs[] = [
{ organisationName: 'Opinberir aðilar', organisationHome: '/' },
{ organisationName: 'Fiskistofa' },
{
organisationName: 'Heilbrigðisstofnun Norðurlands',
organisationHome: '/hsn',
},
{ organisationName: 'Sjúkratryggingar', target: { role: 'link' } },
{ organisationName: 'Ríkislögmaður' },
{ organisationName: 'Landskjörstjórn', target: { role: 'link' } },
{ organisationName: 'Opinber nýsköpun', enabled: true },
{ organisationName: 'Sýslumenn' },
{ organisationName: 'Fjársýslan' },
{
organisationName: 'Heilbrigðisstofnun Suðurlands',
organisationHome: '/hsu',
},
{
organisationName: 'Landlæknir',
target: { role: 'link', options: { name: 'Eyðublöð' } },
},
{ organisationName: 'Útlendingastofnun', target: { role: 'link' } },
]
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved

test.describe('Sites of institutions', () => {
let context: BrowserContext
test.beforeAll(async ({ browser }) => {
context = await session({
browser,
idsLoginOn: false,
homeUrl: '/s',
})
})
test.afterAll(async () => {
await context.close()
})
for (const {
organisationName,
organisationHome = `/${slugify(organisationName, { lower: true })}`,
target,
enabled,
} of orgs) {
test(organisationName, async () => {
if (enabled) return
const page = await context.newPage()
const url = `/s${organisationHome}`
await page.goto(url)
await expect(
page
.getByRole(target?.role ?? 'heading', {
...{ name: organisationName },
...target?.options,
})
.first(),
).toBeVisible()
await page.close()
})
}
})
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved
40 changes: 40 additions & 0 deletions apps/web/e2e/utils/addons.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
import { expect, Locator, Page, sleep } from '@island.is/testing/e2e'

expect.extend({
async toHaveCountGreaterThan(
received: Locator,
value: number,
options: { timeout: number; sleepTime: number } = {
timeout: 10000,
sleepTime: 100,
},
) {
const initialTime = Date.now()
let count = -1
while (count <= value) {
count = await received.count()
if (Date.now() > initialTime + options.timeout)
return { message: () => 'Timeout', pass: false }
await sleep(options.sleepTime)
}
return {
message: () => `Found ${count} elements`,
pass: true,
}
},
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved
async toBeApplication(received: string | Page, applicationType = '\\w+') {
const url: string = typeof received == 'string' ? received : received.url()
const protocol = 'https?://'
const host = '[^/]+'
const applicationId = '(/(\\w|-)*)?'
const applicationRegExp = new RegExp(
`^${protocol}${host}/umsoknir/${applicationType}${applicationId}$`,
)
const pass = applicationRegExp.test(url)
const message = () =>
`Current page is ${pass ? '' : '*not* '}an application
Pattern ${applicationRegExp}
URL is ${url}`
return { message, pass }
},
})
11 changes: 11 additions & 0 deletions apps/web/playwright.config.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
import { createPlaywrightConfig } from '@island.is/testing/e2e'

import './e2e/utils/addons'

const webConfig = createPlaywrightConfig({
webServerUrl: 'http://localhost:4200',
command: 'yarn dev-init web && source .env.secret && yarn dev web',
cwd: '../../',
})
svanaeinars marked this conversation as resolved.
Show resolved Hide resolved

export default webConfig
6 changes: 6 additions & 0 deletions apps/web/project.json
Original file line number Diff line number Diff line change
Expand Up @@ -138,6 +138,12 @@
},
"docker-next": {
"executor": "Intentionally left blank, only so this target is valid when using `nx show projects --with-target docker-next`"
},
"e2e": {
"executor": "@nx/playwright:playwright",
"options": {
"config": "{projectRoot}/playwright.config.ts"
}
Comment on lines +150 to +154
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion

Configure target dependencies for e2e tests

The e2e target should explicitly depend on required services to ensure proper test execution.

 "e2e": {
   "executor": "@nx/playwright:playwright",
   "options": {
     "config": "{projectRoot}/playwright.config.ts"
-  }
+  },
+  "dependsOn": [
+    "dev-services"
+  ]
 }
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"e2e": {
"executor": "@nx/playwright:playwright",
"options": {
"config": "{projectRoot}/playwright.config.ts"
}
"e2e": {
"executor": "@nx/playwright:playwright",
"options": {
"config": "{projectRoot}/playwright.config.ts"
},
"dependsOn": [
"dev-services"
]
}

}
}
}
Loading