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

[tests-only][full-ci]Bump ocis commit to web #8969

Merged
merged 2 commits into from
May 5, 2023

Conversation

SagarGi
Copy link
Member

@SagarGi SagarGi commented May 3, 2023

Description

Bump latest ocis commit-id to web

Related issue:

owncloud/QA#807

@ownclouders
Copy link
Contributor

ownclouders commented May 3, 2023

Results for acceptance oCIS https://drone.owncloud.com/owncloud/web/35442/65/1
The following scenarios passed on retry:

  • webUISharingPublicBasic/publicLinkCreate.feature:85
💥 The acceptance tests failed on retry. Please find the screenshots inside ...

webUISharingPublicBasic-publicLinkCreate_feature-L11.png

webUISharingPublicBasic-publicLinkCreate_feature-L11.png

webUISharingPublicBasic-publicLinkCreate_feature-L28.png

webUISharingPublicBasic-publicLinkCreate_feature-L28.png

webUISharingPublicBasic-publicLinkCreate_feature-L45.png

webUISharingPublicBasic-publicLinkCreate_feature-L45.png

webUISharingPublicBasic-publicLinkCreate_feature-L61.png

webUISharingPublicBasic-publicLinkCreate_feature-L61.png

webUISharingPublicBasic-publicLinkCreate_feature-L75.png

webUISharingPublicBasic-publicLinkCreate_feature-L75.png

@SagarGi SagarGi force-pushed the bump-ocis-latest-commit-3 branch from c1eb586 to ab33059 Compare May 4, 2023 03:38
@SagarGi SagarGi self-assigned this May 4, 2023
@SagarGi SagarGi added the QA:team label May 4, 2023
@SagarGi SagarGi force-pushed the bump-ocis-latest-commit-3 branch from ab33059 to fbbe71e Compare May 4, 2023 05:06
@SagarGi SagarGi force-pushed the bump-ocis-latest-commit-3 branch from fbbe71e to 6db4a1a Compare May 5, 2023 03:31
@SagarGi
Copy link
Member Author

SagarGi commented May 5, 2023

https://drone.owncloud.com/owncloud/web/35441/66/13 this test seems to be the hard failure when bumping latest ocis commit.

@SagarGi
Copy link
Member Author

SagarGi commented May 5, 2023

Results for acceptance oCIS https://drone.owncloud.com/owncloud/web/35441/66/1

The acceptance tests failed on retry. Please find the screenshots inside ...

failed 4 times in the same test feature scenarios

@ownclouders
Copy link
Contributor

ownclouders commented May 5, 2023

Results for acceptance oC10 https://drone.owncloud.com/owncloud/web/35442/71/1

💥 The acceptance tests failed on retry. Please find the screenshots inside ...

webUISharingPublicBasic-publicLinkCreate_feature-L11.png

webUISharingPublicBasic-publicLinkCreate_feature-L11.png

webUISharingPublicBasic-publicLinkCreate_feature-L28.png

webUISharingPublicBasic-publicLinkCreate_feature-L28.png

@SagarGi SagarGi force-pushed the bump-ocis-latest-commit-3 branch from 9dc3418 to bef176c Compare May 5, 2023 06:25
@SagarGi
Copy link
Member Author

SagarGi commented May 5, 2023

Findings:

linkpopup

when we have many pop up message notification. It seems to block the button to be clicked untill the pop is gone. untill then the wait for the time to click the button ends which is the reason why the test is failing. So i have done some refactor in this same PR to wait the pop not to be visible and the only after that click the button.

@sonarcloud
Copy link

sonarcloud bot commented May 5, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@SagarGi SagarGi marked this pull request as ready for review May 5, 2023 06:57
@saw-jan saw-jan merged commit 090e11e into master May 5, 2023
@delete-merged-branch delete-merged-branch bot deleted the bump-ocis-latest-commit-3 branch May 5, 2023 07:59
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