MushroomObserver / mushroom-observer / 26630666066

97%

main: 97%

Pull #4389

github

species_lists: fix CI — sub-controllers render Phlex Edit/New directly

`NameListsController#create_list` did `render("species_lists/new")` and
`UploadsController#update` did `render("species_lists/edit")` — both
relied on the action ERB templates this PR deleted. Swept them onto
the Phlex `New` / `Edit` classes the main controller now renders.

The sub-controllers don't traverse a form-reload path, so
`@dubious_where_reasons` / `@submitted_project_ids` aren't set —
passed as `[]` / `nil` defaults so the form renders the same shape
the main `new` / `edit` page does on first load.

Test-side cleanup: drop two more `assert_template("species_lists/edit")`
/ `assert_template("species_lists/new")` calls in the sub-controller
tests (Phlex views don't show up in `assert_template`); assert
`form#species_list_form` instead.

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

279 of 281 new or added lines in 13 files covered. (99.29%)

39510 of 40940 relevant lines covered (96.51%)

659.1 hits per line