Closed Bug 865191 Opened 11 years ago Closed 8 years ago

'args' parameter should be allowed to be null in execute* calls

Categories

(Remote Protocol :: Marionette, defect)

Product:
Remote Protocol
Component:
Marionette
Platform:
x86
All
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WORKSFORME

People

(Reporter: mdas, Assigned: automatedtester)

Details

(Keywords: pi-marionette-task) 

I was working on supporting our Marionette calls with an external product and it turns out that we don't support sending null values over as our 'args' variable. We expect "no values" to be [] instead of null. If we get null, we throw an error. We should accept null!
I tried to recreate this, removing the None -> [] code from the client code and it worked as expected. Do you still think that this is still an issue?
Pushed to try

mobile:
https://tbpl.mozilla.org/?tree=Try&rev=6a683822592e

desktop:
https://tbpl.mozilla.org/?tree=Try&rev=972d6b377c84
Assignee: nobody → dburns
Flags: needinfo?(dburns)
Flags: needinfo?(dburns)
Keywords: ateam-marionette-task
Can't recreate the issue
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.