Create builder for TransitDataProviderFilter

[habrahamsson-skanetrafiken]

Summary

This PR is a small refactor of the RouteRequestTransitDataProviderFilter. It doesn't change any behavior at all.

• Adds a builder to simplify using this in places where we don't have a RouteRequest.
• Renames the filter to DefaultTransitDataProviderFilter.

Unit tests

No new unit tests are added, but the builder makes a lot of tests much more readable.

Documentation

None needed IMO

Bumping the serialization version id

Nope

[codecov]

Codecov Report

❌ Patch coverage is 95.89041% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.20%. Comparing base (1278304) to head (5474004).
⚠️ Report is 226 commits behind head on dev-2.x.

Files with missing lines	Patch %	Lines
...quest/DefaultTransitDataProviderFilterBuilder.java	94.64%	0 Missing and 3 partials ⚠️

Additional details and impacted files

@@              Coverage Diff              @@
##             dev-2.x    #6974      +/-   ##
=============================================
+ Coverage      72.14%   72.20%   +0.05%     
- Complexity     19772    19866      +94     
=============================================
  Files           2151     2156       +5     
  Lines          79955    80091     +136     
  Branches        8058     8084      +26     
=============================================
+ Hits           57687    57827     +140     
+ Misses         19423    19419       -4     
  Partials        2845     2845              

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

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[t2gran]

[Refactor] ?

[habrahamsson-skanetrafiken]

Edit: Changed the commit messages to use "refactor:" instead of "[Refactor]".

[jessicaKoehnke]

Nice simplification!

[habrahamsson-skanetrafiken]

I noticed a slight change in behavior. An explicitly empty filter list in the request would originally disallow all transit. This behavior is undocumented and differs from what we usually do. In order to not break any clients in the event that they are depending on this I fixed it and added a test.