-
Notifications
You must be signed in to change notification settings - Fork 1
Frontend component testen #526
base: develop
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #526 +/- ##
========================================
Coverage 94.44% 94.44%
========================================
Files 39 39
Lines 1134 1134
Branches 238 238
========================================
Hits 1071 1071
Misses 57 57
Partials 6 6 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Arne had wat testen verwijderd omdat die faalden, bij mij falen die echter niet, dus ik ga ze er even terug insteken en als de ci faalt haal ik ze er terug uit. |
…rottoir-1 into web/test/components
Er zijn inderdaad een aantal tests die falen, (niet alle die Arne verwijderd had). Ik vind dit extreem raar, want als ik ze lokaal uitvoer slagen ze wel. |
Nu snap ik er helemaal niets meer van, ik heb de testen verwijderd die faalden, en nu falen andere testen die daarnet slaagden. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testen slagen niet, en zoals aangegeven in #376 veel te laat. Indien je dit nog werkende krijgt wil ik dit eventueel morgen (22 mei) bekijken, maar voor vandaag laat ik het hierbij.
Beschrijving
Ik heb de component testen en e2e testen op aparte branches gezet, bij de vorige pr stonden die nog samen, maar dat was onoverzichtelijk.
Motivatie en context
Testmethode
Screenshots (indien van toepassing):
Aanpassingen
Checklist