979923 - Refactor Mozmill tests to make use of fat arrow functions

Status: RESOLVED WONTFIX

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

This feature is great and we might want to make use of it. Especially to get rid of the sometimes really confusing 'this' scope. With array functions it's much cleaner.

Here an example:

ARRAY.map(s => s.length);

This is equivalent to:

ARRAY.map(function (s) { return s.length });

Comment 1

[Stefan]

I'll try this one!

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Wonderful. Thanks Stefan! Let me know if you need help.

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Stefan, just one more thing. I think it would be helpful if you could do those changes by testrun. At least for Firefox the patch would become too massive. So I would suggest:

* Patch for all library modules
* Patch for functional testrun of Firefox
* Patch for Remote testrun of Firefox
* Patch for all other testruns and products

How does that sound?

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+2]]

We will transition our Mozmill tests to Marionette soon. Given that no action happened here in the last year I will close the bug as wontfix.