Merge pull request #7 from tompro/develop

Fixed integration tests missing redis connection by using IP 0.0.0.0 instead of localhost. Something changed with GH Actions docker networking?

Author: tompro

.github/workflows/ci.yml

@@ -49,6 +49,9 @@ jobs:
         with:
           command: test
           args: --all-features
+        env:
+          REDIS_HOST: 0.0.0.0
+          REDIS_PORT: 6379
 
   fmt:
     name: Rustfmt

tests/async_command_tests/mod.rs

@@ -9,11 +9,12 @@ use redis_ts::{
     TsAggregationType, TsDuplicatePolicy, TsFilterOptions, TsInfo, TsMget, TsMrange, TsOptions,
     TsRange,
 };
+use std::env;
 use std::thread;
 use std::time::{Duration, SystemTime, UNIX_EPOCH};
 
 async fn get_con() -> Connection {
-    let client = redis::Client::open("redis://localhost/").unwrap();
+    let client = redis::Client::open(get_redis_url()).unwrap();
     client.get_async_connection().await.unwrap()
 }
 
@@ -426,3 +427,20 @@ pub async fn ts_queryindex(name: &str) {
         .unwrap();
     assert!(index.contains(&name.to_string()));
 }
+
+fn get_redis_url() -> String {
+    let redis_host_key = "REDIS_HOST";
+    let redis_host_port = "REDIS_PORT";
+
+    let redis_host = match env::var(redis_host_key) {
+        Ok(host) => host,
+        _ => "localhost".to_string(),
+    };
+
+    let redis_port = match env::var(redis_host_port) {
+        Ok(port) => port,
+        _ => "6379".to_string(),
+    };
+
+    format!("redis://{}:{}/", redis_host, redis_port)
+}