Closed lutien closed 9 months ago
At the moment, the spec for "browsingContext.locateNodes" command(step 18) says that we should serialize the list of the nodes, which would mean that the response would look like:
{ type: “array”, handle: “handle id”, value: [{type: “node”, sharedId: “shared id”, …}, …]} }
Where, in the merged tests, the expectation output should look like:
[{type: “node”, sharedId: “shared id”, handle: “handle id”, …}, …]
That also looks like more something clients would expect. To achieve the output specified in the tests, we should rather serialize each node individually and push them in the list after.
Preview | Diff
At the moment, the spec for "browsingContext.locateNodes" command(step 18) says that we should serialize the list of the nodes, which would mean that the response would look like:
Where, in the merged tests, the expectation output should look like:
That also looks like more something clients would expect. To achieve the output specified in the tests, we should rather serialize each node individually and push them in the list after.
Preview | Diff