jslint pass

src/jio.storage/localstorage.js

@@ -380,7 +380,7 @@
      */
     that.removeAttachment = function (command) {
       setTimeout(function () {
-        var doc, error, i, attachment_list;
+        var doc, error;
         error = function (word) {
           that.error({
             "status": 404,

test/jio.storage/localstorage.tests.js

 /*jslint indent: 2, maxlen: 80, nomen: true */
-/*global define, jIO, jio_tests, window, test, ok, deepEqual, sinon, expect */
+/*global define, module, jIO, jio_tests, window, test, ok, deepEqual, sinon,
+  expect */
 
 // define([module_name], [dependencies], module);
 (function (dependencies, module) {

test/queries/tests.js

 /*jslint indent: 2, maxlen: 80, nomen: true */
-/*global define, complex_queries, window, test, ok, deepEqual, sinon */
+/*global define, exports, require, module, complex_queries, window, test, ok,
+  deepEqual, sinon */
 
 // define([module_name], [dependencies], module);
 (function (dependencies, module) {
@@ -36,7 +37,7 @@
     ];
     complex_queries.QueryFactory.create('identifier: "a"').exec(doc_list);
     deepEqual(doc_list, [
-      {"identifier": "a"},
+      {"identifier": "a"}
     ], 'Document with several identifier should be removed');
 
     doc_list = [

test/run-qunit.js

 /*jslint indent: 2, maxlen: 80 */
-/*global require: true, phantom: true, document: true */
+/*global require, phantom, document, setInterval, clearInterval, console */
 
-"use strict";
+(function () {
+  "use strict";
 
-var system = require('system');
+  var system = require('system'), page;
 
-/**
- * Wait until the test condition is true or a timeout occurs. Useful for waiting
- * on a server response or for a ui change (fadeIn, etc.) to occur.
+  /**
+   * Wait until the test condition is true or a timeout occurs. Useful for
+   * waiting on a server response or for a ui change (fadeIn, etc.) to occur.
    *
    * @method waitFor
    * @param {Function} testFx Condition that evaluates to a boolean
@@ -15,7 +16,7 @@ var system = require('system');
    * @param {Number} time_out_millis The max amount of time to wait.
    *                                 If not specified, 10 sec is used.
    */
-function waitFor(testFx, onReady, time_out_millis) {
+  function waitFor(testFx, onReady, time_out_millis) {
     var maxtime_out_millis, start, condition, interval;
     maxtime_out_millis = time_out_millis || 10001;
     start = new Date().getTime();
@@ -38,22 +39,22 @@ function waitFor(testFx, onReady, time_out_millis) {
         }
       }
     }, 100); //< repeat check every 100ms
-}
+  }
 
-if (system.args.length !== 2) {
+  if (system.args.length !== 2) {
     console.log('Usage: run-qunit.js URL');
     phantom.exit(1);
-}
+  }
 
-var page = require('webpage').create();
+  page = require('webpage').create();
 
-// Route "console.log()" calls from within the Page context to the main Phantom
-// context (i.e. current "this")
-page.onConsoleMessage = function (msg) {
+  // Route "console.log()" calls from within the Page context to the main
+  // Phantom context (i.e. current "this")
+  page.onConsoleMessage = function (msg) {
     console.log(msg);
-};
+  };
 
-page.open(system.args[1], function (status) {
+  page.open(system.args[1], function (status) {
     if (status !== "success") {
       console.log("Unable to access network");
       phantom.exit(1);
@@ -75,9 +76,10 @@ page.open(system.args[1], function (status) {
         console.log(el.innerText);
         try {
           return el.getElementsByClassName('failed')[0].innerHTML;
-      } catch (e) { }
+        } catch (ignore) { }
         return 10000;
       });
       phantom.exit((parseInt(failedNum, 10) > 0) ? 1 : 0);
     });
-});
+  });
+}());

test/tests.require.js

@@ -58,6 +58,6 @@
     "gidstorage_tests",
     "davstorage_tests",
     "xwikistorage_tests",
-    "s3storage_tests",
+    "s3storage_tests"
   ]);
 }());