1116216 - talos generateBrowserCommandLine could benefit from a list of extra parameters

Status: RESOLVED FIXED

(Testing :: Talos, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

Right now the Talos function generateBrowserCommandLine in utils.py takes a string as extra parameters:
http://hg.mozilla.org/build/talos/file/tip/talos/utils.py#l223

It really should be an array, which means that we could extend the args with more meaningful parameters to the browser.

of course allowing for an array means we should consider the callers.  Ideally you could determine if the type is an array and if so, just extend the parameters, otherwise we can assume it is a string and handle it like we currently do.

to learn more about Talos and how to get started here is a wiki outlining the work:
https://wiki.mozilla.org/Auto-tools/Projects/Talos

Comment 1

[Alice Scarpa [:adusca]]

Attachment: Tentative fix

Comment 2

[Alice Scarpa [:adusca]]

Hi! I'd like to work on this bug. Is the above patch what you had in mind?

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8543537 [details] [diff] [review]
Tentative fix

Review of attachment 8543537 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks for this patch and congratulations on getting your first patch r+!

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

https://hg.mozilla.org/build/talos/rev/c59ed57318c7

Comment 5

[Joel Maher ( :jmaher ) (UTC -8)]

Alice, we have a lot of great projects underway related to automation:
https://wiki.mozilla.org/Auto-tools/Projects/Everything#Project_Table

I would be happy to help you get started on one of those or any specific bug!  

Thanks again for your contribution, this really helps clean up our code and make it more robust.