Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[test-only]check that image is open #10565

Merged
merged 2 commits into from
Mar 27, 2024
Merged

Conversation

ScharfViktor
Copy link
Contributor

@ScharfViktor ScharfViktor commented Mar 7, 2024

related owncloud/ocis#8602

we had test for this bug but unfortunately the test didn't expect an error

the image viewer was open, but the image did not appearScreenshot 2024-03-07 at 17 39 25

I'm wait locators now
image

we can check the response, but sometimes that doesn't give us a 100% guarantee that everything works. Example, openning broken pdf file:
image

for txt or md files we can check content(ctr+c and evaluate clickboard) but it dificult when user viewer

Copy link

update-docs bot commented Mar 7, 2024

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@ScharfViktor
Copy link
Contributor Author

don't merge please before fixing owncloud/ocis#8602

Copy link
Member

@SagarGi SagarGi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test failing as the selector was changed. LGTM 👍 if CI is happy.

@ScharfViktor ScharfViktor force-pushed the checkThatImageIsOpen branch from a7b2e4d to 5bfde0b Compare March 27, 2024 08:50
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@ScharfViktor ScharfViktor added the Status:Needs-Review Needs review from a maintainer label Mar 27, 2024
@ScharfViktor ScharfViktor merged commit 67c42cf into stable-8.0 Mar 27, 2024
3 checks passed
@delete-merged-branch delete-merged-branch bot deleted the checkThatImageIsOpen branch March 27, 2024 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status:Needs-Review Needs review from a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants