fix matcher and test

Author: chillaq

splitio/models/grammar/matchers/rule_based_segment.py

@@ -63,7 +63,7 @@ def _match_dep_rb_segments(self, excluded_rb_segments, key, attributes, context)
             else:
                 excluded_segment = context['ec'].rbs_segments.get(excluded_rb_segment.name)
                 if key in excluded_segment.excluded.get_excluded_keys():
-                    return True
+                    return False
 
                 if self._match_dep_rb_segments(excluded_segment.excluded.get_excluded_segments(), key, attributes, context):
                     return True

tests/engine/files/rule_base_segments2.json

@@ -38,7 +38,7 @@
     "status": "ACTIVE",
     "trafficTypeName": "user",
     "excluded":{
-      "keys":["bilal2"],
+      "keys":["bilal2@split.io"],
       "segments":[]
     },
     "conditions": [

tests/engine/test_evaluator.py

@@ -313,8 +313,8 @@ def test_using_rbs_in_excluded(self):
         assert e.eval_with_context('bilal@split.io', 'bilal@split.io', 'some', {'email': 'bilal@split.io'}, ctx)['treatment'] == "off"
         ctx = evaluation_facctory.context_for('bilal', ['some'])
         assert e.eval_with_context('bilal', 'bilal', 'some', {'email': 'bilal'}, ctx)['treatment'] == "on"
-        ctx = evaluation_facctory.context_for('bilal2', ['some'])
-        assert e.eval_with_context('bilal2', 'bilal2', 'some', {'email': 'bilal2'}, ctx)['treatment'] == "off"
+        ctx = evaluation_facctory.context_for('bilal2@split.io', ['some'])
+        assert e.eval_with_context('bilal2@split.io', 'bilal2@split.io', 'some', {'email': 'bilal2@split.io'}, ctx)['treatment'] == "on"
 
     @pytest.mark.asyncio
     async def test_evaluate_treatment_with_rbs_in_condition_async(self):
@@ -385,8 +385,8 @@ async def test_using_rbs_in_excluded_async(self):
         assert e.eval_with_context('bilal@split.io', 'bilal@split.io', 'some', {'email': 'bilal@split.io'}, ctx)['treatment'] == "off"
         ctx = await evaluation_facctory.context_for('bilal', ['some'])
         assert e.eval_with_context('bilal', 'bilal', 'some', {'email': 'bilal'}, ctx)['treatment'] == "on"
-        ctx = await evaluation_facctory.context_for('bilal2', ['some'])
-        assert e.eval_with_context('bilal2', 'bilal2', 'some', {'email': 'bilal2'}, ctx)['treatment'] == "off"
+        ctx = await evaluation_facctory.context_for('bilal2@split.io', ['some'])
+        assert e.eval_with_context('bilal2@split.io', 'bilal2@split.io', 'some', {'email': 'bilal2@split.io'}, ctx)['treatment'] == "on"
 
 class EvaluationDataFactoryTests(object):
     """Test evaluation factory class."""