commit 195f7491deca8be7fcb785f6b6c4c2d97fad4b8e
author Wendall Cada <wendallc@apache.org> Fri Apr 05 11:38:20 2013 -0700
committer Wendall Cada <wendallc@apache.org> Fri Apr 05 11:38:20 2013 -0700
tree 4a6ab42d8f471a497d962e80cb1cc6cba451fc4e
parent 89a6655e5fbc22fe990d5e9f64c5b9669bd39c2d

Ensure that waitForSuccess calls have return values.

script/test/changes.js

diff --git a/script/test/changes.js b/script/test/changes.js
index d76c193..00944f7 100644
--- a/script/test/changes.js
+++ b/script/test/changes.js
@@ -92,6 +92,7 @@
       if (change2.seq != 2) {
           throw "bad seq, try again";
       }
+      return true;
     }, "bar-only");
 
     T(change1.seq == 1);
@@ -112,6 +113,7 @@
       if (change3.seq != 3) {
         throw "bad seq, try again";
       }
+      return true;
     });
 
     T(change3.seq == 3);
@@ -131,6 +133,7 @@
       if (str.charAt(str.length - 1) != "\n" || str.charAt(str.length - 2) != "\n") {
         throw("keep waiting");
       }
+      return true;
     }, "heartbeat");
 
     T(str.charAt(str.length - 1) == "\n");
@@ -152,8 +155,10 @@
       source.addEventListener('message', sourceListener , false);
 
       waitForSuccess(function() {
-        if (results.length != 3)
+        if (results.length != 3) {
           throw "bad seq, try again";
+        }
+        return true;
       });
 
       source.removeEventListener('message', sourceListener, false);
@@ -177,6 +182,7 @@
       if (lines[5] != '"last_seq":3}') {
         throw("still waiting");
       }
+      return true;
     }, "last_seq");
 
     xhr = CouchDB.newXhr();
@@ -201,6 +207,7 @@
       if (lines[3] != '"last_seq":4}') {
         throw("still waiting");
       }
+      return true;
     }, "change_lines");
 
     var change = parse_changes_line(lines[1]);
@@ -233,6 +240,7 @@
       if (lines[3] != '"last_seq":5}') {
         throw("still waiting");
       }
+      return true;
     }, "change_lines");
 
     var change = parse_changes_line(lines[1]);
@@ -313,6 +321,7 @@
 
     waitForSuccess(function() {
       resp = JSON.parse(xhr.responseText);
+      return true;
     }, "longpoll-since");
 
     T(resp.last_seq == 10);
@@ -337,6 +346,7 @@
         if (xhr.readyState != 4) {
           throw("still waiting");
         }
+        return true;
       }, "continuous-rusty");
       lines = xhr.responseText.split("\n");
       var good = false;
@@ -529,6 +539,7 @@
             if (xhr.readyState != 4) {
               throw("still waiting");
             }
+            return true;
         }, "andmore-only");
 
         var line = JSON.parse(xhr.responseText.split("\n")[0]);

script/test/oauth.js

diff --git a/script/test/oauth.js b/script/test/oauth.js
index 89d3779..9199cd0 100644
--- a/script/test/oauth.js
+++ b/script/test/oauth.js
@@ -108,6 +108,7 @@
         if (xhr.responseText.indexOf("\"-pbkdf2-") != 0) {
             throw("still waiting");
         }
+        return true;
       }, "wait-for-admin");
 
       CouchDB.newUuids(2); // so we have one to make the salt