feat: filter __in

[JakNowy]

Pull Request Template for FastCRUD

Description

This PR adds support for in_ filtering in crud methods like FastCRUD.select(item_id__in=[1,2]).

Changes

Extended parse_filters method with addtional elif op == "in", mapping to sqlalchemy/sqlmodel or() filter.

Tests

Added a test to test_parse_filters for both sqlalchemy and sqlmodel.

Checklist

• I have read the CONTRIBUTING document.
• My code follows the code style of this project.
• I have added necessary documentation (if appropriate).
• I have added tests that cover my changes (if applicable).
• All new and existing tests passed.

Additional Notes

I was trying to acomplish something similar for or_() but it turned out to be too complex for now.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (1c9cf66) to head (dad3295).

Additional details and impacted files

@@            Coverage Diff            @@
##              main       #57   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           64        64           
  Lines         4468      4510   +42     
=========================================
+ Hits          4468      4510   +42     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[igorbenav]

Hey, @JakNowy, thanks for another great PR🎉!
Could you please add an example using __in to a docstring somewhere? I think adding not_in would be good as well. How did you think about adding or?

[igorbenav]

@JakNowy did you get a chance to take a look at this?

[JakNowy]

Hey @igorbenav not yet, but I will definetely add some more docs and not_in support soon! For or, I was thinking about nesting the parse_filters() method in that case to produce something like or(parsed_filter1, parsed_filter2).

[JakNowy]

Added not_in support and tests. Although quite simple, looks decent to me. Added short docstring for each fastcrud method, as well as some clarification in docs/advanced/crud. @igorbenav

[igorbenav]

For or, I was thinking about nesting the parse_filters() method in that case to produce something like or(parsed_filter1, parsed_filter2).

I'll create another issue for the missing comparison operators, we can handle it there. Thanks for the pr!