fix tests and start for within

Esri · Jan 11, 2014 · 8f22000 · 8f22000
1 parent 24201c3
commit 8f22000
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 12 deletions.
diff --git a/browser/terraformer-geostore.js b/browser/terraformer-geostore.js
@@ -608,8 +608,6 @@ Stream.prototype.unpipe = function (destination) {
           }
         };
 
-        sync.start();
-
         var error = function(){
           completed++;
           errors++;
@@ -638,9 +636,9 @@ Stream.prototype.unpipe = function (destination) {
             callback(null, results);
           }
         }
-      });
 
-    sync.start();
+        sync.start();
+      });
     }));
 
   };

diff --git a/node/terraformer-geostore.js b/node/terraformer-geostore.js
@@ -482,8 +482,6 @@ var Stream = require('stream');
           }
         };
 
-        sync.start();
-
         var error = function(){
           completed++;
           errors++;
@@ -512,9 +510,9 @@ var Stream = require('stream');
             callback(null, results);
           }
         }
-      });
 
-    sync.start();
+        sync.start();
+      });
     }));
 
   };

diff --git a/spec/geostoreSpec.js b/spec/geostoreSpec.js
@@ -515,6 +515,7 @@ describe("geostore", function() {
           }
         },
         function (err, res) {
+          expect(err).toEqual(null);
           expect(res.length).toEqual(1);
           expect(res.id).toEqual(1);
         });

diff --git a/src/geostore.js b/src/geostore.js
@@ -391,8 +391,6 @@
           }
         };
 
-        sync.start();
-
         var error = function(){
           completed++;
           errors++;
@@ -421,9 +419,9 @@
             callback(null, results);
           }
         }
-      });
 
-    sync.start();
+        sync.start();
+      });
     }));
 
   };