Skip to content

matchOption: fix for broken up select#58

Closed
johnf wants to merge 1 commit intoCodeception:masterfrom
johnf:patch-1
Closed

matchOption: fix for broken up select#58
johnf wants to merge 1 commit intoCodeception:masterfrom
johnf:patch-1

Conversation

@johnf
Copy link

@johnf johnf commented May 15, 2022

Without this I wasn't getting matches on

<option value="1">
  String
</option>

Without this I wasn't getting matches on
```html
<option value="1">
  String
</option>
@Naktibalda
Copy link
Member

It would be great if you could create a test for this case.

Similar test: https://github.com/Codeception/lib-innerbrowser/blob/master/tests/unit/Codeception/Module/TestsForWeb.php#L309-L317
Form that you can reuse https://github.com/Codeception/lib-innerbrowser/blob/master/tests/data/app/view/form/select.php (either modify the option not used by other tests or add your own).

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants