fixed sync test with poll and specs

Author: marcelduran

lib/specs.js

@@ -100,6 +100,12 @@ function specsRunner(specs, reporter, callback, err, data) {
       tests = [],
       path = [];
 
+  // bail if test not complete
+  if (!err && (!data || data && data.response &&
+      data.response.statusCode !== 200)) {
+    return callback(err, data);
+  }
+
   function traverse(specs, data) {
     Object.keys(specs).forEach(function(key) {
       // bail on default

lib/webpagetest.js

@@ -280,8 +280,11 @@ function runTest(what, options, callback) {
   }
 
   function poll(err, data) {
+    // poll again when test started but not complete
+    // and not when specs are done testing
     if (!err && (!data || data && data.response &&
-        data.response.statusCode !== 200)) {
+        data.response.statusCode !== 200) &&
+        !(typeof err === 'number' && data === undefined)) {
       polling = setTimeout(getTestResults.bind(this, testId,
         resultsOptions, poll.bind(this)), options.pollResults);
     } else {