MushroomObserver / mushroom-observer / 26634589984

97%

main: 97%

Pull #4392

github

CI fixups for #4392 sidebar bug fix

The sidebar bug fix (destroy/post/put/patch tuples now dispatch to forms) duplicates the destroy form across BOTH menus — top-bar dropdown AND mobile sidebar. Two existing tests broke as a result:

- LicensesControllerTest#test_show_unused_license expected `count: 1` destroy button on the show page; now finds 2 (one per menu) — bumped to `count: 2` with a comment.
- ImagesIntegrationTest#test_show_image_edit_links used an unscoped `click_button(class: destroy_image_link_X)` which Capybara now sees as ambiguous; scoped to `within("#context_nav")` (the top-bar dropdown's `<ul>` id) so the click resolves to one element.

Plus a Layout/LineLength fix on `top_bar_test.rb:49` (extracted the long selector into a local).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

77 of 78 new or added lines in 4 files covered. (98.72%)

39555 of 40986 relevant lines covered (96.51%)

667.82 hits per line