Differential D7292
Ensure server tests and fixtures properly clean up the global flask app Authored by olasd on Mar 4 2022, 12:58 PM. Tags None Subscribers None
Details
The global variable vault gets reused across test runs, so it needs to Depends on D7291 pytest with pytest-random-order
Diff Detail
Event TimelineComment Actions Build is green Patch application report for D7292 (id=26394)Could not rebase; Attempt merge onto 252aa1464d... Updating 252aa14..869421e Fast-forward swh/vault/api/server.py | 72 +++++++++++++++++--------------------- swh/vault/tests/test_server.py | 78 +++++++++++++++++++++++------------------- 2 files changed, 74 insertions(+), 76 deletions(-) Changes applied before testcommit 869421e8e7171f2107c54932e686e4db4d138467
Author: Nicolas Dandrimont <nicolas@dandrimont.eu>
Date: Fri Mar 4 12:47:38 2022 +0100
Ensure server tests and fixtures properly clean up the global flask app
The global variable `vault` gets reused across test runs, so it needs to
be reset on fixture/test teardown, or any potential new config may not
be taken into account, which can in turn generate confusing postgresql
connection closures.
commit 2a80c26d2e902984eaef0c1fd3393f3eb73c9c87
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Fri Mar 4 11:08:31 2022 +0100
Rewrite the server to use flask instead of aiohttp
The azure backend cannot be run in the same thread as an asyncio
event loop: https://forge.softwareheritage.org/T3981See https://jenkins.softwareheritage.org/job/DVAU/job/tests-on-diff/221/ for more details. |