1194219 - [gui] add the ability to use addons

Status: RESOLVED FIXED

(Testing :: mozregression, defect)

Description

[Julien Pagès (:parkouss)]

This is equivalent to:

-e PATH1, --addon PATH1
                        addon to install; repeat for multiple addons.

command line option.

So, I think we should add in the Profile wizard page a list to be able to add/remove addons. This could looks like: http://doc.qt.io/qt-4.8/images/assistant-preferences-documentation.png

After this is done, we should pass the addon list (or None if there is none) to the Bisection - from memory, it should looks like the profile. The code in mozregression/main.py might help: https://github.com/mozilla/mozregression/blob/0bfa1b8df0d865e3641814a1aa6fefa7c0c11ff9/mozregression/main.py#L361.

Comment 1

[MikeLing]

Attachment: PR for bug 1194219

Hmm, not sure about it. No error shows up, but when I test it locally, It seems like also found nothing to me. Just ask feedback for now, and I will keep diging it until I figure out the problem.

Comment 2

[Julien Pagès (:parkouss)]

Comment on attachment 8649340 [details] [review]
PR for bug 1194219

It works for me with a simple fix:

diff --git a/gui/mozregui/bisection.py b/gui/mozregui/bisection.py
index 61ebc44..063b4e2 100644
--- a/gui/mozregui/bisection.py
+++ b/gui/mozregui/bisection.py
@@ -258,8 +258,7 @@ class BisectRunner(QObject):
                     launcher_kwargs[name] = value
 
         # add add-ons path to the app luncher
-        for path in options['addons']:
-            launcher_kwargs['addons'] = path
+        launcher_kwargs['addons'] = options['addons']
         self.bisector.test_runner.launcher_kwargs = launcher_kwargs
 
         self.thread.start()

I am able to run a process with an addon after that. :) This is just because launcher_kwargs['addons'] needs a list of string.

Also, I will make some comment on the PR, but otherwise this is good!

Comment 3

[Julien Pagès (:parkouss)]

Comment on attachment 8649340 [details] [review]
PR for bug 1194219

This is great!

Thanks MikeLing - I did not tested yet but I assume it works (will try it before merging). Have you tested it ?

I commented a few things on travis PR, just minor nits.

I'll be happy to merge this in after you fixed them!

Comment 4

[MikeLing]

(In reply to Julien Pagès from comment #3)
> Comment on attachment 8649340 [details] [review]
> PR for bug 1194219
> 
> This is great!
> 
> Thanks MikeLing - I did not tested yet but I assume it works (will try it
> before merging). Have you tested it ?
> 
> I commented a few things on travis PR, just minor nits.
> 
> I'll be happy to merge this in after you fixed them!

Just test the PR and it works fine I think :)

Comment 5

[Julien Pagès (:parkouss)]

Thanks MikeLing for the work here!

Just merged this manually in https://github.com/mozilla/mozregression/commit/bd1110e2ee242450458663dd2a1c7e1155a89f7f.