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

NEW
Unassigned

Status

Thunderbird
Testing Infrastructure
--
enhancement
a year ago
a year ago

People

(Reporter: aceman, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

a year ago
+++ 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.)
You need to log in before you can comment on or make changes to this bug.