feat(sea-orm): remove Sea ORM CLI from migration tests (#174)

Author: DanielleHuisman

.github/workflows/ci.yml

@@ -82,11 +82,5 @@ jobs:
                   components: clippy, rustfmt
                   target: wasm32-unknown-unknown
 
-            - name: Install Cargo Binary Install
-              uses: cargo-bins/cargo-binstall@main
-
-            - name: Install crates
-              run: cargo binstall -y sea-orm-cli
-
             - name: Test
               run: cargo test --locked --release

Cargo.lock

@@ -16,7 +16,7 @@ shield-axum = { workspace = true, features = ["utoipa"] }
 shield-memory = { workspace = true, features = ["method-oidc"] }
 shield-oidc = { workspace = true, features = ["native-tls"] }
 time = "0.3.37"
-tokio = { workspace = true, features = ["rt-multi-thread"] }
+tokio = { workspace = true, features = ["macros", "rt-multi-thread"] }
 tower-sessions.workspace = true
 tracing.workspace = true
 tracing-subscriber.workspace = true

examples/axum/Cargo.toml

@@ -35,6 +35,9 @@ shield-dioxus.workspace = true
 shield-dioxus-axum = { workspace = true, optional = true }
 shield-memory = { workspace = true, optional = true }
 shield-oidc = { workspace = true, features = ["native-tls"], optional = true }
-tokio = { workspace = true, features = ["rt-multi-thread"], optional = true }
+tokio = { workspace = true, features = [
+    "macros",
+    "rt-multi-thread",
+], optional = true }
 tower-sessions = { workspace = true, optional = true }
 tracing.workspace = true

examples/dioxus-axum/Cargo.toml

@@ -54,7 +54,10 @@ shield-leptos-axum = { workspace = true, features = [
 shield-memory = { workspace = true, optional = true }
 shield-oidc = { workspace = true, features = ["native-tls"], optional = true }
 time = "0.3.37"
-tokio = { workspace = true, features = ["rt-multi-thread"], optional = true }
+tokio = { workspace = true, features = [
+    "macros",
+    "rt-multi-thread",
+], optional = true }
 tower-sessions = { workspace = true, optional = true }
 tracing.workspace = true
 tracing-subscriber.workspace = true

examples/leptos-axum/Cargo.toml

@@ -34,6 +34,9 @@ shield-dioxus.workspace = true
 shield-dioxus-axum = { workspace = true, optional = true }
 shield-memory = { workspace = true, optional = true }
 shield-workos = { workspace = true, optional = true }
-tokio = { workspace = true, features = ["rt-multi-thread"], optional = true }
+tokio = { workspace = true, features = [
+    "macros",
+    "rt-multi-thread",
+], optional = true }
 tower-sessions = { workspace = true, optional = true }
 tracing.workspace = true

examples/workos/Cargo.toml

@@ -42,3 +42,10 @@ shield-oauth = { workspace = true, optional = true }
 shield-oidc = { workspace = true, optional = true }
 # shield-webauthn = { workspace = true, optional = true }
 utoipa = { workspace = true, optional = true }
+
+[dev-dependencies]
+tokio = { workspace = true, features = [
+    "macros",
+    "rt-multi-thread",
+    "test-util",
+] }

packages/storage/shield-sea-orm/Cargo.toml

@@ -1,12 +1,6 @@
-use std::{
-    env,
-    path::{Path, PathBuf},
-    process::Command,
-};
-
-fn example_path() -> PathBuf {
-    Path::new(env!("CARGO_MANIFEST_DIR")).join("../../../examples/sea-orm")
-}
+use sea_orm::Database;
+use sea_orm_migration::migrator::MigratorTrait;
+use shield_sea_orm::migrations::Migrator;
 
 const BACKENDS: &[(&str, &str)] = &[
     ("mysql", "mysql://shield:shield@localhost:13306/shield"),
@@ -17,49 +11,26 @@ const BACKENDS: &[(&str, &str)] = &[
     ("sqlite", "sqlite:///tmp/shield-seaorm.sqlite?mode=rwc"),
 ];
 
-#[test]
-pub fn migrations() {
+#[tokio::test]
+async fn migrations() {
     for (backend, url) in BACKENDS {
-        // Check up migrations
-        assert!(
-            Command::new("sea-orm-cli")
-                .arg("migrate")
-                .arg("fresh")
-                .arg("-u")
-                .arg(url)
-                .arg("-d")
-                .arg(example_path())
-                .status()
-                .unwrap_or_else(|_| panic!("{backend} up migrations should succeed."))
-                .success()
-        );
+        let database = Database::connect(url.to_owned())
+            .await
+            .unwrap_or_else(|err| panic!("Connect to backend `{backend}` failed: {err}"));
+
+        // Up migrations
+        Migrator::fresh(&database)
+            .await
+            .unwrap_or_else(|err| panic!("Up migrations for backend `{backend}` failed: {err}"));
 
-        // Check down migrations
-        assert!(
-            Command::new("sea-orm-cli")
-                .arg("migrate")
-                .arg("refresh")
-                .arg("-u")
-                .arg(url)
-                .arg("-d")
-                .arg(example_path())
-                .status()
-                .unwrap_or_else(|_| panic!("{backend} down migrations should succeed."))
-                .success()
-        );
+        // Down migrations
+        Migrator::refresh(&database)
+            .await
+            .unwrap_or_else(|err| panic!("Down migrations for backend `{backend}` failed: {err}"));
 
         // Cleanup
-        assert!(
-            Command::new("sea-orm-cli")
-                .arg("migrate")
-                .arg("reset")
-                .arg("-u")
-                .arg(url)
-                .arg("-d")
-                .arg(example_path())
-                .status()
-                .unwrap_or_else(|_| panic!("{backend} cleanup should succeed."))
-                .success()
-        );
+        Migrator::reset(&database)
+            .await
+            .unwrap_or_else(|err| panic!("Cleanup for backend `{backend}` failed: {err}"));
     }
 }