Skip to content
This repository has been archived by the owner on Feb 5, 2024. It is now read-only.

Frontend component testen #526

Open
wants to merge 57 commits into
base: develop
Choose a base branch
from
Open

Frontend component testen #526

wants to merge 57 commits into from

Conversation

ArnoutAllaert
Copy link
Contributor

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

  • New feature (non-breaking wijziging met nieuwe functionaliteit)

Checklist

  • De code volgt de stijl en guidelines van dit project.

@codecov
Copy link

codecov bot commented May 21, 2023

Codecov Report

Merging #526 (9367364) into develop (afd4220) will not change coverage.
The diff coverage is n/a.

❗ Current head 9367364 differs from pull request most recent head 70f7658. Consider uploading reports for the commit 70f7658 to get more accurate results

@@           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

@ArnoutAllaert
Copy link
Contributor Author

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.

@ArnoutAllaert
Copy link
Contributor Author

ArnoutAllaert commented May 21, 2023

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.

@ArnoutAllaert
Copy link
Contributor Author

Nu snap ik er helemaal niets meer van, ik heb de testen verwijderd die faalden, en nu falen andere testen die daarnet slaagden.

Copy link
Contributor

@jenspots jenspots left a 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.

@ArnoutAllaert
Copy link
Contributor Author

ik begrijp echt niet waarom sommige testen falen. Ik heb al mijn cypress data gewist, de testen gerunned en ze slagen allemaal
image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants