From 11cf47c8a6048b1196fa99a58d19db21fa94572a Mon Sep 17 00:00:00 2001 From: Jonathan Ong Date: Tue, 3 Jun 2014 21:35:11 -0700 Subject: [PATCH] remove .request accept tests the fail due to https://github.com/expressjs/accepts/commit/5975ce464575bd0fddfdd3d41796 958cd6bfc20b. passing nothing into accepts() is silly anyways. --- test/request/accepts.js | 7 ------- test/request/acceptsCharsets.js | 9 +-------- test/request/acceptsLanguages.js | 9 +-------- 3 files changed, 2 insertions(+), 23 deletions(-) diff --git a/test/request/accepts.js b/test/request/accepts.js index b72b0f2..34f3a91 100644 --- a/test/request/accepts.js +++ b/test/request/accepts.js @@ -10,13 +10,6 @@ describe('ctx.accepts(types)', function(){ ctx.accepts().should.eql(['text/html', 'text/plain', 'image/jpeg', 'application/*']); }) }) - - describe('when Accept is not populated', function(){ - it('should return []', function(){ - var ctx = context(); - ctx.accepts().should.eql([]); - }) - }) }) describe('with no valid types', function(){ diff --git a/test/request/acceptsCharsets.js b/test/request/acceptsCharsets.js index 2185d4a..be905bd 100644 --- a/test/request/acceptsCharsets.js +++ b/test/request/acceptsCharsets.js @@ -10,13 +10,6 @@ describe('ctx.acceptsCharsets()', function(){ ctx.acceptsCharsets().should.eql(['utf-8', 'utf-7', 'iso-8859-1']); }) }) - - describe('when Accept-Charset is not populated', function(){ - it('should return an empty array', function(){ - var ctx = context(); - ctx.acceptsCharsets().should.eql([]); - }) - }) }) describe('with multiple arguments', function(){ @@ -53,4 +46,4 @@ describe('ctx.acceptsCharsets()', function(){ ctx.acceptsCharsets(['utf-7', 'utf-8']).should.equal('utf-8'); }) }) -}) \ No newline at end of file +}) diff --git a/test/request/acceptsLanguages.js b/test/request/acceptsLanguages.js index d7ebb47..6e8c3b7 100644 --- a/test/request/acceptsLanguages.js +++ b/test/request/acceptsLanguages.js @@ -10,13 +10,6 @@ describe('ctx.acceptsLanguages(langs)', function(){ ctx.acceptsLanguages().should.eql(['es', 'pt', 'en']); }) }) - - describe('when Accept-Language is not populated', function(){ - it('should return an empty array', function(){ - var ctx = context(); - ctx.acceptsLanguages().should.eql([]); - }) - }) }) describe('with multiple arguments', function(){ @@ -53,4 +46,4 @@ describe('ctx.acceptsLanguages(langs)', function(){ ctx.acceptsLanguages(['es', 'en']).should.equal('es'); }) }) -}) \ No newline at end of file +})