Skip to content

Commit 56ccfc8

Browse files
author
Francesco Faraone
committed
Fix null lookups for missed fields
1 parent 663caa2 commit 56ccfc8

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

py_rql/helpers.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ def apply_operator(prop, operator, value, obj):
1515
try:
1616
prop_value = extract_value(obj, prop)
1717
except KeyError:
18-
return False
18+
prop_value = None
1919
result = operator(prop_value, value)
2020
return result
2121

tests/test_helpers.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,9 +47,8 @@ def test_apply_operator(mocker):
4747

4848

4949
def test_apply_operator_prop_not_found(mocker):
50-
mocked_op = mocker.MagicMock(return_value=True)
50+
mocked_op = mocker.MagicMock(return_value=False)
5151
assert apply_operator('other', mocked_op, 'value_b', {'root': 'value_a'}) is False
52-
mocked_op.assert_not_called()
5352

5453

5554
def test_apply_logical_operator(mocker):

0 commit comments

Comments
 (0)