Skip to content

fix(test): temporarily revert pdf viewer assertion#5578

Merged
elzody merged 1 commit into
mainfrom
fix/pdf-viewer-tests
Apr 21, 2026
Merged

fix(test): temporarily revert pdf viewer assertion#5578
elzody merged 1 commit into
mainfrom
fix/pdf-viewer-tests

Conversation

@elzody

@elzody elzody commented Apr 21, 2026

Copy link
Copy Markdown
Collaborator

There is a bug in the files_pdfviewer app that causes an internal server error when opening PDFs. There is a pull request there already, but until then we can assert that the PDF viewer does not exist, that way once it's fixed it'll start failing again, bringing it to our attention. This commit can be reverted once that happens.

@elzody elzody self-assigned this Apr 21, 2026
@elzody elzody requested a review from juliusknorr as a code owner April 21, 2026 17:08
@elzody elzody added the 3. to review Ready to be reviewed label Apr 21, 2026
There is a bug in the files_pdfviewer app that causes an internal server error when opening PDFs. There is a pull request there already, but until then we can assert that the PDF viewer does not exist, that way once it's fixed it'll start failing again, bringing it to our attention. This commit can be reverted once that happens.

Signed-off-by: Elizabeth Danzberger <elizabeth@elzody.dev>
@elzody elzody force-pushed the fix/pdf-viewer-tests branch from 922fecc to d578137 Compare April 21, 2026 17:20
@elzody elzody merged commit 88cd97c into main Apr 21, 2026
48 checks passed
@elzody elzody deleted the fix/pdf-viewer-tests branch April 21, 2026 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Ready to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants