Confirmed users
14,525
edits
QA/Mozmill Test Automation/Test Modules Refactor/Style Guide: Difference between revisions
Jump to navigation
Jump to search
QA/Mozmill Test Automation/Test Modules Refactor/Style Guide (view source)
Revision as of 05:48, 19 November 2010
, 19 November 2010→Refactor
No edit summary |
|||
| Line 57: | Line 57: | ||
* Remove any trailing whitespace | * Remove any trailing whitespace | ||
* Make sure all tests have 1 newline EOF | * Make sure all tests have 1 newline EOF | ||
* Make sure | * Make sure most lines of code respect the 2-space indent rule | ||
* Make sure most lines of code respect the 80-char limit (wrap where necessary) | |||
* Investigate removal of sleep() in favour of waitFor() where possible | |||
* Extract any element-instantiation-as-parameter into it's own declaration | |||
* Replace for each() with Array.forEach() for arrays and for() for strings | |||
* Replace anonymous functions with named functions? | |||
* Ensure proper commenting of code | |||