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

Added Unit test, folder structure, naming #25 #22 #28

Merged
merged 2 commits into from
Sep 29, 2021
Merged

Conversation

rodeknopje
Copy link
Contributor

Added a xUnit project to write unit tests in.
There are now 2 projects in the solution Dartin.Application and Dartin.UnitTests
image

Unit test can be written like this, functionality and models from the Application can be used in the tests.
image

To find more info about writing xunit test.
https://xunit.net/docs/getting-started/netcore/visual-studio

@rodeknopje rodeknopje requested review from boele and Jacsquare and removed request for boele and Jacsquare September 29, 2021 09:09
@rodeknopje rodeknopje self-assigned this Sep 29, 2021
@Hoekstraa Hoekstraa requested review from Hoekstraa and boele and removed request for boele and Hoekstraa September 29, 2021 09:12
Copy link
Contributor

@Hoekstraa Hoekstraa left a comment

Choose a reason for hiding this comment

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

Helemaal bole.

@tr4wzified
Copy link
Contributor

Goedgekeurd

@tr4wzified tr4wzified requested review from tr4wzified and removed request for boele September 29, 2021 09:18
Copy link
Contributor

@tr4wzified tr4wzified left a comment

Choose a reason for hiding this comment

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

Goede / logische rename van bestanden, logische opzet tests in ander project

@rodeknopje rodeknopje requested a review from boele September 29, 2021 09:24
Copy link
Contributor

@boele boele left a comment

Choose a reason for hiding this comment

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

Ziet er goed uit.

@rodeknopje rodeknopje merged commit 5412c0b into main Sep 29, 2021
@rodeknopje rodeknopje deleted the unit-tests branch September 29, 2021 09:25
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.

[Feature] Voeg unit test project toe [Feature] Verander naam van het project
4 participants