app-generator / sample-django-best-practices

MIT License
2 stars 1 forks source link

Unitary Tests - No tests are executed #7

Closed app-generator closed 2 years ago

app-generator commented 2 years ago

Hello @koladev32

Please add a minimal test suite for the existing features:

All test data should be deleted at the end of the test suite.

P.S. The tests section should be also mentioned on the README.

Ty!

koladev32 commented 2 years ago

Hey @app-generator

For these tests, I am thinking about testing the forms to make sure these functionalities are working.

What do you think?

app-generator commented 2 years ago

Hello @koladev32

The goal is to cover the use case and maximize the code coverage. Please proceed with your proposal.

Ty!

koladev32 commented 2 years ago

Hey @app-generator

Added some test cases here

app-generator commented 2 years ago

Tested with the latest version - No tests are executed:

(env) PS D:\work\repo-samples\sample-django-best-practices> git pull
Already up to date.
(env) PS D:\work\repo-samples\sample-django-best-practices> 
(env) PS D:\work\repo-samples\sample-django-best-practices>
(env) PS D:\work\repo-samples\sample-django-best-practices>
(env) PS D:\work\repo-samples\sample-django-best-practices> python manage.py test
System check identified no issues (0 silenced).

----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK
(env) PS D:\work\repo-samples\sample-django-best-practices> 
app-generator commented 2 years ago

@koladev32 - any progress here? ty!

koladev32 commented 2 years ago

Ah yes. Actually, pytest was used to write the tests. So you need to use the pytest command instead. Might add it to the README.md.

app-generator commented 2 years ago

Cool, please update the README with a small section that mentions the command + executed test suite

app-generator commented 2 years ago

ty for your work @koladev32