mirror of
https://gitlab.archlinux.org/archlinux/aurweb.git
synced 2025-02-03 10:43:03 +01:00
a5943bf2ad
For SQLAlchemy to automatically understand updates from the
external world, it must use an `autocommit=True` in its session.
This change breaks how we were using commit previously, as
`autocommit=True` causes SQLAlchemy to commit when a
SessionTransaction context hits __exit__.
So, a refactoring was required of our tests: All usage of
any `db.{create,delete}` must be called **within** a
SessionTransaction context, created via new `db.begin()`.
From this point forward, we're going to require:
```
with db.begin():
db.create(...)
db.delete(...)
db.session.delete(object)
```
With this, we now get external DB modifications automatically
without reloading or restarting the FastAPI server, which we
absolutely need for production.
Signed-off-by: Kevin Morris <kevr@0cost.org>
73 lines
2.3 KiB
Python
73 lines
2.3 KiB
Python
""" Test our Session model. """
|
|
from datetime import datetime
|
|
from unittest import mock
|
|
|
|
import pytest
|
|
|
|
from aurweb import db
|
|
from aurweb.models.account_type import AccountType
|
|
from aurweb.models.session import Session, generate_unique_sid
|
|
from aurweb.models.user import User
|
|
from aurweb.testing import setup_test_db
|
|
|
|
account_type = user = session = None
|
|
|
|
|
|
@pytest.fixture(autouse=True)
|
|
def setup():
|
|
global account_type, user, session
|
|
|
|
setup_test_db("Users", "Sessions")
|
|
|
|
account_type = db.query(AccountType,
|
|
AccountType.AccountType == "User").first()
|
|
with db.begin():
|
|
user = db.create(User, Username="test", Email="test@example.org",
|
|
ResetKey="testReset", Passwd="testPassword",
|
|
AccountType=account_type)
|
|
|
|
with db.begin():
|
|
session = db.create(Session, UsersID=user.ID, SessionID="testSession",
|
|
LastUpdateTS=datetime.utcnow().timestamp())
|
|
|
|
|
|
def test_session():
|
|
assert session.SessionID == "testSession"
|
|
assert session.UsersID == user.ID
|
|
|
|
|
|
def test_session_cs():
|
|
""" Test case sensitivity of the database table. """
|
|
with db.begin():
|
|
user2 = db.create(User, Username="test2", Email="test2@example.org",
|
|
ResetKey="testReset2", Passwd="testPassword",
|
|
AccountType=account_type)
|
|
|
|
with db.begin():
|
|
session_cs = db.create(Session, UsersID=user2.ID,
|
|
SessionID="TESTSESSION",
|
|
LastUpdateTS=datetime.utcnow().timestamp())
|
|
assert session_cs.SessionID == "TESTSESSION"
|
|
assert session.SessionID == "testSession"
|
|
|
|
|
|
def test_session_user_association():
|
|
# Make sure that the Session user attribute is correct.
|
|
assert session.User == user
|
|
|
|
|
|
def test_generate_unique_sid():
|
|
# Mock up aurweb.models.session.generate_sid by returning
|
|
# sids[i % 2] from 0 .. n. This will swap between each sid
|
|
# between each call.
|
|
sids = ["testSession", "realSession"]
|
|
i = 0
|
|
|
|
def mock_generate_sid(length):
|
|
nonlocal i
|
|
sid = sids[i % 2]
|
|
i += 1
|
|
return sid
|
|
|
|
with mock.patch("aurweb.util.make_random_string", mock_generate_sid):
|
|
assert generate_unique_sid() == "realSession"
|