MushroomObserver / mushroom-observer / 26460528778

96%

push

github

Bundle update with Rails ~> 7.0 (#4326)

* Bundle update wtih Rails ~> 7.0

* Fix two Rails 7.2.3.1 compatibility breaks in index/query flow

1. `redirect_to` now returns the HTTP status Integer (302) as its last
   expression instead of "" — `build_index_with_query` was treating
   that Integer as a query and passing it to `filtered_index`, causing
   `NoMethodError: undefined method 'id' for an instance of Integer`.
   Fix: extract `filtered_subaction_index` (private) and guard with
   `performed?` before testing `query.blank?`.

2. `query_from_q_param_hash` passed the raw `to_unsafe_hash` result
   (string-keyed HWIA) via `**` to `Query.lookup`. String keys fail
   the `params.slice(*attribute_names)` match (attribute names are
   symbols) and cause a TypeError in Ruby 3.4+. Fix: add
   `.symbolize_keys` to match the pattern used everywhere else in the
   codebase.

Adds a targeted controller test for the symbolize_keys fix.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Fix line-length offenses

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

5 of 5 new or added lines in 1 file covered. (100.0%)

38993 of 40441 relevant lines covered (96.42%)

663.27 hits per line