Closed Bug 604703 Opened 10 years ago Closed 8 years ago

Refactor tests for general formatting guidelines

Categories

(Mozilla QA Graveyard :: Mozmill Tests, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: u279076, Unassigned)

References

Details

(Whiteboard: [mozmill-refactor])

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
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.
Blocks: 604700
Depends on: 602365
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.
Product: Testing → Mozilla QA
Moving into a [mozmill-refactor] bucket.
Whiteboard: [mozmill-refactor]
Whiteboard: [mozmill-refactor] → [mozmill-test-refactor]
Whiteboard: [mozmill-test-refactor] → [mozmill-refactor]
This tracking bug is not needed anymore. Closing out.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → INCOMPLETE
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.