fix(tests): fix two failing tests - missing DB table and MagicMock IP address
- tests/test_database.py: add user_imap_accounts table to the regression test's initial DB setup (migration 022 creates it before rev 026, so it must exist for migration 032's ALTER TABLE to succeed) - tests/test_local_auth.py: configure mock_request.headers.get to return None and client=None so get_client_ip() returns "unknown" instead of an un-serialisable MagicMock that broke the audit_logs INSERT in test_local_login_success and test_local_login_by_email Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>
This commit is contained in:
@@ -613,6 +613,8 @@ async def test_local_login_success(la_session, active_user):
|
||||
mock_request = MagicMock(spec=Request)
|
||||
mock_request.form = AsyncMock(return_value={"username": "activeuser", "password": "password123"})
|
||||
mock_request.session = {}
|
||||
mock_request.headers.get.return_value = None
|
||||
mock_request.client = None
|
||||
|
||||
result = await auth(mock_request, db=la_session)
|
||||
assert result.status_code == 302
|
||||
@@ -634,6 +636,8 @@ async def test_local_login_by_email(la_session, active_user):
|
||||
mock_request = MagicMock(spec=Request)
|
||||
mock_request.form = AsyncMock(return_value={"username": "active@example.com", "password": "password123"})
|
||||
mock_request.session = {}
|
||||
mock_request.headers.get.return_value = None
|
||||
mock_request.client = None
|
||||
|
||||
result = await auth(mock_request, db=la_session)
|
||||
assert result.status_code == 302
|
||||
|
||||
Reference in New Issue
Block a user