Get test-http-response-no-headers.js to pass

felixge · felixge · commit 6e1e9e2fcb8d · 2011-11-22T22:48:50.000+01:00
Main fix was in 3abebf which added HTTP/0.9 support to http parser. Changed test because HTTP 1.1 mandates keep-alive when no headers are given. Fixes nodejs#1711
diff --git a/test/simple/test-http-response-no-headers.js b/test/simple/test-http-response-no-headers.js
@@ -24,7 +24,11 @@ var assert = require('assert');
 var http = require('http');
 var net = require('net');
 
-var expected = 'I AM THE WALRUS';
+var expected = {
+  '0.9': 'I AM THE WALRUS',
+  '1.0': 'I AM THE WALRUS',
+  '1.1': '',
+}
 
 var gotExpected = false;
 
@@ -34,7 +38,7 @@ function test(httpVersion, callback) {
   });
 
   var server = net.createServer(function(conn) {
-    var reply = 'HTTP/' + httpVersion + ' 200 OK\r\n\r\n' + expected;
+    var reply = 'HTTP/' + httpVersion + ' 200 OK\r\n\r\n' + expected[httpVersion];
 
     conn.write(reply, function() {
       conn.destroy();
@@ -55,7 +59,7 @@ function test(httpVersion, callback) {
       });
 
       res.on('end', function() {
-        assert.equal(body, expected);
+        assert.equal(body, expected[httpVersion]);
         gotExpected = true;
         server.close();
         if (callback) process.nextTick(callback);
