GrottoCenter / grottocenter-api / 28334949427

87%

develop: 88%

Pull #1708

github

fix(import): address code review findings on async CSV import

- Add atomic test-and-set guard in checkBatchCompletion to prevent
  duplicate notifications when concurrent completion-check jobs fire
- Add error handling around generateAndUploadReports in aggregateBatch
  so batch transitions to 'failed' instead of staying stuck at 'active'
- Push remaining unprocessed rows as failures when outer catch fires
  in processOneChunk to avoid silent row loss
- Remove TOKEN_SALT hardcoded fallback in test-async-import.sh
- Remove CODE_REVIEW.md (does not belong in version control)
- Add 'aggregating' status to TJobBatch model and Swagger spec

4950 of 5981 branches covered (82.76%)

Branch coverage included in aggregate %.

197 of 307 new or added lines in 8 files covered. (64.17%)

9234 of 10347 relevant lines covered (89.24%)

130.14 hits per line