Comment out Keep-Alive and remove destroy() calls in tests

Will not work in Node v0.2.4 or v0.2.5
gsf · Jan 24, 2011 · e8098d3 · e8098d3
1 parent 45bd53f
commit e8098d3
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 5 deletions.
diff --git a/lib/solr.js b/lib/solr.js
@@ -125,10 +125,10 @@ Client.prototype.update = function (data, callback) {
     method: "POST",
     path: updatePath,
     headers: {
-      // Keep-Alive needed for v0.2.5, but sometimes leaves 
-      // connection hanging that must be manually destroyed
+      // Keep-Alive needed for Node v0.2.4 and v0.2.5, but sometimes 
+      // leaves connection hanging that must be manually destroyed
       // http://groups.google.com/group/nodejs-dev/browse_thread/thread/e9044c0778ac67d0
-      "Connection": "Keep-Alive", 
+      //"Connection": "Keep-Alive", 
       "Content-Length": Buffer.byteLength(data), 
       "Host": this.fullHost
     },

diff --git a/test/test-update-array.js b/test/test-update-array.js
@@ -38,7 +38,6 @@ client.del(null, '*:*', function(err) {  // Clean up index
           client.query('wakak_i:5', function(err, res) {
             if (err) throw err;
             assert.equal(JSON.parse(res).response.numFound, 3, 'Wrong number of docs in index.');
-            client.destroy();
           });
         });
       });

diff --git a/test/test-update-xml.js b/test/test-update-xml.js
@@ -19,7 +19,6 @@ client.del(null, '*:*', function(err) {  // Clean up index
       assert.equal(solr.getStatus(res), 0, 'Add document with value with XML characters failed.');
       client.commit(function(err, res) {
         if (err) throw err;
-        client.destroy();
       });
     });
   });