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>
41 lines
1,019 B
Python
41 lines
1,019 B
Python
import pytest
|
|
|
|
from sqlalchemy.exc import IntegrityError
|
|
|
|
from aurweb import db
|
|
from aurweb.models.term import Term
|
|
from aurweb.testing import setup_test_db
|
|
|
|
|
|
@pytest.fixture(autouse=True)
|
|
def setup():
|
|
setup_test_db("Terms")
|
|
|
|
yield None
|
|
|
|
# Wipe em out just in case records are leftover.
|
|
setup_test_db("Terms")
|
|
|
|
|
|
def test_term_creation():
|
|
with db.begin():
|
|
term = db.create(Term, Description="Term description",
|
|
URL="https://fake_url.io")
|
|
assert bool(term.ID)
|
|
assert term.Description == "Term description"
|
|
assert term.URL == "https://fake_url.io"
|
|
assert term.Revision == 1
|
|
|
|
|
|
def test_term_null_description_raises_exception():
|
|
with pytest.raises(IntegrityError):
|
|
with db.begin():
|
|
db.create(Term, URL="https://fake_url.io")
|
|
db.rollback()
|
|
|
|
|
|
def test_term_null_url_raises_exception():
|
|
with pytest.raises(IntegrityError):
|
|
with db.begin():
|
|
db.create(Term, Description="Term description")
|
|
db.rollback()
|