ExecuteAsyncScript: Only check for thenable #332

vmi / selenese-runner-java

Selenium IDE native format (selenese and side) interpreter.

Other

140 stars 83 forks source link

Closed blutorange closed 2 years ago

blutorange commented 2 years ago

Fixes #332, see there for more details
Add test cases for executeAsyncScript to illustrate the issue with Firefox
Only check whether promise is thenable, e.g. whether it is truthy and has a then property of type function

blutorange commented 2 years ago

That was quick, thanks. Doesn't have to be immediate, but it would great if you could make a patch release during the next few weeks.

vmi commented 2 years ago

@blutorange I released 4.1.0. Please check it out.

blutorange commented 2 years ago

@vmi I just tried it with our tests and it works now, thanks!