1
95%
main: 96%

Ran 18 Feb 2024 03:24AM UTC

Files 741

Run time 22s

Badge

Embed ▾

Committed 18 Feb 2024 03:18AM UTC coverage: 94.505% (-0.02%) from 94.523%

Job # 7946252360.1

Build Type

Pull #1920

github

Committed by

nimmolo

Commit Message

`Observations::LocationsController`

Combines two controllers I created without really understanding what they did: `Locations::MergesController` (a GET form action) and `Locations::ObservationsController` (a PUT/PATCH callback for the same form) into one controller.

The form gives a UI to deal with “Observations that have a `where` but no `Location`". In other words, they just have a string a user entered, that maybe matches one or more of our Locations. So it’s not a merge, properly speaking. It’s a bulk assignment of a Location association, where there was no association before. It seems to belong more properly under "Observations".

Clears up confusion from my misleadingly-named `Locations::MergesController`: this form does not merge Location records.

Pull Request Pull Request #1920: `Observations::LocationsController`, not `Locations::MergesController`

Run Details

25075 of 26533 relevant lines covered (94.5%)

548.28 hits per line

MushroomObserver / mushroom-observer / 7946252360 / 1
95%
main: 96%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 7946252360.1

MushroomObserver / mushroom-observer / 7946252360 / 1 95% main: 96%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 7946252360.1

MushroomObserver / mushroom-observer / 7946252360 / 1
95%
main: 96%

README BADGES
x