full-stack-fastapi-template

full-stack-fastapi-template copied to clipboard

✅ Fix incorrect mocking in unit tests (issue #1780)

2

Fix issue #1780: the 2 `test_init_successful_connection` tests were basically noops due to misuse of MagicMock. The errors went undetected until python 3.12+ tightened up the MagicMock implementation.

vicaya

The 2 `test_init_successful_connection` tests in `backend/app/tests/scripts/test_\*_pre_start.py` are basically noops, due to misuses of MagicMock(return_value=True). The 2 unit tests would error out due to unknown attributes (`called_once_with`) when running `bash scripts/test.sh`...

vicaya

Out-of-the-box the username/email field is case sensitive. A single email could create many users by changing the capitalization. i.e. [email protected] + [email protected] I think the best way to handle this...

ghost

related also to #24. At first, the tests/api/* are not using fixture ` server_api ` ``` from app.tests.utils.utils import get_server_api def test_celery_worker_test(superuser_token_headers): server_api = get_server_api() ``` shall be ``` def...

skrchnavy