Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 6, 2025

The PDF processing workflow fails in the docker-tests job because it references test_pdf_processing.py, which doesn't exist. The actual test file is test_pdf_processor_unit.py.

Changes

.github/workflows/pdf_processing_ci.yml (line 335):

- python -m pytest tests/unit/test_pdf_processing.py::TestPDFProcessor::test_pdf_processor_initialization -v
+ python -m pytest tests/unit/test_pdf_processor_unit.py::TestPDFProcessorInitialization::test_given_no_parameters_when_initializing_pdf_processor_then_creates_with_defaults -v

Corrects both the filename and the test path to match the actual test structure that follows GIVEN-WHEN-THEN naming conventions.

Original prompt

Fix PDF Processing Pipeline CI/CD workflow failure (issue #738)

The pdf_processing_ci.yml workflow is failing. Please analyze the workflow logs, identify the root cause, and implement a fix.

References:

Please ensure all PDF processing tests pass after your fix.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Fix PDF processing pipeline CI/CD workflow failure Fix docker-tests job referencing non-existent test file in pdf_processing_ci.yml Nov 6, 2025
Copilot AI requested a review from endomorphosis November 6, 2025 21:25
Copilot finished work on behalf of endomorphosis November 6, 2025 21:25
@endomorphosis endomorphosis marked this pull request as ready for review November 6, 2025 21:30
@endomorphosis endomorphosis merged commit cdc8bdd into main Nov 6, 2025
4 of 5 checks passed
@endomorphosis endomorphosis deleted the copilot/fix-pdf-processing-ci-workflow branch November 6, 2025 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants