Fix Contained filter with Range Criteria and add unit tests

supabase-community / postgrest-csharp

A C# Client library for Postgrest

https://supabase-community.github.io/postgrest-csharp/api/Postgrest.html

MIT License

114 stars 22 forks source link

Fix Contained filter with Range Criteria and add unit tests #20

Closed elrhomariyounes closed 3 years ago

elrhomariyounes commented 3 years ago

What kind of change does this PR introduce?

Buf fix and chore unit tests

What is the current behavior?

Same as #19 Contained filter behaved same as contains operator with Range criteria

What is the new behavior?

Allowing Contained operator with Range criteria
Add units tests for : sl, sr, nxl, nxr, adj, ov filters

elrhomariyounes commented 3 years ago

Some unit tests failed I think it's due to some records in my db, I didn't find a way to test those filters with range types without using hard coded values

acupofjose commented 3 years ago

@elrhomariyounes I'm going to close this in favor of #21 (built on top of this PR) which uses CollectionAssert instead of Count asserts - could you review it for me?