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/simplify visual regression testing #356

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

remypar5
Copy link
Contributor

@remypar5 remypar5 commented Jan 9, 2025

Contents

In de laatste weken waarin we de visuele regressie tests voor (vrijwel) alle react componenten hebben gemaakt zagen we veel repetitie en copy/pasting. Wat we kopiëerden was voornamelijk de structuur van een VR-test. Dit zorgt er voor dat alle tests er het zelfde uit zien, los van de de daadwerkelijke inhoud.

Om dit te versimpelen heb ik die structuur in een React component gestopt waardoor het makkelijker wordt om deze structuur te hergebruiken en onderhouden.

Voor de reviewers

  • Laat mij weten als je de layout van de tests anders wilt zien. We kunnen bijvoorbeeld de verschillende modes naast elkaar tonen i.p.v. onder elkaar.
  • Met deze nieuwe opzet kun je moeiteloos nieuwe producten aan je testcase toevoegen door products=[...] mee te geven aan het VisualRegressionTest component.
    Let op: het logius product wordt altijd getest en is altijd het eerste product dat getoond wordt.
  • Als jullie ervoor kiezen deze nieuwe structuur aan te nemen zal ik alle huidige visuele tests vervangen met de nieuwe

Checklist

  • [ ] New features/components and bugfixes are covered by tests
  • [ ] Changesets are created
  • [ ] Definition of Done is checked

Copy link

vercel bot commented Jan 9, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
lux ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 21, 2025 8:44am

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.

1 participant