GrottoCenter / grottocenter-api / 28332874832

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

4938 of 5967 branches covered (82.76%)

Branch coverage included in aggregate %.

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

9215 of 10328 relevant lines covered (89.22%)

129.41 hits per line