Skip to content

Conversation

vpaturet
Copy link
Contributor

Summary

Follow-up after #6868:
This PR simplifies the equals() method in TripPatternForDate to align it on the hashcode method.
The natural key of a TripPatternForDate is the pair (routing trip pattern id, service date)

Issue

Relates to #6820

Unit tests

No

Documentation

Updated Javadoc

Changelog

skip

@vpaturet vpaturet self-assigned this Sep 23, 2025
@vpaturet vpaturet added +Skip Changelog This is not a relevant change for a product owner since last release. !Optimization The feature is to improve performance. +Real-Time The issue/PR is related to RealTime updates labels Sep 23, 2025
Copy link

codecov bot commented Sep 23, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 72.10%. Comparing base (3e501b2) to head (c9d451c).
⚠️ Report is 373 commits behind head on dev-2.x.

Files with missing lines Patch % Lines
...ithm/raptoradapter/transit/TripPatternForDate.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             dev-2.x    #6883   +/-   ##
==========================================
  Coverage      72.10%   72.10%           
+ Complexity     19540    19537    -3     
==========================================
  Files           2106     2106           
  Lines          79004    79000    -4     
  Branches        8009     8006    -3     
==========================================
- Hits           56967    56965    -2     
+ Misses         19217    19214    -3     
- Partials        2820     2821    +1     

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

@vpaturet vpaturet marked this pull request as ready for review September 25, 2025 12:09
@vpaturet vpaturet requested a review from a team as a code owner September 25, 2025 12:09
@leonardehrenfried
Copy link
Member

Vincent will test this before merging.

Copy link
Member

@leonardehrenfried leonardehrenfried left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code looks good and it sounds like the correct thing to do, but this is a very tricky area of the code base, which was subtly broken by changes like this before, so we need to be extra-careful testing this.

@optionsome optionsome added the Digitransit Test Feature is under testing in Digitransit environment(s) label Oct 3, 2025
@vpaturet vpaturet added the Entur Test This is currently being tested at Entur label Oct 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Digitransit Test Feature is under testing in Digitransit environment(s) Entur Test This is currently being tested at Entur !Optimization The feature is to improve performance. +Real-Time The issue/PR is related to RealTime updates +Skip Changelog This is not a relevant change for a product owner since last release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants