Refactor tests to use sqlx::test and improve test isolation

.github/actions/postgres/action.yml

@@ -13,6 +13,10 @@ runs:
         pg_isready
         sudo -u postgres psql -c "ALTER USER postgres PASSWORD 'password'"
 
-    - name: Setup database
+    # `#[sqlx::test]` gives each test its own database and applies the
+    # migrations to it, so this only has to exist for sqlx to connect to and
+    # keep its bookkeeping in. No migrations are run here, and sqlx-cli is not
+    # needed.
+    - name: Create database
       shell: bash
-      run: cargo sqlx database setup
+      run: sudo -u postgres createdb sqlxmq

.github/workflows/toolchain.yml

@@ -10,7 +10,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@stable
       - uses: Swatinem/rust-cache@v2
-      - run: cargo check
+      - run: cargo check --workspace --all-targets
 
   fmt:
     name: Rustfmt
@@ -20,7 +20,7 @@ jobs:
       - uses: dtolnay/rust-toolchain@stable
         with:
           components: rustfmt
-      - run: cargo fmt -- --check
+      - run: cargo fmt --all -- --check
 
   clippy:
     name: Clippy
@@ -31,7 +31,7 @@ jobs:
         with:
           components: clippy
       - uses: Swatinem/rust-cache@v2
-      - run: cargo clippy --all-targets -- -D warnings
+      - run: cargo clippy --workspace --all-targets -- -D warnings
 
   test:
     name: Test
@@ -42,7 +42,6 @@ jobs:
       - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@stable
       - uses: Swatinem/rust-cache@v2
-      - run: cargo install sqlx-cli --locked
       - uses: ./.github/actions/postgres
       - run: cargo test -- --nocapture
 
@@ -55,7 +54,6 @@ jobs:
       - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@nightly
       - uses: Swatinem/rust-cache@v2
-      - run: cargo install sqlx-cli --locked
       - uses: ./.github/actions/postgres
       - run: cargo test -- --nocapture
 

Cargo.toml

@@ -36,3 +36,7 @@ dotenvy = "0.15.3"
 pretty_env_logger = "0.4.0"
 futures = "0.3.13"
 tokio = { version = "1", features = ["full"] }
+# `migrate` is only needed by `#[sqlx::test]`, which gives each test its own
+# freshly migrated database. Keeping it in dev-dependencies means it is not
+# enabled for consumers of the library.
+sqlx = { version = "0.9", default-features = false, features = ["migrate"] }

sqlxmq_stress/Cargo.toml

@@ -10,7 +10,7 @@ edition = "2018"
 sqlxmq = { path = ".." }
 tokio = { version = "1.4.0", features = ["full"] }
 dotenvy = "0.15"
-sqlx = "0.8"
+sqlx = { version = "0.9", default-features = false, features = ["postgres"] }
 serde = "1.0.125"
 lazy_static = "1.4.0"
 futures = "0.3.13"

sqlxmq_stress/migrations

@@ -0,0 +1 @@
+../migrations