polish

Author: chillaq

splitio/models/grammar/matchers/prerequisites.py

@@ -28,9 +28,6 @@ def match(self, key, attributes=None, context=None):
         if self._prerequisites == None:
             return True
 
-        if not isinstance(key, str):
-            return False
-        
         evaluator = context.get('evaluator')
         bucketing_key = context.get('bucketing_key')
         for prerequisite in self._prerequisites:

tests/models/grammar/test_matchers.py

@@ -1166,9 +1166,4 @@ def test_matcher_behaviour(self, mocker):
         assert parsed.match('SPLIT_2', {}, {'evaluator': evaluator, 'ec': [{'flags': ['prereq_chain'], 'segment_memberships': {}}]}) is True
 
         evaluator.eval_with_context.return_value = {'treatment': 'off'}
-        assert parsed.match('SPLIT_2', {}, {'evaluator': evaluator, 'ec': [{'flags': ['prereq_chain'], 'segment_memberships': {}}]}) is False
-
-        assert parsed.match([], {}, {'evaluator': evaluator, 'ec': [{'flags': ['prereq_chain'], 'segment_memberships': {}}]}) is False
-        assert parsed.match({}, {}, {'evaluator': evaluator, 'ec': [{'flags': ['prereq_chain'], 'segment_memberships': {}}]}) is False
-        assert parsed.match(123, {}, {'evaluator': evaluator, 'ec': [{'flags': ['prereq_chain'], 'segment_memberships': {}}]}) is False
-        assert parsed.match(object(), {}, {'evaluator': evaluator, 'ec': [{'flags': ['prereq_chain'], 'segment_memberships': {}}]}) is False
+        assert parsed.match('SPLIT_2', {}, {'evaluator': evaluator, 'ec': [{'flags': ['prereq_chain'], 'segment_memberships': {}}]}) is False