604703 - Refactor tests for general formatting guidelines

Status: RESOLVED INCOMPLETE

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

[u279076]

This is a tracking bug for phase I of the Refactor Tests project.  In this bug, we will address general formatting issues across the tests.

This phase will consist of the following tasks:
1. Make sure all constants use the SOME_CONSTANT naming convention
2. Move all opening braces to be in-line with if() and function()
3. Remove all instances of module parameter in setupModule() and teardownModule()
4. Make sure any Component.Utils and Component.Classes declarations use proper Mozilla-style formatting
5. Make sure any arrays use the "single element per line block formatting" style
6. Make sure every indentation is only 2-spaces

Comment 1

[u279076]

I expect this work to clash with the CommonJS landing, so I'll hold off on doing it until CommonJS is landed and the tree is reopened.

Comment 2

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

Move of Mozmill Test related project bugs to newly created components. You can
filter out those emails by using "Mozmill-Tests-to-MozillaQA" as criteria.

Comment 3

[u279076]

Moving into a [mozmill-refactor] bucket.

Comment 4

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

This tracking bug is not needed anymore. Closing out.