Skip to content
This repository has been archived by the owner on Jun 8, 2021. It is now read-only.

Commit

Permalink
test(set all cors headers when request is options): No more check Con…
Browse files Browse the repository at this point in the history
…tent-Length 0 and status 204

CORS Middleware don't send empty response when OPTIONS so this check is not relevant now

#13
  • Loading branch information
tpoisseau committed Oct 18, 2018
1 parent 5f3eec8 commit 1abf5fa
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions test/cors.spec.js
Expand Up @@ -353,10 +353,8 @@ test.group('Cors', () => {
{ key: 'Access-Control-Allow-Credentials', value: true },
{ key: 'Access-Control-Allow-Methods', value: 'GET,PUT,POST' },
{ key: 'Access-Control-Allow-Headers', value: 'Authorization' },
{ key: 'Access-Control-Max-Age', value: 90 },
{ key: 'Content-length', value: 0 }
{ key: 'Access-Control-Max-Age', value: 90 }
])
assert.equal(response._status, 204)
})

test('set vary header when options.origin is a string', async (assert) => {
Expand Down

0 comments on commit 1abf5fa

Please sign in to comment.