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

Web: e2e testen #542

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

Web: e2e testen #542

wants to merge 50 commits into from

Conversation

ArnoutAllaert
Copy link
Contributor

Beschrijving

Toevoeging van e2e testen. Enkel building create en edit zijn nog niet afgewerkt aangezien die pagina nog in pr staat.
Het zal mij echter niet meer lukken om die toe te voegen, want Jozko zijn server ligt juist plat, dus ik kan niet aan een db (docker gaat niet bij mij). Ik wil gerust iemand helpen daarbij als die bvb zijn scherm kan streamen of zo.
Als een test faalt, is dit (hoogst waarschijnlijk) doordat niet de juiste data in de db aanwezig is, bvb een student die een ronde wilt starten die er niet in zit. Ik wou daarvoor kijken dat de juiste data aanwezig is, maar dit bleek wat lastiger dan gedacht. Ik ga kijken om misschien een script toe te voegen daarvoor nog.

Motivatie en context

Testmethode

Screenshots (indien van toepassing):

Aanpassingen

  • Bug fix (non-breaking wijziging die een probleem oplost)
  • New feature (non-breaking wijziging met nieuwe functionaliteit)
  • Breaking change (bestaande functionaliteit breekt door deze aanpassingen)

Checklist

  • De code volgt de stijl en guidelines van dit project.
  • Vereist een aanpassing aan de documentatie.
  • De documentatie is gewijzigd.

@codecov
Copy link

codecov bot commented May 21, 2023

Codecov Report

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

❗ Current head a4e10e5 differs from pull request most recent head 5b077b2. Consider uploading reports for the commit 5b077b2 to get more accurate results

@@           Coverage Diff            @@
##           develop     #542   +/-   ##
========================================
  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 ArnoutAllaert changed the title Web/test/e2e Web: e2e testen May 21, 2023
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.

1 participant