w3c / w3c/webdriver

Execute script tries to set a data property on errors

Open
#1,615 6 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Execute Script
Dominant language
HTML
Stars
718
Forks
228
PR merge metrics
No merged PRs in 30d

Description

e.g. https://w3c.github.io/webdriver/#execute-script step 8 says:

Upon rejection of promise with reason r, let result be a JSON clone of r, and return error with error code javascript error and data result.

But there's no data property on error objects, so this doesn't make sense. The way the spec currently works it's an implementation detail whether the UA fills in useful information about the error or not.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the WebDriver specification's Execute Script algorithm and inspect step 8 alongside the definition of WebDriver error objects. Determine how the rejected promise's reason and error data are intended to be represented, then update the specification so the algorithm and error definition agree.

Written by the indexing model from the issue text.

Assessment

Domain
documentation
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.