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

BUG: The new conflict resolution e2e tests are sadly a bit flaky in ci :O #3785

Open
mhsdesign opened this issue May 18, 2024 · 0 comments · May be fixed by #3769
Open

BUG: The new conflict resolution e2e tests are sadly a bit flaky in ci :O #3785

mhsdesign opened this issue May 18, 2024 · 0 comments · May be fixed by #3769
Labels
9.0 Bug Label to mark the change as bugfix Technical debt Tooling / Testing

Comments

@mhsdesign
Copy link
Member

The new e2e tests are sadly a bit flaky in ci :O

Originally posted by @mhsdesign in #3762 (comment)

 Syncing
 ✖ Syncing: Create a conflict state between two editors and choose "Discard all" as a resolution strategy during rebase

   1) - Error while restoring configuration after Role switch -
      A JavaScript error occurred on "http://onedimension.localhost:8081/neos/content?node=user-editor__eyJsYW5ndWFnZSI6ImVuX1VTIn0%3D__f676459d-ca77-44bc-aeea-44114814c279".
      Repeat test actions in the browser and check the console for errors.
      Enable the “skipJsErrors” option to ignore JavaScript errors during test execution. Learn more: "https://testcafe.io/documentation/404038/recipes/debugging/skip-javascript-errors"
      If the website only throws this error when you test it with TestCafe, please create a new issue at:
      "https://github.com/DevExpress/testcafe/issues/new?template=bug-report.md".

      JavaScript error details:
      Error: NetworkError when attempting to fetch resource.
      generalErrorHandler@http://onedimension.localhost:8081/_Resources/Static/Packages/Neos.Neos.Ui/Build/Host.js?b403a802:14:155902
      Lye</zye</m</<@http://onedimension.localhost:8081/_Resources/Static/Packages/Neos.Neos.Ui/Build/Host.js?b403a802:15:28034

      Browser: Firefox 126.0 / Windows 10 (https://app.saucelabs.com/tests/65e9848a30964224968563e3504567e3)

         138 |            timeout: 30000
         139 |        });
         140 |}
         141 |
         142 |async function switchToRole(t, role) {
       > 143 |    await t.useRole(role);
         144 |    await waitForReact(30000);
         145 |    await Page.goToPage('Home');
         146 |}
         147 |
         148 |async function chooseDiscardAllAndFinishSynchronization(t) {

         at <anonymous> (/home/circleci/app/Packages/Application/Neos.Neos.Ui/Tests/IntegrationTests/Fixtures/1Dimension/syncing.e2e.js:143:13)
         at asyncGeneratorStep (/home/circleci/app/Packages/Application/Neos.Neos.Ui/Tests/IntegrationTests/Fixtures/1Dimension/syncing.e2e.js:7:28)
         at _next (/home/circleci/app/Packages/Application/Neos.Neos.Ui/Tests/IntegrationTests/Fixtures/1Dimension/syncing.e2e.js:7:28)
         at <anonymous> (/home/circleci/app/Packages/Application/Neos.Neos.Ui/Tests/IntegrationTests/Fixtures/1Dimension/syncing.e2e.js:7:28)
         at <anonymous> (/home/circleci/app/Packages/Application/Neos.Neos.Ui/Tests/IntegrationTests/Fixtures/1Dimension/syncing.e2e.js:7:28)
         at switchToRole (/home/circleci/app/Packages/Application/Neos.Neos.Ui/Tests/IntegrationTests/Fixtures/1Dimension/syncing.e2e.js:142:28)
         at <anonymous> (/home/circleci/app/Packages/Application/Neos.Neos.Ui/Tests/IntegrationTests/Fixtures/1Dimension/syncing.e2e.js:128:11)
         at asyncGeneratorStep (/home/circleci/app/Packages/Application/Neos.Neos.Ui/Tests/IntegrationTests/Fixtures/1Dimension/syncing.e2e.js:7:28)
         at _next (/home/circleci/app/Packages/Application/Neos.Neos.Ui/Tests/IntegrationTests/Fixtures/1Dimension/syncing.e2e.js:7:28)


 ✖ Syncing: Create a conflict state between two editors and choose "Drop conflicting changes" as a resolution strategy during rebase

   1) The specified selector does not match any element in the DOM tree.

       > | Selector('#neos-workspace-rebase')

      Browser: Firefox 126.0 / Windows 10 (https://app.saucelabs.com/tests/65e9848a30964224968563e3504567e3)

         67 |    await switchToRole(t, editorUserOnOneDimensionTestSite);
         68 |
         69 |    //
         70 |    // Sync changes from "admin"
         71 |    //
       > 72 |    await t.click(Selector('#neos-workspace-rebase'));
         73 |    await t.click(Selector('#neos-SyncWorkspace-Confirm'));
         74 |    await t.wait(1000);
         75 |
         76 |    //
         77 |    // Assert that all 3 documents are now visible in the document tree

         at <anonymous> (/home/circleci/app/Packages/Application/Neos.Neos.Ui/Tests/IntegrationTests/Fixtures/1Dimension/syncing.e2e.js:72:13)
         at asyncGeneratorStep (/home/circleci/app/Packages/Application/Neos.Neos.Ui/Tests/IntegrationTests/Fixtures/1Dimension/syncing.e2e.js:7:28)
         at _next (/home/circleci/app/Packages/Application/Neos.Neos.Ui/Tests/IntegrationTests/Fixtures/1Dimension/syncing.e2e.js:7:28)
@grebaldi grebaldi changed the title The new conflict resolution e2e tests are sadly a bit flaky in ci :O BUG: The new conflict resolution e2e tests are sadly a bit flaky in ci :O May 21, 2024
@grebaldi grebaldi added Bug Label to mark the change as bugfix Tooling / Testing Technical debt 9.0 labels May 21, 2024
grebaldi added a commit that referenced this issue May 21, 2024
This is to prevent the flakiness described in #3785.
grebaldi added a commit that referenced this issue May 27, 2024
grebaldi added a commit that referenced this issue Jun 14, 2024
This is to prevent the flakiness described in #3785.
grebaldi added a commit that referenced this issue Jun 14, 2024
grebaldi added a commit that referenced this issue Jun 28, 2024
This is to prevent the flakiness described in #3785.
grebaldi added a commit that referenced this issue Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
9.0 Bug Label to mark the change as bugfix Technical debt Tooling / Testing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants