Permalink
Browse files

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

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 #1711
  • Loading branch information...
1 parent 246805d commit 6e1e9e2fcb8dd79eada404581c2cc3eb2cd245f8 @felixge felixge committed Nov 22, 2011
Showing with 7 additions and 3 deletions.
  1. +7 −3 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);

0 comments on commit 6e1e9e2

Please sign in to comment.