[Raphael Fakhri (RaphaelFakhri)]

Summary

parse_filters looks for the = separator with str.index, which raises ValueError: substring not found when the separator is missing. The -1 check and the Invalid filter message that follow it never run. This change switches to str.find so a filter without = raises ValueError: Invalid filter <filter>, matching the JavaScript SDK.

Changes

• Use str.find("=") in parse_filters so the existing Invalid filter error is reachable.
• Add tests for the error message and for splitting on the first = only.

Testing

cd py
python -m pytest src/braintrust/test_framework.py -k "parse_filters or evaluate_filter"

Before the change: 1 failed, 9 passed. After the change: 10 passed.

[Abhijeet Prasad (AbhiPrasad)]

released with https://github.com/braintrustdata/braintrust-sdk-python/releases/tag/py-sdk-v0.44.0