Fix: _WD_ElementSelectAction / selectAll error handling

[Danp2]

Pull request

Proposed changes

Unlike the structurally identical multiSelect/singleSelect cases, the selectAll case's $vResult = False check wasn't guarded by Not @error.

On a genuine _WD_ExecuteScript failure, AutoIt's loose comparison coerces the non-numeric failure value to 0, so $vResult = False evaluated True and overwrote the real error code with $_WD_ERROR_NoMatch, hiding the actual failure from callers.

Checklist

Put an x in the boxes that apply. If you're unsure about any of them, don't hesitate to ask. We are here to help!

This is simply a reminder of what we are going to look for before merging your code.

• I have read and noticed the CODE OF CONDUCT document
• I have read and noticed the CONTRIBUTING document
• I have added necessary documentation or screenshots (if appropriate)

Types of changes

Please check x the type of change your PR introduces:

• Bugfix (change which fixes an issue)
• Feature (change which adds functionality)
• Code style update (formatting, renaming)
• Refactoring (functional, structural)
• Documentation content changes
• Other (please describe)

What is the current behavior?

Please describe the current behavior that you are modifying, or link to a relevant issue.

What is the new behavior?

Please describe the behavior or changes that are being added by this PR.

Influences and relationship to other functionality

Describe how the changes will affect other functions, potential script breaking changes, etc.

Additional context

Add any other context about the problem here.

System under test

Please complete the following information.

• OS: [e.g. Windows 10]
• OS Arch.: [e.g. X64]
• Browser [e.g. firefox]
• Browser version [e.g. 96.0.3]