feat: segment

Author: SSebo

resources/fixtures/repo.json

@@ -58,7 +58,7 @@
             {
               "type": "segment",
               "subject": "user",
-              "predicate": "in",
+              "predicate": "is in",
               "objects": [
                 "some_segment1-fjoaefjaam"
               ]
@@ -108,7 +108,7 @@
             {
               "type": "segment",
               "subject": "user",
-              "predicate": "in",
+              "predicate": "is in",
               "objects": [
                 "some_segment1-fjoaefjaam"
               ]
@@ -158,7 +158,7 @@
             {
               "type": "segment",
               "subject": "user",
-              "predicate": "in",
+              "predicate": "is in",
               "objects": [
                 "some_segment1-fjoaefjaam"
               ]
@@ -230,7 +230,7 @@
             {
               "type": "segment",
               "subject": "user",
-              "predicate": "in",
+              "predicate": "is in",
               "objects": [
                 "some_segment1-fjoaefjaam"
               ]
@@ -336,6 +336,41 @@
           "disabled_key": "disabled_value"
         }
       ]
+    },
+    "not_in_segment": {
+      "key": "not_in_segment",
+      "enabled": true,
+      "forClient": false,
+      "version": 1,
+      "disabledServe": {
+        "select": 0
+      },
+      "defaultServe": {
+        "select": 0
+      },
+      "rules": [
+        {
+          "serve": {
+            "select": 1
+          },
+          "conditions": [
+            {
+              "type": "segment",
+              "subject": "user",
+              "predicate": "is not in",
+              "objects": [
+                "some_segment1-fjoaefjaam"
+              ]
+            }
+          ]
+        }
+      ],
+      "variations": [
+        {},
+        {
+          "not_in": true
+        }
+      ]
     }
   }
 }

resources/fixtures/segments.json

@@ -10,7 +10,7 @@
             {
               "type": "string",
               "subject": "city",
-              "predicate": "in",
+              "predicate": "is in",
               "objects": [
                 "4"
               ]

resources/fixtures/toggles.json

@@ -43,7 +43,7 @@
                         {
                             "type": "string",
                             "subject": "city",
-                            "predicate": "in",
+                            "predicate": "is one of",
                             "objects": [
                                 "1",
                                 "2",
@@ -60,7 +60,7 @@
                         {
                             "type": "segment",
                             "subject": "user",
-                            "predicate": "in",
+                            "predicate": "is in",
                             "object": ["some_segment1-fjoaefjaam"]
                         }
                     ]
@@ -99,7 +99,7 @@
                         {
                             "type": "string",
                             "subject": "city",
-                            "predicate": "in",
+                            "predicate": "is one of",
                             "objects": [
                                 "1",
                                 "2",
@@ -109,7 +109,7 @@
                         {
                             "type": "string",
                             "subject": "os",
-                            "predicate": "in",
+                            "predicate": "is one of",
                             "objects": [
                                 "mac",
                                 "linux"

src/evalutate.rs

@@ -301,19 +301,26 @@ impl Condition {
     pub fn meet(&self, user: &FPUser, segment_repo: Option<&HashMap<String, Segment>>) -> bool {
         match &self.r#type {
             ConditionType::String => self.match_string_condition(user, &self.predicate),
-            ConditionType::Segment => self.match_segment_condition(user, segment_repo),
+            ConditionType::Segment => {
+                self.match_segment_condition(user, &self.predicate, segment_repo)
+            }
             _ => false,
         }
     }
 
     fn match_segment_condition(
         &self,
         user: &FPUser,
+        predicate: &str,
         segment_repo: Option<&HashMap<String, Segment>>,
     ) -> bool {
         match segment_repo {
             None => false,
-            Some(repo) => self.user_in_segments(user, repo),
+            Some(repo) => match predicate {
+                "is in" => self.user_in_segments(user, repo),
+                "is not in" => !self.user_in_segments(user, repo),
+                _ => false,
+            },
         }
     }
 
@@ -453,6 +460,23 @@ mod tests {
         assert!(r.get("variation_1").is_some());
     }
 
+    #[test]
+    fn test_not_in_segment_condition() {
+        let mut path = PathBuf::from(env!("CARGO_MANIFEST_DIR"));
+        path.push("resources/fixtures/repo.json");
+        let json_str = fs::read_to_string(path).unwrap();
+        let repo = load_json(&json_str);
+        assert!(repo.is_ok());
+        let repo = repo.unwrap();
+
+        let user = FPUser::new("key11").with("city", "100");
+        let toggle = repo.toggles.get("not_in_segment").unwrap();
+        let r = toggle.eval(&user, &repo.segments);
+        let r = r.unwrap();
+        let r = r.as_object().unwrap();
+        assert!(r.get("not_in").is_some());
+    }
+
     #[test]
     fn test_multi_condition() {
         let mut path = PathBuf::from(env!("CARGO_MANIFEST_DIR"));