kobotoolbox / kpi / 22580089244 / 1

82%

master: 76%

push

github

feat(subsequences): add unit tests for subsequence polling tasks and failure handlers DEV-1787 (#6776)

### 📣 Summary
Added unit tests for `poll_run_external_process` and its Celery signal
failure handler `poll_run_external_process_failure` to ensure external
NLP timeouts and crashes are accurately recorded in the database.

### 📖 Description
Currently, we lacked testing around the Celery polling mechanism for
external subsequences processes. This PR introduces a test suite
(`TestPollRunExternalProcess` and `TestPollRunExternalProcessFailure`)
covering both standard task execution and the `@task_failure.connect`
signal handler.

Test Coverage Includes:
- Polling task correctly raises `SubsequenceTimeoutError` for
`in_progress` API responses to trigger Celery retries.
- Polling task correctly intercepts graceful API failures without
crashing the worker.
- Failure handler catches unhandled exceptions and logs a `failed`
status to `SubmissionSupplement`.
- Failure handler intercepts max-retry exhaustion and rewrites the
internal timeout error to a user-friendly "Maximum retries exceeded."
message.

7277 of 11561 branches covered (62.94%)

27900 of 34951 relevant lines covered (79.83%)

0.8 hits per line