Add recording logger for further tests

src/builder.rs

@@ -3,7 +3,7 @@ use crate::constants::SDK_KEY_PROXY_PREFIX;
 use crate::errors::{ClientError, ErrorKind};
 use crate::model::enums::DataGovernance;
 use crate::modes::PollingMode;
-use crate::r#override::FlagOverrides;
+use crate::r#override::{FlagOverrides, OptionalOverrides};
 use crate::{Client, ConfigCache, OverrideBehavior, OverrideDataSource};
 use std::borrow::Borrow;
 use std::time::Duration;
@@ -255,7 +255,9 @@ impl ClientBuilder {
                 "SDK Key cannot be empty".to_owned(),
             ));
         }
-        if !self.is_sdk_key_valid(self.sdk_key.as_str(), self.base_url.is_some()) {
+        if !self.overrides.is_local()
+            && !self.is_sdk_key_valid(self.sdk_key.as_str(), self.base_url.is_some())
+        {
             return Err(ClientError::new(
                 ErrorKind::InvalidSdkKey,
                 format!("SDK Key '{}' is invalid.", self.sdk_key),

src/eval/evaluator.rs

@@ -491,7 +491,7 @@ fn eval_prerequisite_cond(
             .iter()
             .map(|k| format!("'{k}'"))
             .collect::<Vec<String>>()
-            .join(" => ");
+            .join(" -> ");
         return Fatal(output);
     }
 

tests/eval.rs

@@ -0,0 +1,38 @@
+#![allow(dead_code)]
+
+use crate::utils::RecordingLogger;
+use configcat::OverrideBehavior::LocalOnly;
+use configcat::{Client, FileDataSource};
+use log::set_max_level;
+
+mod utils;
+
+fn init() {
+    set_max_level(log::LevelFilter::Info);
+    _ = log::set_logger(&RecordingLogger {});
+}
+
+#[tokio::test]
+async fn prerequisite_circular_deps() {
+    init();
+
+    let tests = vec![
+        ("key1", "'key1' -> 'key1'"),
+        ("key2", "'key2' -> 'key3' -> 'key2'"),
+        ("key4", "'key4' -> 'key3' -> 'key2' -> 'key3'"),
+    ];
+
+    let client = Client::builder("local")
+        .overrides(
+            Box::new(FileDataSource::new("tests/data/test_circulardependency_v6.json").unwrap()),
+            LocalOnly,
+        )
+        .build()
+        .unwrap();
+
+    for test in tests {
+        _ = client.get_flag_details(test.0, None).await;
+        let logs = RecordingLogger::LOGS.take();
+        assert!(logs.contains(test.1));
+    }
+}

tests/override.rs

@@ -1,4 +1,4 @@
-use crate::utils::{construct_bool_json_payload, produce_mock_path, rand_sdk_key};
+use crate::utils::{construct_bool_json_payload, produce_mock_path};
 use configcat::OverrideBehavior::{LocalOnly, LocalOverRemote, RemoteOverLocal};
 use configcat::Value::{Bool, Float, Int};
 use configcat::{Client, FileDataSource, MapDataSource, Value};
@@ -8,7 +8,7 @@ mod utils;
 
 #[tokio::test]
 async fn file_simple() {
-    let client = Client::builder(rand_sdk_key().as_str())
+    let client = Client::builder("local")
         .overrides(
             Box::new(FileDataSource::new("tests/data/test_json_simple.json").unwrap()),
             LocalOnly,
@@ -33,7 +33,7 @@ async fn file_simple() {
 
 #[tokio::test]
 async fn file_complex() {
-    let client = Client::builder(rand_sdk_key().as_str())
+    let client = Client::builder("local")
         .overrides(
             Box::new(FileDataSource::new("tests/data/test_json_complex.json").unwrap()),
             LocalOnly,
@@ -58,7 +58,7 @@ async fn file_complex() {
 
 #[tokio::test]
 async fn map() {
-    let client = Client::builder(rand_sdk_key().as_str())
+    let client = Client::builder("local")
         .overrides(
             Box::new(MapDataSource::new(HashMap::from([
                 ("enabledFeature".to_owned(), Bool(true)),

tests/utils/mod.rs

@@ -1,5 +1,6 @@
 use log::{Level, Log, Metadata, Record};
 use rand::distributions::{Alphanumeric, DistString};
+use std::cell::RefCell;
 
 pub fn produce_mock_path() -> (String, String) {
     let sdk_key = rand_sdk_key();
@@ -44,3 +45,31 @@ impl Log for PrintLog {
 
     fn flush(&self) {}
 }
+
+pub struct RecordingLogger {}
+
+impl RecordingLogger {
+    thread_local!(pub static LOGS: RefCell<String> = RefCell::new(String::default()));
+}
+
+impl Log for RecordingLogger {
+    fn enabled(&self, metadata: &Metadata) -> bool {
+        metadata.level() <= log::max_level() && metadata.target().contains("configcat")
+    }
+
+    fn log(&self, record: &Record) {
+        if !self.enabled(record.metadata()) {
+            return;
+        }
+        let level = match record.level() {
+            Level::Error => "[ERROR]",
+            Level::Warn => "[WARN]",
+            Level::Info => "[INFO]",
+            Level::Debug => "[DEBUG]",
+            Level::Trace => "[TRACE]",
+        };
+        Self::LOGS.with_borrow_mut(|l| l.push_str(format!("{level} {}", record.args()).as_str()));
+    }
+
+    fn flush(&self) {}
+}