-
Notifications
You must be signed in to change notification settings - Fork 159
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]Refactor webUILogin/adminBlocksUser.feature:20 to make it follow the current behaviour #8074
Conversation
weird the screenshot shows that the user is navigated to the login page but locally for me they are navigated to the disabled page. Need to look further into it. |
fea04d7
to
33818c6
Compare
140b230
to
ae5c19d
Compare
Results for e2e-tests oC10 https://drone.owncloud.com/owncloud/web/30598/11/1 💥 To see the trace, please open the link in the console ...
npx playwright show-trace https://cache.owncloud.com/public/owncloud/web/30598/tracing/upload-resources-in-personal-space-alice-2022-12-9-10-59-41.zip |
6636a1d
to
3bc76ad
Compare
cbb1fa7
to
80e69c1
Compare
d6fc429
to
a2c536e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
… current behaviour use the user disabled page step change the selector fix lint try with single selector
a2c536e
to
025a777
Compare
Kudos, SonarCloud Quality Gate passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm 👍
…o make it follow the current behaviour #8074
…ble branch (#8138) * Refactor test steps * [tests-only][full-ci]Refactor webUILogin/adminBlocksUser.feature:20 to make it follow the current behaviour #8074 * Remove tests from expected to failure linked to closed issue * Skip private link related tests on ocis * add notToImplementOnOCIS tag instead of skip on * Skip shares related tests in OCIS * Skip tests that open shares folder in ocis * Remove steps that browses to panel * update expected to fail
Description
The changed test was failing because it was not compatible with the current behavior of the system, this PR makes it work according to the current behavior and removes closed issue #4795 from the expected to fail.
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist:
Open tasks: