Exclude tests {mongo, redis} using features (gluesql#1463)

* build: separate test-redis as a feature * fix: exclude all mongo cases
ChobobDev · Mar 9, 2024 · cd80c03 · cd80c03
1 parent 01dc4bc
commit cd80c03
Show file tree

Hide file tree

Showing 9 changed files with 19 additions and 5 deletions.
diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml
@@ -68,7 +68,7 @@ jobs:
           cd pkg/rust
           cargo test --lib --bins --tests --examples --verbose --no-default-features --features "memory-storage sled-storage"
           cd ../../
- 
+
   run_storage_tests:
     name: Run storage tests
     runs-on: ubuntu-latest
@@ -112,7 +112,7 @@ jobs:
           redis-version: 6
       - run: |
           cd storages/redis-storage
-          cargo test --verbose
+          cargo test --verbose --features test-redis
 
   run_examples:
     name: Run examples

diff --git a/storages/mongo-storage/tests/mongo_indexes.rs b/storages/mongo-storage/tests/mongo_indexes.rs
@@ -1,3 +1,5 @@
+#![cfg(feature = "test-mongo")]
+
 use {
     bson::{doc, Document},
     gluesql_core::prelude::{Glue, Payload},

diff --git a/storages/mongo-storage/tests/mongo_storage.rs b/storages/mongo-storage/tests/mongo_storage.rs
@@ -1,3 +1,5 @@
+#![cfg(feature = "test-mongo")]
+
 use {
     async_trait::async_trait, gluesql_core::prelude::Glue, gluesql_mongo_storage::MongoStorage,
     test_suite::*,
@@ -25,5 +27,4 @@ impl Tester<MongoStorage> for MongoTester {
     }
 }
 
-#[cfg(feature = "test-mongo")]
 generate_store_tests!(tokio::test, MongoTester);
diff --git a/storages/mongo-storage/tests/mongo_types.rs b/storages/mongo-storage/tests/mongo_types.rs
@@ -1,3 +1,5 @@
+#![cfg(feature = "test-mongo")]
+
 use {
     bson::{doc, Bson},
     gluesql_core::prelude::{Glue, Payload, Value},

diff --git a/storages/redis-storage/Cargo.toml b/storages/redis-storage/Cargo.toml
@@ -24,3 +24,6 @@ futures = "0.3"
 test-suite.workspace = true
 tokio = { version = "1", features = ["rt", "macros"] }
 toml = "0.8.6"
+
+[features]
+test-redis = []
diff --git a/storages/redis-storage/tests/redis_errors.rs b/storages/redis-storage/tests/redis_errors.rs
@@ -1,3 +1,5 @@
+#![cfg(feature = "test-redis")]
+
 use {
     gluesql_core::prelude::{Error, Payload},
     gluesql_redis_storage::RedisStorage,

diff --git a/storages/redis-storage/tests/redis_reconnect.rs b/storages/redis-storage/tests/redis_reconnect.rs
@@ -1,3 +1,5 @@
+#![cfg(feature = "test-redis")]
+
 use {
     gluesql_core::prelude::{Payload, Value},
     gluesql_redis_storage::RedisStorage,

diff --git a/storages/redis-storage/tests/redis_store.rs b/storages/redis-storage/tests/redis_store.rs
@@ -1,3 +1,5 @@
+#![cfg(feature = "test-redis")]
+
 use {
     async_trait::async_trait,
     gluesql_core::prelude::Glue,
@@ -51,9 +53,7 @@ impl Tester<RedisStorage> for RedisStorageTester {
 }
 
 generate_store_tests!(tokio::test, RedisStorageTester);
-
 generate_alter_table_tests!(tokio::test, RedisStorageTester);
-
 generate_metadata_table_tests!(tokio::test, RedisStorageTester);
 
 macro_rules! exec {

diff --git a/storages/redis-storage/tests/redis_table.rs b/storages/redis-storage/tests/redis_table.rs
@@ -1,3 +1,5 @@
+#![cfg(feature = "test-redis")]
+
 use {
     gluesql_core::prelude::{Glue, Payload, Value},
     gluesql_redis_storage::RedisStorage,