add some tests for all_flags_detail

Author: cwaldren-ld

launchdarkly-server-sdk/src/client.rs

@@ -845,7 +845,7 @@ mod tests {
     use hyper::client::HttpConnector;
     use launchdarkly_server_sdk_evaluation::Reason;
     use std::collections::HashMap;
-
+    use assert_json_diff::assert_json_eq;
     use tokio::time::Instant;
 
     use crate::data_source::MockDataSource;
@@ -854,10 +854,7 @@ mod tests {
     use crate::events::event::{OutputEvent, VariationKey};
     use crate::events::processor_builders::EventProcessorBuilder;
     use crate::stores::store_types::{PatchTarget, StorageItem};
-    use crate::test_common::{
-        self, basic_flag, basic_flag_with_prereq, basic_int_flag, basic_migration_flag,
-        basic_off_flag,
-    };
+    use crate::test_common::{self, basic_flag, basic_flag_with_prereq, basic_flag_with_prereqs_and_visibility, basic_int_flag, basic_migration_flag, basic_off_flag};
     use crate::{ConfigBuilder, MigratorBuilder, Operation, Origin};
     use test_case::test_case;
 
@@ -965,6 +962,137 @@ mod tests {
         ));
     }
 
+    #[test]
+    fn all_flags_detail_is_invalid_when_offline() {
+        let (client, _event_rx) = make_mocked_offline_client();
+        client.start_with_default_executor();
+
+        let context = ContextBuilder::new("bob")
+            .build()
+            .expect("Failed to create context");
+
+        let all_flags = client.all_flags_detail(&context, FlagDetailConfig::new());
+        assert_json_eq!(all_flags, json!({"$valid": false, "$flagsState" : {}}));
+    }
+
+    #[test]
+    fn all_flags_detail_is_invalid_when_not_initialized() {
+        let (client, _event_rx) = make_mocked_client();
+
+        let context = ContextBuilder::new("bob")
+            .build()
+            .expect("Failed to create context");
+
+        let all_flags = client.all_flags_detail(&context, FlagDetailConfig::new());
+        assert_json_eq!(all_flags, json!({"$valid": false, "$flagsState" : {}}));
+    }
+
+
+    #[test]
+    fn all_flags_detail_returns_flag_states() {
+        let (client, _event_rx) = make_mocked_client();
+        client.start_with_default_executor();
+        client
+            .data_store
+            .write()
+            .upsert(
+                "myFlag1",
+                PatchTarget::Flag(StorageItem::Item(basic_flag("myFlag1"))),
+            )
+            .expect("patch should apply");
+        client
+            .data_store
+            .write()
+            .upsert(
+                "myFlag2",
+                PatchTarget::Flag(StorageItem::Item(basic_flag("myFlag2"))),
+            )
+            .expect("patch should apply");
+        let context = ContextBuilder::new("bob")
+            .build()
+            .expect("Failed to create context");
+
+        let all_flags = client.all_flags_detail(&context, FlagDetailConfig::new());
+
+        client.close();
+
+        assert_json_eq!(all_flags, json!({
+            "myFlag1": true,
+            "myFlag2": true,
+            "$flagsState": {
+                "myFlag1": {
+                    "version": 42,
+                    "variation": 1
+                },
+                 "myFlag2": {
+                    "version": 42,
+                    "variation": 1
+                },
+            },
+            "$valid": true
+        }));
+    }
+
+
+    #[test]
+    fn all_flags_detail_returns_prerequisite_relations() {
+        let (client, _event_rx) = make_mocked_client();
+        client.start_with_default_executor();
+        client
+            .data_store
+            .write()
+            .upsert(
+                "prereq1",
+                PatchTarget::Flag(StorageItem::Item(basic_flag("prereq1"))),
+            )
+            .expect("patch should apply");
+        client
+            .data_store
+            .write()
+            .upsert(
+                "prereq2",
+                PatchTarget::Flag(StorageItem::Item(basic_flag("prereq2"))),
+            )
+            .expect("patch should apply");
+
+        client.data_store
+            .write()
+            .upsert(
+                "toplevel",
+                PatchTarget::Flag(StorageItem::Item(basic_flag_with_prereqs_and_visibility("toplevel", &["prereq1", "prereq2"], false))),
+            )
+            .expect("patch should apply");
+
+        let context = ContextBuilder::new("bob")
+            .build()
+            .expect("Failed to create context");
+
+        let all_flags = client.all_flags_detail(&context, FlagDetailConfig::new());
+
+        client.close();
+
+        assert_json_eq!(all_flags, json!({
+            "prereq1": true,
+            "prereq2": true,
+            "toplevel": true,
+            "$flagsState": {
+                "toplevel": {
+                    "version": 42,
+                    "variation": 1,
+                    "prerequisites": ["prereq1", "prereq2"]
+                },
+                "prereq1": {
+                    "version": 42,
+                    "variation": 1
+                },
+                 "prereq2": {
+                    "version": 42,
+                    "variation": 1
+                },
+            },
+            "$valid": true
+        }));
+    }
     #[test]
     fn variation_tracks_events_correctly() {
         let (client, event_rx) = make_mocked_client();

launchdarkly-server-sdk/src/evaluation.rs

@@ -600,6 +600,5 @@ mod tests {
         });
 
         assert_json_eq!(expected, flag_detail);
-
     }
 }