- Committer: Brad Crittenden
- Date: 2012-09-09 19:57:35 UTC
- mfrom: (82.4.16 jshint)
- Revision ID: bac@canonical.com-20120909195735-y8ssoac8232mbmuk
Update mocha.js, wire up jshint, fix lint.
A problem with the old version of mocha.js caused the tests to fail in
Firefox. The version in test/assets was manually updated and the
version bumped in package.json.
A 'lint' target was added to the Makefile. All of the reported lint
was either fixed or silenced.
Many of the files were changed to remove tabs, one thing jshint complains about. Trailing whitespace was also deleted.
jshint complained about the following idiom:
a.foo && a.foo() // ensure the property foo exists before referencing it.
There were only two occurrences so they were fixed by converting to use 'if'.
R=gary.poster, benjamin.saller
CC=
https://codereview.appspot.com/6488096
A problem with the old version of mocha.js caused the tests to fail in
Firefox. The version in test/assets was manually updated and the
version bumped in package.json.
A 'lint' target was added to the Makefile. All of the reported lint
was either fixed or silenced.
Many of the files were changed to remove tabs, one thing jshint complains about. Trailing whitespace was also deleted.
jshint complained about the following idiom:
a.foo && a.foo() // ensure the property foo exists before referencing it.
There were only two occurrences so they were fixed by converting to use 'if'.
R=gary.poster, benjamin.saller
CC=
https://codereview.appspot.com/6488096
| Filename | Latest Rev | Last Changed | Committer | Comment | Size | ||
|---|---|---|---|---|---|---|---|
 ..
|
|||||||
assets
|
53 | 12 years ago | kapil.foss at gmail | test harness using mocha in browser |
|
||
index.html |
82.4.4 | 12 years ago | Brad Crittenden | Cleaned up lint. Tests still pass. | 1.3 KB |
|
|
|
mocha.opts |
46 | 12 years ago | kapil.foss at gmail | do nothing unit view, comment some of modules.js o | 68 bytes |
|
|
|
test_app.js |
82.4.6 | 12 years ago | Brad Crittenden | Cleaned up lint. Tests still pass. | 2 KB |
|
|
|
test_env.js |
82.4.6 | 12 years ago | Brad Crittenden | Cleaned up lint. Tests still pass. | 2.4 KB |
|
|
|
test_model.js |
82.4.12 | 12 years ago | Brad Crittenden | Merge from trunk and fix conflicts | 3.4 KB |
|
|