fix: filtering by date when stop is included as a filter param

[bfauble]

Summary of changes

Asana Ticket: [extra] Route pattern date filter returning weird results

the way the matches within the RoutePattern state were setup, the by_ids function was being called due to the way the date implicitly adds the :id filter field. Updated to handle :id + :stop_ids + :route_ids filters if present.

[bfauble]

a better approach potentially could be to integrate the call to expand_stop_filters here: https://github.com/mbta/api/blob/master/apps/api_web/lib/api_web/controllers/route_pattern_controller.ex#L83 into the :date -> route_pattern_ids process that combines the filters into a set of ids before being sent to the state to query.

[github-actions]

Coverage of commit 3aaac0c

Summary coverage rate:
  lines......: 89.1% (4128 of 4631 lines)
  functions..: 70.9% (2231 of 3145 functions)
  branches...: no data found

Files changed coverage rate:
                                                                        |Lines       |Functions  |Branches    
  Filename                                                              |Rate     Num|Rate    Num|Rate     Num
  ============================================================================================================
  apps/state/lib/state/route_pattern.ex                                 |76.7%     30|63.4%    41|    -      0

Download coverage report