Skip to content

Unittest pytest example - #114

Open
VincentJilesen wants to merge 15 commits into
betafrom
unittest-pytest-example
Open

Unittest pytest example#114
VincentJilesen wants to merge 15 commits into
betafrom
unittest-pytest-example

Conversation

@VincentJilesen

Copy link
Copy Markdown
Collaborator

Een paar voorbeelden van unittests met pytest.

@VincentJilesen VincentJilesen self-assigned this Aug 21, 2026
@chrispijo
chrispijo self-requested a review August 21, 2026 09:36

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

De test controleert de functie created_project() in project.py

In de test ontbreekt de keuze applicatie afsluiten. Ook wordt het standaardgeval waarbij geen bekende keuze wordt teruggegeven niet getest. Lijkt me geen probleem

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Het is een meerkeuze vraag dus er is geen onbekende keuze mogelijk.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Implementatie van Mock is ok. Alleen de routeringslogica wordt getest. De echte request_binnenteenlijn_filepath en de import/write-logica blijven nog ongetest.

@skapinga
skapinga self-requested a review September 10, 2026 09:49

@skapinga skapinga left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

goede implementatie om de scope van de tests uit te breiden

@skapinga
skapinga removed the request for review from chrispijo September 10, 2026 09:52
@skapinga
skapinga marked this pull request as ready for review September 10, 2026 09:54
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