blockworks-foundation
/
node-fetch
mirror of https://github.com/blockworks-foundation/node-fetch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

better test for headers, only support string and array

This commit is contained in:
David Frank 2015-04-29 12:15:08 +08:00
parent c02cb4229d
commit 596232fd90
2 changed files with 28 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/headers.js
View File

@ -31,7 +31,7 @@ function Headers(headers) {
if (typeof headers[prop] === 'string') {
this.set(prop, headers[prop]);
} else if (headers[prop].length > 0) {
} else if (headers[prop] instanceof Array) {
headers[prop].forEach(function(item) {
self.append(prop, item);
});

33
test/test.js
View File

@ -620,18 +620,39 @@ describe('node-fetch', function() {
});
});
it('should skip prototype when reading response headers', function() {
it('should ignore unsupported attributes while reading headers', function() {
var FakeHeader = function() {};
FakeHeader.prototype.c = 'string';
FakeHeader.prototype.d = [1,2,3,4];
FakeHeader.prototype.z = 'string';
var res = new FakeHeader;
res.a = 'string';
res.b = [];
res.b = ['1','2'];
res.c = '';
res.d = [];
res.e = { a:1 };
res.f = 1;
res.g = undefined;
res.h = null;
res.i = NaN;
res.j = true;
res.k = false;
var headers = new Headers(res);
expect(headers._headers['a']).to.include('string');
expect(headers._headers['b']).to.be.undefined;
expect(headers._headers['c']).to.be.undefined;
expect(headers._headers['b']).to.include('1');
expect(headers._headers['b']).to.include('2');
expect(headers._headers['c']).to.include('');
expect(headers._headers['d']).to.be.undefined;
expect(headers._headers['e']).to.be.undefined;
expect(headers._headers['f']).to.be.undefined;
expect(headers._headers['g']).to.be.undefined;
expect(headers._headers['h']).to.be.undefined;
expect(headers._headers['i']).to.be.undefined;
expect(headers._headers['j']).to.be.undefined;
expect(headers._headers['k']).to.be.undefined;
expect(headers._headers['z']).to.be.undefined;
});
it('should wrap headers', function() {