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 18:21, 8 November 2010
, 8 November 2010→Refactoring
| Line 34: | Line 34: | ||
Dependencies: Agreed upon style guidelines | Dependencies: Agreed upon style guidelines | ||
== Refactoring == | == Refactoring == | ||
=== Variable Naming === | |||
;Contants | |||
All ''unchanging'' variables should be declared as constants. All constants should be named in ALL_CAPS with whitespace indicated by an underscore (_). All constants should be declared directly the ''required modules'' block and setupModule() function. | |||
Example: | |||
<pre>* ***** END LICENSE BLOCK ***** */ | |||
// Include required modules var utils = require("../../shared-modules/utils"); | |||
const TIMEOUT = 5000; | |||
var setupModule = function() { </pre> | |||