search

hackforla / HomeUniteUs

We're working with community non-profits who have a Host Home or empty bedrooms initiative to develop a workflow management tool to make the process scalable (across all providers), reduce institutional bias, and effectively capture data.
https://homeunite.us/
GNU General Public License v2.0
36 stars 21 forks source link

Update connexion to 2.14.2, pytest to 7.4.2, and downgrade werkzeug to 2.2.3 #600

Closed johnwroge closed 8 months ago

johnwroge commented 8 months ago

Closes #580

What changes did you make?

This PR includes the following changes:

· Removes the encoder.py file from the openapi_server directory and removes the encoder use in main, and init test files to eliminate test errors. · Updates version of Connexion from 2.13.1 to 2.14.2 · Updates pytest from 7.3.1 to 7.4.2 · Downgrades Werkzeug from 2.3.7 to 2.2.3 to remove dependency conflicts

Rationale behind the changes?

The rationale behind these changes was to update Connexion and Flask to newer versions and to remove the deprecation warning the team was receiving when starting the virtual environment and running the test suite. Flask version 2.2.5 was unable to be updated due to conflicts with dependencies required for Connexion. Flask will have to wait to be updated until Connexion version 3.0 is released. The Werkzeug version had to be downgraded because Connexion version 2.14.2 requires version < 2.3 while the older version was compatible with 2.3.7. Pytest was updated because there were no breaking changes.

Testing done for these changes

To confirm the test suite still passes, tests were run in the virtual environment in which 27 test cases passed.

What did you learn or can share that is new?(optional)

I learned through the process that when updating to later versions, it's important to check if there are any coexisting dependencies that require versions which are incompatible with each other.

Since this is my first PR please let me know if any additional information is needed for documentation.

johnwroge commented 8 months ago

Python version was incorrect for requirements.txt.