Fix flaky. Workaround for SQLite driver bug returning a result before the commit.

[lizardoluis]

There is a bug on SQLite driver, which makes it return a result before the actual commit.

When using fetch_one() for queries with a RETURNING clause, the query prematurely returns after fetching the first result row without ensuring the SQLite statement is fully completed.

This bug was causing the test test_sqlite_repository_read_only() to fail sometimes when creating a database at

seafowl/src/repository/default.rs

Lines 142 to 143 in 8a47781

	let id = sqlx::query(r#"INSERT INTO database (name) VALUES ($1) RETURNING (id)"#)
	.bind(database_name)

.

In this PR, I implement a workaround by manually starting a transaction before running the query and committing it afterwards.

Flaky failure example:

https://github.com/splitgraph/seafowl/actions/runs/10721496337/job/29730146388

[gruuya]

Great find!

For the sake of simplicity my only recommendation would be to use the tx only for DML using the RETURNING clause (the potentially buggy ones), and leave the rest of the queries as before (the pure read ones such as get_database).

[lizardoluis]

Ok, I removed the tx from the others