1
89%
develop: 89%

Ran 03 Aug 2023 10:41PM UTC

Files 86

Run time 1s

Badge

Embed ▾

Committed 03 Aug 2023 10:36PM UTC coverage: 88.49%. Remained the same

Job # 5756308443.1

Build Type

push

github

Committed by

web-flow

Commit Message

Fix adding duplicate officers to incidents (#1006)

## Description of Changes
Merging fix https://github.com/OrcaCollective/OpenOversight/pull/347
upstream.

**Original PR description**:
Only add new officers to incidents.

When a user edits an incident currently, we retrieve the Incident from
the database and append all officers in the form to that Incident. This
led to sqlalchemy trying to insert duplicate rows into the
officer_incidents association table.

As far as I can tell, this was silently ignored by sqlalchemy in the
past and no unexpected rows were created. With the upgrade to sqlalchemy
2.0.19 (https://github.com/OrcaCollective/OpenOversight/pull/344), the
library appears to have started raising exceptions related to this
behavior.

This change checks to see whether an officer already exists in
Incident.officers before adding them so as to not add duplicate
officers.

## Notes for Deployment
N/A

## Screenshots (if appropriate)
N/A

## Tests and linting
 - [x] This branch is up-to-date with the `develop` branch.
 - [x] `pytest` passes on my local development environment.
 - [x] `pre-commit` passes on my local development environment.

Run Details

7427 of 8393 relevant lines covered (88.49%)

0.88 hits per line

lucyparsons / OpenOversight / 5756308443 / 1
89%
develop: 89%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 5756308443.1

lucyparsons / OpenOversight / 5756308443 / 1 89% develop: 89%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 5756308443.1

lucyparsons / OpenOversight / 5756308443 / 1
89%
develop: 89%

README BADGES
x