Code Review

Struktura a dedicnost

• Dedi z BasePage
• Vsechny lokatory jako readonly properties
• Lokatory inicializovane v konstruktoru na jeden radek
• Implementuje povinnou metodu goto(language?: string)
• Ma metodu isLoaded() pro overeni nacteni

Lokatory (viz skill playwright-locators)

• Zadne CSS selektory: page.locator('.class'), page.locator('#id'), page.locator('div > span')
• Zadne XPath selektory
• Preferuje getByTestId() > getByRole() > getByLabel() > getByPlaceholder() > getByText()
• Fallback na getByRole/getByLabel ma komentar s TODO na pridani testId
• Lokalizovane texty pres test data (loader) nebo regex

Interakce

• Pouziva BasePage metody: clickElement(), fillInput(), clearInput(), selectOption(), checkCheckbox(), uncheckCheckbox()
• Nepouziva primo locator.click() nebo locator.fill() – pouzij BasePage

Zakazane v Page Objects

• Zadne assertions (expect) – patri do testu
• Zadne page.waitForTimeout() – pouzij WaitHelper nebo BasePage
• Zadne hardcodovane credentials – pouze parametry

Konvence metod

• Akcni metody vraceji Promise<void>
• Pojmenovani: fillX(), clickX(), getX(), slozene jako login(), submit()
• Zadne typy navracejici assertion vysledky

Checklist – testovaci soubory (tests/*.spec.ts)

Importy a fixtures

• Import z ./fixtures/page: import { test, expect } from './fixtures/page'
• Test data z loader: loginpageData() atd.
• Konstanty z ./constants: TEST_USER pro credentials

Struktura testu

• test.describe() pro logicke skupiny
• Popis testu jasny, oznamovaci zpusob
• Tagy: @smoke, @regression, @security, @performance, @integration kde vhodne

Lokatory v testech

• Preferuje metody Page Objectu pred ad-hoc lokatory
• Pri ad-hoc lokatorech: getByTestId > getByRole > getByLabel > ...
• Zadne page.locator() s CSS/XPath

Aserce a cekani

• Web-first assertions: expect(locator).toBeVisible(), toHaveText(), toBeEnabled()
• Zadne page.waitForTimeout()
• Explicitni aserce – zadne expect(result).toBeTruthy()

Izolace a data

• Kazdy test nezavisly – zadne sdileni stavu
• Credentials z TEST_USER nebo fixture
• Texty z test dat (testData.en, testData.cs) – ne hardcode

Checklist – fixtures (tests/fixtures/page.ts)

• Typ PagesFixtures obsahuje vsechny page fixtures
• Konvence: pageName (bez prednacteni), pageNameCS / pageNamePL (s goto)
• Fixture vola await use(pageObject) po priprave
• Page tridy s parametrem v konstruktoru maji spravny predany jazyk

Checklist – obecne (TypeScript, styl)

• Async/await, single quotes, 2 mezery, max 120 znaku
• Explicitni typy pro parametry a navratove hodnoty
• Zaverecny novy radek, zadny trailing whitespace
• Zadne credentials/API klice/tokeny natvrdo

Checklist – test data a helpers

testData/*.json

• Struktura podle jazyka (en, cs, pl)
• Zadna citliva data v JSON

helpers

• WaitHelper misto page.waitForTimeout()
• WaitHelper.sleep() jen pro debug – nikoli v produkcnim kodu

Format zpetne vazby

Pouzivej tristupnove oznaceni:

Priklad vystupu

## Code Review – login.spec.ts

### 🔴 Kriticke
- **Lokatory v LoginPage.ts**: Pouzity `page.locator('input[name="Username"]')` – prepis na `getByTestId()` nebo `getByLabel()`. Viz skill playwright-locators.

### 🟡 Navrh
- **Aserce**: `expect(validSummaryErrors).toContain(...)` – zvaz `expect(locator).toHaveText()` pro web-first assertion.
- **Test data**: Konstanta `testData` na urovni describe – v poradku, ale u vetsich souboru zvaz beforeEach pro cerstva data.

### 🟢 Nice to have
- Pridani testId do frontend komponent pro stabilnejsi lokatory.

Nejcastejsi problemy (antipatterny)

Reference na souvisejici skills

• Lokatory: playwright-locators/SKILL.md – hierarchie, migrace, fallback vzory
• Page Objects: page-objects/SKILL.md – BasePage metody, EXAMPLE.md – kod vzory

Rychly pruchod (minimalni review)

1. Lokatory: Zadne CSS/XPath?
2. Cekani: Zadne waitForTimeout?
3. POM: Dedi BasePage, pouziva jeho metody?
4. Credentials: Z env/constants?
5. Assertions: Jen v testech, ne v POM?
6. Import: Z fixtures/page?