kobotoolbox / kpi / 27325556830

82%

master: 76%

Build Type

push

github

Committed by web-flow

Commit Message

feat(bulkProcessing): skip already-processed submissions in bulk actions DEV-2239   (#7142)

### 📣 Summary
Previously, if any submission in a bulk action request had an existing
result or an active conflicting job, the entire request was rejected
with a 400. This PR changes the behavior so eligible submissions are
processed and ineligible ones are silently skipped, with their UUIDs
returned in `skipped_uuids` on the POST response. A 400 is only returned
if every submission is ineligible.

### 📖 Description
Previously `_validate_no_existing_results` and
`_validate_no_active_bulk_conflicts` raised a 400 if any submission was
ineligible. Now, 400 is only raised if all submissions are ineligible.
The POST response includes `skipped_uuids` so callers know which
submissions were excluded.

Coverage Stats

9028 of 12261 branches covered (73.63%)

10 of 27 new or added lines in 4 files covered. (37.04%)

318 existing lines in 17 files now uncovered.

29644 of 36337 relevant lines covered (81.58%)

5.56 hits per line