kobotoolbox / kpi / 22580089244

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.

7472 of 11502 branches covered (64.96%)

28716 of 34958 relevant lines covered (82.14%)

1.62 hits per line