Skip to content
Snippets Groups Projects

Update to selenium/standalone-chrome:133

Open Michael Strelan requested to merge issue/drupal-3515403:3515403-chrome-133 into 11.x
1 unresolved thread

Closes #3515403

Merge request reports

Members who can merge are allowed to add commits.
Code Quality is loading
Test summary results are being parsed
Ready to merge by members who can write to the target branch.
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
14 14 '@tags': ['core'],
15 15 before(browser) {
16 16 browser
17 // We need to resize the browser in order to grant permissions.
18 .setWindowSize(1024, 768)
  • Comment on lines +17 to +18

    Today I had the displeasure of learning the setup process of Nightwatch tests. Installing Drupal, modules, creating user roles, assigning permissions and creating users are all performed via the UI. In between each step the user is logged out and logged back in. When the user finally gets to the permissions page they have to manually check each box for each permission to assign. Due to whatever changed in standalone-chrome 128, the viewport is not big enough for the checkboxes to be clickable :exploding_head:. This results in a TimeoutError with element click intercepted messages. But the test doesn't fail! So we carry on we try to click on an element that doesn't exist, because we don't have adequate permissions! I sincerely hope someone is working towards refactoring these nightwatch setup steps so this can be API driven.

  • Michael Strelan changed this line in version 5 of the diff

    changed this line in version 5 of the diff

  • Please register or sign in to reply
  • added 1 commit

    • 405409b9 - Resize the browser in drupalCreateRole instead

    Compare with previous version

  • Michael Strelan added 2 commits

    added 2 commits

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Please register or sign in to reply
    Loading