Skip to content

Nok et forsøk på visuell regresjonstesting med Storybook - #6026

Draft
piofinn wants to merge 3 commits into
release/nextfrom
storybook-visual-regression-tests
Draft

Nok et forsøk på visuell regresjonstesting med Storybook#6026
piofinn wants to merge 3 commits into
release/nextfrom
storybook-visual-regression-tests

Conversation

@piofinn

@piofinn piofinn commented May 8, 2026

Copy link
Copy Markdown
Contributor

💬 Endringer

Siden det har vært et par tilfeller av visuell regresjon i de siste ukene tok jeg opp igjen visuell regresjonstesting med Storybook. Tweaket litt på oppsettet så det er enda mer robust, og fikk KI til å skrive noen stories som representerer det som allerede ble testet for to av komponentene: CheckboxPanel og CookieConsent.

I tillegg til den visuelle testingen inneholdt de gamle Playwright-testene også noen tester av funksjonalitet/interaktivitet. Det er laget en egen spec.ts-fil for disse testene for hver av de to komponentene. Her skjer også tilgjengelighetstesting med Axe på samme måte som før.

Begge testene bruker altså nå nye stories fra Storybook. Det kan være samme stories for begge typer tester, eller forskjellige for hver, ut fra hva som trengs. Stories som finnes fra før kan potensielt også brukes til disse testene ved å tagge dem med visual.

📋 To Do

  • Her er det laget egne stories for test av hvert color scheme og size. Dette kan vi potensielt legge inn i visual.spec.ts, slik at det gjøres for hver visuelle test (det kan jo styres vha URLen i Storybook). (mulig jeg bare fikser det i denne PRen)
  • Det som trengs av tester må legges til for de andre komponentene
  • Når vi er fornøyd med testene for en komponent kan vi fjerne development-mappen, den gamle spec.ts-fila og mappen med gamle screenshots fra den komponenten.

@piofinn piofinn self-assigned this May 8, 2026
@piofinn
piofinn requested a review from a team as a code owner May 8, 2026 09:19
@piofinn
piofinn requested review from fremtindelise, kristianulv23, lmfaole and zenabii and removed request for a team May 8, 2026 09:19
@fremtind-bot

fremtind-bot commented May 8, 2026

Copy link
Copy Markdown
Collaborator

Hei, @piofinn! Takk for at du bidrar til Jøkul! 🙌
For å sørge for god dokumentasjon og brukeropplevelse er det fint om du husker å gjøre disse tingene:

  • Test endringene dine ♿️
    Sørg for å teste endringene dine for både responsivitet og universell utforming (tastaturnavigasjon, skjermleser, etc.).
  • Skriv endringslogg 📝
    Sørg for at du har dokumentert endringene i et changeset hvis de skal føre til en ny release av @fremtind/jokul.
    false

Hvis du lurer på hvordan du kommer i gang med noe av dette, er det bare å spørre oss i Jøkul-teamet om hjelp! Tagg oss i en kommentar her, eller spør oss på Teams.

@fremtind-bot

fremtind-bot commented May 8, 2026

Copy link
Copy Markdown
Collaborator

Forhåndsvisning: https://jokul.fremtind.no/preview/storybook-visual-regression-tests/
🔍 Commit: ff9d548

Forhåndsvisningen blir tilgjengelig innen et par minutter. Den fjernes automatisk når pull requesten lukkes.

fremtind-bot added a commit that referenced this pull request May 8, 2026
@piofinn

piofinn commented May 8, 2026

Copy link
Copy Markdown
Contributor Author

Hmm.. Hadde håpet at testene skulle bli trigget her. Får evt. prøve å gjøre en ubetydelig endring i en av komponentene og se.

@piofinn
piofinn marked this pull request as draft May 11, 2026 08:12
@piofinn
piofinn force-pushed the storybook-visual-regression-tests branch from 32b037a to 7acddf6 Compare May 11, 2026 10:10
fremtind-bot added a commit that referenced this pull request May 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants