Skip to content
Snippets Groups Projects

#3458503: Improve the page hierarchy display

Merged Harumi Jang requested to merge issue/experience_builder-3458503:3458503-improve-layers into 0.x

Closes #3458503

:rotating_light:This MR still has the temporary cy.wait()'s in tests that are to be replaced but everything else can still be reviewed.

This MR uses cy.wait()'s in tests that are marked with a TODO to replace them.

Edited by Harumi Jang

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
16 });
17
18 it('Drag a component from the column one slot to the root level then to the column two slot', () => {
19 cy.loadURLandWaitForXBLoaded();
20 cy.get('.primaryMenuContent').find('.treeItem[data-xb-uuid="two-column-uuid"]').find('button').click();
21 cy.get('.primaryMenuContent').find('.treeItem[data-xb-uuid="two-column-uuid-slot-column_one"]').find('button').click();
22
23 // Before dragging, check that the component is in the column one slot in the layers menu and preview.
24 cy.get('.treeItem[data-xb-uuid="dynamic-image-udf7d"]').parent('[data-xb-uuid="two-column-uuid-slot-column_one"]').should('exist');
25 cy.get('.treeItem[data-xb-uuid="dynamic-image-udf7d"]').siblings('.treeItem[data-xb-uuid="two-column-uuid"]').should('not.exist');
26 cy.getIframeBody().find('[data-xb-uuid="dynamic-image-udf7d"]').parent('[data-xb-uuid="two-column-uuid-slot-column_one"]').should('exist');
27 cy.getIframeBody().find('[data-xb-uuid="dynamic-image-udf7d"]').siblings('[data-xb-uuid="two-column-uuid"]').should('not.exist');
28
29 // Drag image component out of the slot and to the root level.
30 cy.get('.treeItem[data-xb-uuid="dynamic-image-udf7d"]').realDnd('.rootNodeWrapper[data-xb-uuid="root"]');
31 cy.wait(1000);
  • :thinking: Ideally we wouldn't need wait for a second but instead wait for something observable for at most a second. That'd speed this test up.

    If not feasible, can we at least document the justification for using wait in a comment at the top of this *.cy.js file, because it is used a lot? :pray:

  • Ah I see this in the MR description now:

    :rotating_light:This MR still has the temporary cy.wait()'s in tests that are to be replaced but everything else can still be reviewed.

    :thumbsup: I'm fine with this landing as-is, as long as that's explicitly called out as a @todo then :nerd:

  • Added a todo on the test linking to Drupal issue

  • Please register or sign in to reply
  • Works well! :blush: :thumbsup:

    This MR is 100% JS, so no deep review of the logic from me, only making sure I understand the E2E test. One question/request about that.

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading