Closed Bug 1070307 Opened 10 years ago Closed 10 years ago

when using findElements we dont return a dict with key ELEMENT like we do on findElement

Categories

(Testing :: Marionette Client and Harness, defect, P1)

Product:
Testing
Component:
Marionette Client and Harness
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla38

People

(Reporter: automatedtester, Assigned: automatedtester)

References

(Blocks 1 open bug)

Details

(Keywords: pi-marionette-client, pi-marionette-server, Whiteboard: [marionette=1.0])

Attachments

(1 file, 1 obsolete file)

MozReview Request: Bug 1070307: have findElements return an array of objects with a key instead of just an array of strings; r=jgriffin
(deleted), text/x-review-board-request
jgriffin
: review+
Details
This is clearly wrong and we should return [{ "ELEMENT": <some UUID> }, { "ELEMENT": <some UUID> }] instead of [<some UUID>, <some UUID>, <some UUID>] This will allow us to have webdriver compat too
Assignee: nobody → dburns
Blocks: webdriver
OS: Mac OS X → All
Hardware: x86 → All
Whiteboard: [marionette=1.0]
Priority: -- → P1
Attached file MozReview Request: bz://1070307/AutomatedTester (obsolete) (deleted) —
/r/2767 - Bug 1070307: have findElements return an array of objects with a key instead of just an array of strings; r=jgriffin Pull down this commit: hg pull review -r b5b817d35c3821d766550744a88a4cd5faca7ec5
Attachment #8552084 - Flags: review?(jgriffin)
Comment on attachment 8552084 [details] MozReview Request: bz://1070307/AutomatedTester https://reviewboard.mozilla.org/r/2765/#review2001 Hooray for consistency!
Attachment #8552084 - Flags: review?(jgriffin) → review+
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/4594c8c47c57
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
Depends on: 1124815
Attachment #8552084 - Attachment is obsolete: true
Attachment #8618374 - Flags: review+
Product: Testing → Remote Protocol

Moving bugs for Marionette client due to component changes.

Component: Marionette → Marionette Client and Harness
Product: Remote Protocol → Testing
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: