ehmatthes / django-simple-deploy

A reusable Django app that configures your project for deployment
BSD 3-Clause "New" or "Revised" License
307 stars 26 forks source link

Ongoing unit test improvements #235

Closed ehmatthes closed 4 months ago

ehmatthes commented 1 year ago

Most of these are left over from 233, which is ready to be closed.

ehmatthes commented 1 year ago

@darjeeling the unit tests are now entirely written in Python. If you have a chance, will you see if they run on your system? I have one system that uses just the macOS Python installer from python.org, and one that uses pyenv. The tests run on both of these systems. I'm curious if you will see any issues. I haven't tried it on Windows yet.

ehmatthes commented 1 year ago

Speeding up tests

darjeeling commented 1 year ago

I tested with 1b75ca4c14906461a6cc2f92a1231422dbe064c4

It looks fine. ( mac M1 latest, python 3.11.3 pyenv virtualenv )

(djangosimpledeploy) ➜  unit_tests git:(main) ✗ pytest
============================================= test session starts ==============================================
platform darwin -- Python 3.11.3, pytest-7.0.1, pluggy-1.0.0
rootdir: /Users/dj/github/darjeeling/django-simple-deploy
collected 93 items

platform_agnostic_tests/test_invalid_cli_commands.py .........                                           [  9%]
platform_agnostic_tests/test_project_inspection.py sss                                                   [ 12%]
platform_agnostic_tests/test_valid_cli_commands.py ...                                                   [ 16%]
platforms/fly_io/test_flyio_config.py ...........................                                        [ 45%]
platforms/heroku/test_heroku_config.py ...........................                                       [ 74%]
platforms/platform_sh/test_platformsh_config.py ........................                                 [100%]

======================================== 90 passed, 3 skipped in 19.12s ========================================
ehmatthes commented 1 year ago

Thanks, that's helpful to know!