Add a test case for invalid headers

See feae6d6.

Author: TimothyGu

test/server.js

@@ -117,6 +117,20 @@ export default class TestServer {
 			res.end('fake gzip string');
 		}
 
+		if (p === '/invalid-header') {
+			res.setHeader('Content-Type', 'text/plain');
+			res.writeHead(200);
+			// HACK: add a few invalid headers to the generated header string before
+			// it is actually sent to the socket.
+			res._header = res._header.replace(/\r\n$/, [
+				'Invalid-Header : abc\r\n',
+				'Invalid-Header-Value: \x07k\r\n',
+				'Set-Cookie: \x07k\r\n',
+				'Set-Cookie: \x07kk\r\n',
+			].join('') + '\r\n');
+			res.end('hello world\n');
+		}
+
 		if (p === '/timeout') {
 			setTimeout(function() {
 				res.statusCode = 200;

test/test.js

@@ -477,6 +477,20 @@ describe('node-fetch', () => {
 		});
 	});
 
+	it('should ignore invalid headers', function() {
+		const url = `${base}invalid-header`;
+		return fetch(url).then(res => {
+			expect(res.headers.get('Invalid-Header')).to.be.null;
+			expect(res.headers.get('Invalid-Header-Value')).to.be.null;
+			expect(res.headers.get('Set-Cookie')).to.be.null;
+			expect(Array.from(res.headers.keys()).length).to.equal(4);
+			expect(res.headers.has('Connection')).to.be.true;
+			expect(res.headers.has('Content-Type')).to.be.true;
+			expect(res.headers.has('Date')).to.be.true;
+			expect(res.headers.has('Transfer-Encoding')).to.be.true;
+		});
+	});
+
 	it('should handle client-error response', function() {
 		const url = `${base}error/400`;
 		return fetch(url).then(res => {