mirror of
https://github.com/simonw/sqlite-utils.git
synced 2026-07-23 17:34:32 +02:00
adfcbb4731Database() now accepts all three Python transaction handling modes and behaves identically in each. For autocommit=False connections - where the driver holds an implicit transaction open at all times - the library tracks transaction ownership itself: - A new _explicit_transaction flag distinguishes transactions opened with begin()/atomic() from the driver's implicit one, exposed as a new db.in_transaction property (conn.in_transaction is always True in this mode) - begin() claims the driver's implicit transaction instead of executing BEGIN, which the driver would reject; BEGIN/COMMIT/ ROLLBACK passed to db.execute() are routed through begin()/commit()/ rollback() - Writes outside a user transaction commit the implicit transaction immediately, preserving the library's auto-commit contract - Row-returning statements outside a transaction fetch eagerly and commit, so the implicit read transaction does not hold a shared lock that blocks writes from other connections - PRAGMA and VACUUM run in temporary driver autocommit mode (ensure_autocommit_on() now flips conn.autocommit), since PRAGMAs are silently ignored and VACUUM refused inside the implicit transaction A new pytest --sqlite-autocommit-false option runs the entire suite in this mode, wired into CI alongside --sqlite-autocommit. Tests that asserted on conn.in_transaction or wrote through db.conn without committing now use the mode-aware library API instead. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PFkrS2nuP8mo1jmT594VCd
59 lines
1.7 KiB
YAML
59 lines
1.7 KiB
YAML
name: Test
|
|
|
|
on: [push, pull_request]
|
|
|
|
env:
|
|
FORCE_COLOR: 1
|
|
|
|
jobs:
|
|
test:
|
|
runs-on: ${{ matrix.os }}
|
|
strategy:
|
|
matrix:
|
|
python-version: ["3.10", "3.11", "3.12", "3.13", "3.14", "3.15-dev"]
|
|
numpy: [0, 1]
|
|
os: [ubuntu-latest, macos-latest, windows-latest, macos-14]
|
|
steps:
|
|
- uses: actions/checkout@v7
|
|
- name: Set up Python ${{ matrix.python-version }}
|
|
uses: actions/setup-python@v6
|
|
with:
|
|
python-version: ${{ matrix.python-version }}
|
|
allow-prereleases: true
|
|
cache: pip
|
|
cache-dependency-path: pyproject.toml
|
|
- name: Install dependencies
|
|
run: |
|
|
pip install . --group dev
|
|
- name: Optionally install numpy
|
|
if: matrix.numpy == 1
|
|
run: pip install numpy
|
|
- name: Install SpatiaLite
|
|
if: matrix.os == 'ubuntu-latest'
|
|
run: sudo apt-get install libsqlite3-mod-spatialite
|
|
- name: Build extension for --load-extension test
|
|
if: matrix.os == 'ubuntu-latest'
|
|
run: |-
|
|
(cd tests && gcc ext.c -fPIC -shared -o ext.so && ls -lah)
|
|
- name: Run tests
|
|
run: |
|
|
pytest -v
|
|
- name: Run autocommit tests just on 3.14/Ubuntu
|
|
if: matrix.os == 'ubuntu-latest' && matrix.python-version == '3.14'
|
|
run: |
|
|
pytest --sqlite-autocommit
|
|
pytest --sqlite-autocommit-false
|
|
- name: run mypy
|
|
run: mypy sqlite_utils tests
|
|
- name: run flake8
|
|
run: flake8
|
|
- name: run ty
|
|
if: matrix.os != 'windows-latest' && matrix.python-version == '3.14'
|
|
run: |
|
|
pip install uv
|
|
uv run ty check sqlite_utils
|
|
- name: Check formatting
|
|
run: black . --check
|
|
- name: Check if cog needs to be run
|
|
run: |
|
|
cog --check --diff README.md docs/*.rst
|