add filtering to non-subscription reads of $all

[NikodemMazur]

Added: an EventStore.Client.EventStoreClient.ReadAllAsync(...) overload accepting IEventFilter.

The change is to use EventStore/EventStore#3133

It'd be great to have this feature available.

If something is out of ordinary let me know :)

[josephcummings]

Hey @NikodemMazur

Thanks for this PR and contribution 👍

The changes look OK with the exception of the tests you have added which are failing on the prev LTS, not your fault, though!

When ran against prev LTS server (v22.X.X) the tests fail because the system event PersistentConfig1 is being returned (which has OriginalStreamId $persistentSubscriptionConfig) and system events are expected to have been filtered out.

This is caused by this issue that has since been fixed in current LTS.

To get around this, I think the best approach would be to change the tests to instead filter out some custom events instead of system events, which will have been working since implemented on the server way back at v21.10.0

This will allow us to workaround this problem for the prev LTS CI, without straying from our tests expectation.... Happy to implement this from my side or let you take a stab at it, whatever works for you 🙂

[NikodemMazur]

Thanks for the review and workaround suggestion, @josephcummings! I've applied the changes as advised. Let me know if any other adjustments are needed.