Merge pull request #76 from hiterm/switch-test-by-feature

Switch tests by feature flag

.github/workflows/ci.yml

@@ -25,7 +25,7 @@ jobs:
       - run: pg_isready
       - run: sudo -u postgres psql --command="CREATE USER bookshelf WITH SUPERUSER PASSWORD 'password'" --command="\du" postgres
       - run: sudo -u postgres createdb --owner=bookshelf bookshelf
-      - run: cargo test -- --include-ignored
+      - run: cargo test --all-features
 
   test-image-building:
     name: Test Image Building

Cargo.toml

@@ -4,6 +4,9 @@ version = "2.0.12"
 edition = "2021"
 default-run = "bookshelf-api"
 
+[features]
+test-with-database = []
+
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 
 [dependencies]

src/infrastructure/author_repository.rs

@@ -154,6 +154,7 @@ impl InternalAuthorRepository {
     }
 }
 
+#[cfg(feature = "test-with-database")]
 #[cfg(test)]
 mod tests {
 
@@ -165,7 +166,6 @@ mod tests {
     use super::*;
 
     #[sqlx::test]
-    #[ignore] // Depends on PostgreSQL
     async fn create_and_find_by_id(pool: PgPool) -> anyhow::Result<()> {
         let user_repository = PgUserRepository::new(pool.clone());
         let author_repository = PgAuthorRepository::new(pool.clone());
@@ -185,7 +185,6 @@ mod tests {
     }
 
     #[sqlx::test]
-    #[ignore] // Depends on PostgreSQL
     async fn create_and_find_all(pool: PgPool) -> anyhow::Result<()> {
         let user_repository = PgUserRepository::new(pool.clone());
         let author_repository = PgAuthorRepository::new(pool.clone());
@@ -211,7 +210,6 @@ mod tests {
     }
 
     #[sqlx::test]
-    #[ignore] // Depends on PostgreSQL
     async fn create_and_find_by_ids_as_hash_map(pool: PgPool) -> anyhow::Result<()> {
         let user_repository = PgUserRepository::new(pool.clone());
         let author_repository = PgAuthorRepository::new(pool.clone());

src/infrastructure/book_repository.rs

@@ -400,6 +400,7 @@ impl InternalBookRepository {
     }
 }
 
+#[cfg(feature = "test-with-database")]
 #[cfg(test)]
 mod tests {
 
@@ -423,7 +424,6 @@ mod tests {
     };
 
     #[sqlx::test]
-    #[ignore] // Depends on PostgreSQL
     async fn test_create_and_find_by_id(pool: PgPool) -> anyhow::Result<()> {
         let user_repository = PgUserRepository::new(pool.clone());
         let author_repository = PgAuthorRepository::new(pool.clone());
@@ -445,7 +445,6 @@ mod tests {
     }
 
     #[sqlx::test]
-    #[ignore] // Depends on PostgreSQL
     async fn test_create_and_find_all(pool: PgPool) -> anyhow::Result<()> {
         let user_repository = PgUserRepository::new(pool.clone());
         let author_repository = PgAuthorRepository::new(pool.clone());
@@ -478,7 +477,6 @@ mod tests {
     }
 
     #[sqlx::test]
-    #[ignore] // Depends on PostgreSQL
     async fn test_update(pool: PgPool) -> anyhow::Result<()> {
         // setup
         let user_repository = PgUserRepository::new(pool.clone());
@@ -512,7 +510,6 @@ mod tests {
     }
 
     #[sqlx::test]
-    #[ignore] // Depends on PostgreSQL
     async fn test_delete(pool: PgPool) -> anyhow::Result<()> {
         let user_repository = PgUserRepository::new(pool.clone());
         let author_repository = PgAuthorRepository::new(pool.clone());

src/infrastructure/user_repository.rs

@@ -63,12 +63,12 @@ impl InternalUserRepository {
     }
 }
 
+#[cfg(feature = "test-with-database")]
 #[cfg(test)]
 mod tests {
     use super::*;
 
     #[sqlx::test]
-    #[ignore] // Depends on PostgreSQL
     async fn test_user_repository(pool: PgPool) -> anyhow::Result<()> {
         dotenv::dotenv().ok();