976832 - Investigate Mixin class alternatives

Status: RESOLVED WONTFIX

(Remote Protocol :: Marionette, defect)

Description

[Malini Das [:mdas] - Away, not checking bugmail]

From http://mxr.mozilla.org/mozilla-central/source/testing/marionette/client/marionette/marionette.py#380, perhaps instead of forcing the users to inherit mixin classes, we should investigate looking into adding the mixins through a function call. This came up as a problem when we needed to mix in some behaviour in some code that assumed one particular class was being used, so we had to update this class' bases to include this mixin on the fly just to get special runtime properties on it, which isn't ideal or elegant.

We have to consider any benefits/setbacks to making a client to inherit from theses classes versus mixing them in after instantiation.

Comment 1

[David Burns :automatedtester]

This is not something we should worry about now and do a full refactor of the runner at some stage