GrottoCenter / grottocenter-api / 30674436535

87%

develop: 91%

Build Type

Pull #1750

github

Committed by ClemRz

Commit Message

fix(document): address PR review items on m2m collection update

- fix: preserve existing authors when newAuthors added without document.authors
  (use addToCollection instead of replaceCollection to avoid silent data loss)
- fix: move TCaver/TDescription createEach inside transaction for true atomicity
- fix: swagger new-entities endpoint wrongly referenced the multipart '[]' sentinel
- fix: swagger PUT /documents/{id} listed languages as an m2m sentinel field
- refactor: extract replaceM2MCollections helper to DocumentService (DRY)
- test: add regression test for newAuthors without explicit document.authors
- chore: remove dead createdMassifIds variable from multiple-validate test

Pull Request Pull Request #1750: fix(document): replace m2m collections via replaceCollection on update and validate

Coverage Stats

5154 of 6194 branches covered (83.21%)

Branch coverage included in aggregate %.

53 of 54 new or added lines in 4 files covered. (98.15%)

1 existing line in 1 file now uncovered.

9521 of 10609 relevant lines covered (89.74%)

132.84 hits per line