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

[QA] WebUITests fails while asserting breadcrumb visibility #7385

Closed
PrajwolAmatya opened this issue Sep 29, 2023 · 5 comments · Fixed by owncloud/web#9761
Closed

[QA] WebUITests fails while asserting breadcrumb visibility #7385

PrajwolAmatya opened this issue Sep 29, 2023 · 5 comments · Fixed by owncloud/web#9761
Labels

Comments

@PrajwolAmatya
Copy link
Contributor

PrajwolAmatya commented Sep 29, 2023

Multiple builds failing in breadCrumb.feature

Scenario: Check breadCrumb for home folder (attempt 1, retried) # features/webUIFiles/breadcrumb.feature:50
    ✔ Before # setup.js:38
    ✔ Before # setup.js:42
    ✔ Before # setup.js:53
    ✔ Before # setup.js:60
    ✔ Before # setup.js:71
    ✔ Before # stepDefinitions/filesContext.js:17
    ✔ Before # stepDefinitions/generalContext.js:196
    ✔ Before # stepDefinitions/generalContext.js:234
    ✔ Before # stepDefinitions/middlewareContext.js:46
    ✔ Given user "Alice" has been created with default attributes and without skeleton files in the server # stepDefinitions/middlewareContext.js:66
    ✔ Given the property "homeFolder" of "options" has been set to "/0" in web config file # stepDefinitions/generalContext.js:34
    ✔ And user "Alice" has logged in using the webUI # stepDefinitions/loginContext.js:68
    ✖ Then breadcrumb for folder "0" should be displayed on the webUI # stepDefinitions/filesContext.js:206
        Error while running "strictEqual" command: Failed [strictEqual]: (Resource //nav[@id="files-breadcrumb"]//*[(self::a or self::button)]/span[contains(text(),'0')] expected to be visible but is not visible .) - expected "true" but got: "false" (0ms)
            at assertBreadcrumbIsDisplayedFor (/drone/src/webTestRunner/tests/acceptance/stepDefinitions/filesContext.js:738:24)
            at processTicksAndRejections (node:internal/process/task_queues:95:5)
    ✔ After # stepDefinitions/middlewareContext.js:60
    ✔ After # stepDefinitions/generalContext.js:252
    ✔ After # stepDefinitions/generalContext.js:221
    ✔ After # stepDefinitions/generalContext.js:162
    ✔ After # setup.js:104
    ✔ After # setup.js:100
    ✔ After # setup.js:96
    ✔ After # setup.js:89
    ✔ After # setup.js:80

Builds:

@nabim777 nabim777 self-assigned this Sep 29, 2023
@PrajwolAmatya PrajwolAmatya changed the title [QA] WebUITests fails while asserting breadcrum visibility [QA] WebUITests fails while asserting breadcrumb visibility Sep 29, 2023
@nabim777
Copy link
Member

nabim777 commented Oct 2, 2023

Since there are some changes to webconfig file in this PR #7369, should we remove the following scenario or not? @ScharfViktor @saw-jan @kobergj

  Scenario: Check breadCrumb for home folder
    Given the property "homeFolder" of "options" has been set to "/0" in web config file
    And user "Alice" has logged in using the webUI
    Then breadcrumb for folder "0" should be displayed on the webUI

@saw-jan
Copy link
Member

saw-jan commented Oct 2, 2023

Since now we cannot update the web config during the test execution, I would vote to remove the scenario and the step defs.

// tests/acceptance/stepDefinitions/generalCntext.js

the property {string} has been set to {string} in web config file
the property {string} of {string} has been set to {string} in web config file
the property {string} of {string} has been set to true in web config file
the property {string} of {string} has been set to false in web config file
the property {string} of {string} is changed to true in web config file
the property {string} of {string} is changed to false in web config file
the property {string} has been deleted in web config file

@saw-jan
Copy link
Member

saw-jan commented Oct 3, 2023

Need to have a web bump in ocis

CC @ScharfViktor

@saw-jan
Copy link
Member

saw-jan commented Oct 4, 2023

web bump ready in #7404

@saw-jan
Copy link
Member

saw-jan commented Oct 6, 2023

web bump is done and webUI test pipelines are all green. Closing here

@saw-jan saw-jan closed this as completed Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants