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

Resume work #211

Closed ehmatthes closed 1 year ago

ehmatthes commented 1 year ago

I had to step away from this project, but I'd like to resume the push to a 1.0 release. I believe the biggest issue is ensuring correct functionality on Windows for all three platforms. I haven't tested on Linux, but there's enough similarity with macOS that any Linux issues should be quick to resolve.

Initial work:

ehmatthes commented 1 year ago

Unit tests

Okay, test infrastructure is pretty good to work with. I can see from the output the path to the test requirements file that was incorrect, and the reference file is in the unit_tests/platforms/heroku/reference_files/ directory. The path to the test file doesn't work, though, because other tests after the failing test wrote over that file.

Fix: run pytest -x, which should end on that failed test, and I should be able to look at the incorrect requirements file. To be even more efficient, only test the platform that's generating the failure: pytest -x platforms/heroku.

Failed because Poetry is more precise about versions, I believe.

ehmatthes commented 1 year ago

Integration tests