Confirmed users
14,525
edits
QA/Mozmill Test Automation/Test Modules Refactor: Difference between revisions
QA/Mozmill Test Automation/Test Modules Refactor (view source)
Revision as of 20:29, 8 November 2010
, 8 November 2010→Style Guidelines
| Line 63: | Line 63: | ||
** Delay: Use discrete value | ** Delay: Use discrete value | ||
** Timeout: Use global TIMEOUT | ** Timeout: Use global TIMEOUT | ||
* Henrik: Timeouts should be encapsulated within a global shared module | * Henrik: Timeouts should be encapsulated within a global shared module | ||
* Geo: Use default params where appropriate, wrapper functions where not | * Geo: Use default params where appropriate, wrapper functions where not | ||
| Line 188: | Line 187: | ||
]; | ]; | ||
</pre> | </pre> | ||
; sleep | |||
* Proposed Guideline: | |||
** Avoid use of sleep() where possible and use waitFor() instead | |||
* Geo: If you’re only saving a few milliseconds and not causing a robustness issue, you should use sleep for simplicity’s sake. | |||