1308784 - cleanup headers of TB mozmill tests (mail/test/mozmill/) - real test files

Status: RESOLVED WORKSFORME

(Thunderbird :: Testing Infrastructure, enhancement)

Description

[:aceman]

+++ This bug was initially created as a clone of Bug #876503 +++

Some of the mozmill tests contain unneded declarations in their headers and the needed ones are in various orders.

I propose this order:
const MODULE_NAME = '...';

const RELATIVE_ROOT = '../shared-modules';
const MODULE_REQUIRES = [...];

<other module imports - Cu.import() >

<this module constants, variables, setupModule, etc.>

Many of the files declare Components.{classes, interfaces, utils} which are told to be unneeded. So I try to clean up something.

This does not change the logic of including modules in any way. For those see bug 876250 and bug 876089.

The files in shared-modules directory were already converted in bug 876503.

(Inspired by the investigations of Tessarakt.)