Remove Job Cancellation Code from VBADocuments::UploadProcessor

[kristen-brown]

Summary

A production issue began on March 25, 2024 where Benefits Intake API submissions were not being processed out of "uploaded" status. The issue is ongoing, and we suspect that it is caused by the job cancellation code, which is not in regular use. The job is currently succeeding quickly, but short-circuiting any of the real processing code, so it is not behaving as expected. No code changes were made this past week, so we suspect it may be related to the Redis upgrade. The job cancellation code calls Redis under the hood.

My team (Lighthouse Team Banana Peels) maintains the vba_documents module.

Related issue(s)

None, as this was discovered as a production issue.

Testing done

Ensured that all Benefits Intake endpoints still return their expected response after removing the job cancellation code, and that records can be processed locally.

Screenshots

None

What areas of the site does it impact?

This PR impacts the vba_documents module only.

Acceptance criteria

• I fixed|updated|added unit tests and integration tests for each feature (if applicable).
• No error nor warning in the console.
• Events are being sent to the appropriate logging solution
• Documentation has been updated (link to documentation) – N/A
• No sensitive information (i.e. PII/credentials/internal URLs/etc.) is captured in logging, hardcoded, or specs
• Feature/bug has a monitor built into Datadog or Grafana (if applicable)
• If app impacted requires authentication, did you login to a local build and verify all authenticated routes work as expected
• I added a screenshot of the developed feature – N/A

Requested Feedback

No specific feedback requested for this PR.