-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Orca Fixes Part 2 #5469
Orca Fixes Part 2 #5469
Conversation
src/ext-test/java/org/opentripplanner/ext/fares/impl/OrcaFareServiceTest.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also run the formatter.
Thanks for the feedback, it should be ready for another review. @leonardehrenfried |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## dev-2.x #5469 +/- ##
=============================================
- Coverage 66.84% 66.82% -0.02%
- Complexity 15539 15573 +34
=============================================
Files 1802 1806 +4
Lines 69659 69823 +164
Branches 7351 7359 +8
=============================================
+ Hits 46564 46662 +98
- Misses 20640 20707 +67
+ Partials 2455 2454 -1
☔ View full report in Codecov by Sentry. |
Summary
This is just a few more fixes for ORCA agencies
Unit tests
This includes a new test for non ORCA agencies. It needs some more work, but that should go along with a refactor of
ItineraryFares
to fully utilize Leg Products.