Most WebAssert Exists and notExists methods should allow passing a $container

Open on Drupal.org →

Created on 11 June 2024, 11 months ago

Updated 12 September 2024, 8 months ago

Problem/Motivation

Some methods in WebAssert allow the scope to be limited to a TraversableElement $container, yet others do not. They should be consistent.

Currently supported:

buttonExists
buttonNotExists
selectExists
optionExists
optionNotExists
hiddenFieldExists
hiddenFieldNotExists
hiddenFieldValueEquals
hiddenFieldValueNotEquals
fieldDisabled
fieldEnabled

Not supported:

linkExists
linkExistsExact
linkNotExists
linkNotExistsExact
linkByHrefExists
linkByHrefExistsExact
linkByHrefNotExists
linkByHrefNotExistsExact

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

✨ Feature request

Status

Needs work

Version

11.0 🔥

Component

Last updated about 13 hours ago

Maintained by
🇬🇧United Kingdom @alexpott
🇨🇭Switzerland @berdir
🇩🇪Germany @dawehner
🇮🇹Italy @mondrake

Created by

🇦🇺Australia mstrelan

Live updates comments and jobs are added and updated live.

Needs tests
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

Sign in to follow issues

Merge Requests

!8368Most WebAssert Exists and notExists methods should allow passing a $container
Open
🇦🇺Australia mstrelan
updated 11 months ago

Comments & Activities

Issue created by @mstrelan
Merge request !8368Issue #3453818: Most WebAssert Exists and notExists methods should allow passing a $container → (Open) created by mstrelan
Status changed to Needs review 11 months ago6:01am 11 June 2024
Comment 11 months ago →
🇦🇺Australia mstrelan
Pipeline finished with Failed
11 months ago
Total: 714s
#196082
Pipeline finished with Success
11 months ago
Total: 597s
#196100
Status changed to Needs work 11 months ago10:18pm 11 June 2024
Comment 11 months ago →
🇦🇺Australia acbramley
Let's add some tests to https://git.drupalcode.org/project/drupal/-/blob/11.x/core/tests/Drupal/...

Probably could get away with a single test case to test all functions with containers?
Comment 8 months ago →
🇦🇺Australia mstrelan
The test mentioned in #12 has moved to https://git.drupalcode.org/project/drupal/-/blob/11.x/core/tests/Drupal/... and now that it's a unit test it should be much quicker to add the needed coverage.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024