Sometimes, the tests don't work in the CI. This is because there are no checks on whether the server has got time to launch properly if BLINDAI_TEST_NO_LAUNCH_SERVER is set (this env var is set on the CI because the CI launches the server itself, using docker)
This caused the tests to sometimes run before the app was running.
I have also changed some other things, such as adding a bit of sleep, fixing the timeout (which, turns out, wasn't really working) and checking for the right port. The tests should be much much more reliable now on the CI.
Related Issue
None
Type of change
[ ] This change requires a documentation update
[ ] This change affects the client
[ ] This change affects the server
[ ] This change affects the API
[ ] This change only concerns the documentation
How Has This Been Tested?
Run the CI, and run the tests locally
Checklist:
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I have updated the documentation according to my changes
Description
Sometimes, the tests don't work in the CI. This is because there are no checks on whether the server has got time to launch properly if
BLINDAI_TEST_NO_LAUNCH_SERVER
is set (this env var is set on the CI because the CI launches the server itself, using docker) This caused the tests to sometimes run before the app was running.I have also changed some other things, such as adding a bit of
sleep
, fixing the timeout (which, turns out, wasn't really working) and checking for the right port. The tests should be much much more reliable now on the CI.Related Issue
None
Type of change
How Has This Been Tested?
Run the CI, and run the tests locally
Checklist: