Backend tests failing in fresh repository after editing .env

[dupanovic]

First Check

• I added a very descriptive title here.
• I used the GitHub search to find a similar question and didn't find it.
• I searched in the documentation/README.
• I already searched in Google "How to do X" and didn't find any information.
• I already read and followed all the tutorial in the docs/README and didn't find an answer.

Commit to Help

• I commit to help with one of those options 👆

Example Code

- Clone the repo
- Replace `changethis` values in `.env`
- In `~`, run: `docker compose watch`
- In `~/backend`, run:  `bash ./scripts/test.sh`

Description

• Clone the repo
• In ~, run: docker compose watch
• In ~/backend, run: bash ./scripts/test.sh

Result: all test cases pass if env is not edited and has default changethis.

However if I edit the .env and change the values that had changethis set, 15 test cases fail, and the first superuser row is deleted from the users table.

Every error comes from the post request in ~/backend/app/tests/utils/utils.py at L23:

r = client.post(f"{settings.API_V1_STR}/login/access-token", data=login_data)
print(r.json()) # {'detail': 'Incorrect email or password'}

Before running the tests, it is possible to successfully login with the email and password into the dashboard on :5173.

Operating System

Linux

Operating System Details

Debian 12

Python Version

3.11.2

Additional Context

No response

[AleksDominik]

I have the same issue and I think it come from this test .
Once you change the password on .env file it doesn't correspond to the updated password("changethis").
The dummy solution wil be to delete this test.
Or to find a better way to scope the session fixture to avoid the test modification on db.

[cubinet-code]

I just updated the data sent in this test to use the same password:
data = {"new_password": settings.FIRST_SUPERUSER_PASSWORD, "token": token}

[53845714nF]

oh damn, that problem took me 2 hours. And now I find it here in the discussion. Replacing the old password with the same one defeats the purpose of the test, but I'll take it.