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

Finish adding unit tests for basic assembly app #5

Open
Benedict-Carling opened this issue Aug 7, 2020 · 1 comment
Open

Finish adding unit tests for basic assembly app #5

Benedict-Carling opened this issue Aug 7, 2020 · 1 comment
Labels
help wanted Extra attention is needed unit tests

Comments

@Benedict-Carling
Copy link
Collaborator

Hello,

this issue is to finish the unit tests inside repositry

basic_assembly/tests/...

we have unit tests set up to run on every pull request and everytime someone runs the command specified in the read me

python manage.py test

I have poorly written some some unit tests for the tests_urls.py file and a single unit test in tests_views.py file, these files or located within basic_assembly/tests folder

how our programme finds unit tests is it looks for folders called tests which contain a init.py file and within files which start with test_... inside there it finds Class which starts with test and runs all "unit test" which are function inside this class which begin with test

A description on how django finds unit tests
https://stackoverflow.com/questions/36546168/how-to-run-all-tests-with-python-manage-py-test-command-in-django

A great youtube series on adding unittest for django
https://www.youtube.com/watch?v=qwypH3YvMKc&list=PLbpAWbHbi5rMF2j5n6imm0enrSD9eQUaM

@Benedict-Carling Benedict-Carling added help wanted Extra attention is needed unit tests labels Aug 7, 2020
@Benedict-Carling Benedict-Carling changed the title Finish adding unit tests for basic assembly branch Finish adding unit tests for basic assembly app Aug 7, 2020
@GabrielleJohnston
Copy link
Contributor

To do:
Check that paths returned by dnabot() are correct, and that error catching is successful

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed unit tests
Projects
None yet
Development

No branches or pull requests

2 participants