Skip to content

[FIX] hr_recruitment_ux: use a non-PDF attachment in the talent pool test - #78

Closed
cav-adhoc wants to merge 1 commit into
ingadhoc:20.0from
adhoc-dev:20.0-talent-pool-test-cv-cav
Closed

cav-adhoc wants to merge 1 commit into
ingadhoc:20.0from
adhoc-dev:20.0-talent-pool-test-cv-cav

Conversation

@cav-adhoc

Copy link
Copy Markdown
Contributor

The talent pool test creates an attachment named cv.pdf with non-PDF content. When documents is installed, _get_is_multipage tries to read it as a PDF and logs a warning with a traceback, so the runbot build fails with "Traceback found in logs".

The test only checks which attachments exist, not their content, so the attachment is now named cv.txt.

Tarea: https://www.adhoc.inc/odoo/project.task/76640

…test

The test attachment had a .pdf name with non-PDF content. When documents
is installed, the attachment is read as a PDF and a warning with a
traceback is logged, which makes the runbot build fail.
@roboadhoc

Copy link
Copy Markdown
Contributor

Pull request status dashboard

@les-adhoc

Copy link
Copy Markdown
Contributor

@roboadhoc r+ nobump

@roboadhoc

Copy link
Copy Markdown
Contributor

@les-adhoc you may want to rebuild or fix this PR as it has failed CI.

@cav-adhoc

Copy link
Copy Markdown
Contributor Author

@roboadhoc retry

@roboadhoc

Copy link
Copy Markdown
Contributor

@cav-adhoc retry makes no sense when the PR is not in error.

@cav-adhoc

Copy link
Copy Markdown
Contributor Author

@roboadhoc check

@roboadhoc

Copy link
Copy Markdown
Contributor

Updated message. No update to pr head.

This branch was previously deployed

1 inactive deployment
merge — 652683d6 Deployed Nov 3, 2026 by roboadhoc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants