SUBMISSION-167: Cleanup (remove hack used while compilation was under development). Added code to allow PDF-only to get past conditional requiring a review.#71
Open
DavidLFielding wants to merge 2 commits into
Conversation
…e ConfirmPreview.validate() to match the workflow's has_non_processing_content pattern. [SUBMISSION-167] David
…ode tests now set source_format=TEX explicitly (otherwise they'd hit the new lenient PDF/HTML path and stop raising). Adds PDF/HTML lenient-path tests and a PostScript test verifying it still requires preview. Covers all three reachable branches of the validator. [SUBMISSION-167] David
DavidLFielding
requested review from
bdc34,
bmaltzan,
jweiskoff and
norbusan
as code owners
bdc34
reviewed
Jun 9, 2026
| ) | ||
|
|
||
| # Minimal "working" submission (unannounced) | ||
| def _working_submission(uid: str = "u1"): |
Contributor
There was a problem hiding this comment.
Be aware that submit_ce/ui/conftest.py has fixtures which provide submissions in many stages of completion. This is likely to remain correct (having the correct state for a given stage) as we develop the software.
No need for change.
bdc34
reviewed
Jun 9, 2026
| def test_confirm_preview_pdf_no_preview_passes(): | ||
| """ | ||
| For PDF-only submissions, ConfirmPreview should pass even when | ||
| submission.preview is None -- the source PDF IS the preview, no |
Contributor
There was a problem hiding this comment.
In 1.5 we don't make the submitter preview the PDF file the uploaded to confirm that they didn't upload the wrong file? like their tax returns?
bdc34
approved these changes
Jun 9, 2026
DavidLFielding
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
We added a hack to allow us to continue working while the convert/compile service was not functional. This basically allow us to copy a PDF into the submission directory so we have a 'compiled' PDF available. Compilation service is working so we can delete this.
Also added code to update ConfirmPreview condition to allow non-processed formats like PDF and HTML. Existing tests that use ConfirmPreview were updated to include source format and additional tests were added to test new functionality.