Repurpose the RealtimeTestEnvironment into a TransitTestEnvironment

[habrahamsson-skanetrafiken]

Summary

This is a first refactor of the RealtimeTestEnvironment after discussion with @t2gran and @leonardehrenfried. There will be more upcoming changes.

The purpose is to make the RealtimeTestEnvironment into a more general framework for setting up and testing transit entities.

It only touches tests and should be a pure refactor.

• Separate gtfs-rt and siri specific methods in their own classes.
• Rename RealtimeTestEnvironment to TransitTestEnvironment
• Reduce dependencies on TimetableRepositoryForTest.
• Encapsulate date and timezone in the test environment
• Create a thin helper class TripOnDateDataFetcher to simplify fetching data from the TransitModel.
• The default feed id is not configurable in the TransitTestEnvironment. The reason is that having a static method for creating test ids will greatly simplify converting existing tests to this new framework.

Issue

#6887

Bumping the serialization version id

No

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.17%. Comparing base (8d33b0e) to head (9bd7180).
⚠️ Report is 375 commits behind head on dev-2.x.

Additional details and impacted files

@@              Coverage Diff              @@
##             dev-2.x    #6899      +/-   ##
=============================================
+ Coverage      72.10%   72.17%   +0.06%     
- Complexity     19654    19833     +179     
=============================================
  Files           2123     2155      +32     
  Lines          79410    80044     +634     
  Branches        8049     8082      +33     
=============================================
+ Hits           57257    57770     +513     
- Misses         19316    19425     +109     
- Partials        2837     2849      +12     

☔ 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.

[leonardehrenfried]

This will probably conflict with #6894

[leonardehrenfried]

#6894 has been merged so you will have to resolve the conflicts.

The service dates are now part of TripInput, not the environment so it's configurable.

[t2gran]

This is not a full review, only touching a few architectural issues - when these are resolved I will look at the rest of the code.

[leonardehrenfried]

I've picked up on a few things.

I know that we have different opinions on string constants, so I'm happy to go along with your way of doing things.

[leonardehrenfried]

I'm not sure the how easily we can move away from TimetableRepositoryForTest if we don't return builders.

[leonardehrenfried]

I will give you permission to merge this in my absence even if you have to make changes.