Skip to content

Commit

Permalink
tests: remove deep-equal dependency
Browse files Browse the repository at this point in the history
  • Loading branch information
dougwilson committed Mar 15, 2020
1 parent ff3a789 commit 747f5a1
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 14 deletions.
1 change: 0 additions & 1 deletion package.json
Expand Up @@ -17,7 +17,6 @@
"cookie-signature": "1.0.6"
},
"devDependencies": {
"deep-equal": "1.0.1",
"eslint": "6.8.0",
"eslint-config-standard": "14.1.0",
"eslint-plugin-import": "2.20.1",
Expand Down
25 changes: 12 additions & 13 deletions test/cookieParser.js
@@ -1,7 +1,6 @@

var assert = require('assert')
var cookieParser = require('..')
var deepEqual = require('deep-equal')
var http = require('http')
var request = require('supertest')
var signature = require('cookie-signature')
Expand Down Expand Up @@ -155,7 +154,7 @@ describe('cookieParser.JSONCookie(str)', function () {
})

it('should return object for JSON cookie string', function () {
deepEqual(cookieParser.JSONCookie('j:{"foo":"bar"}'), { foo: 'bar' })
assert.deepEqual(cookieParser.JSONCookie('j:{"foo":"bar"}'), { foo: 'bar' })
})

it('should return undefined on invalid JSON', function () {
Expand Down Expand Up @@ -213,18 +212,18 @@ describe('cookieParser.signedCookie(str, secret)', function () {

describe('cookieParser.signedCookies(obj, secret)', function () {
it('should ignore non-signed strings', function () {
deepEqual(cookieParser.signedCookies({}, 'keyboard cat'), {})
deepEqual(cookieParser.signedCookies({ foo: 'bar' }, 'keyboard cat'), {})
assert.deepEqual(cookieParser.signedCookies({}, 'keyboard cat'), {})
assert.deepEqual(cookieParser.signedCookies({ foo: 'bar' }, 'keyboard cat'), {})
})

it('should include tampered strings as false', function () {
deepEqual(cookieParser.signedCookies({ foo: 's:foobaz.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE' }, 'keyboard cat'), {
assert.deepEqual(cookieParser.signedCookies({ foo: 's:foobaz.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE' }, 'keyboard cat'), {
foo: false
})
})

it('should include unsigned strings', function () {
deepEqual(cookieParser.signedCookies({ foo: 's:foobar.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE' }, 'keyboard cat'), {
assert.deepEqual(cookieParser.signedCookies({ foo: 's:foobar.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE' }, 'keyboard cat'), {
foo: 'foobar'
})
})
Expand All @@ -234,17 +233,17 @@ describe('cookieParser.signedCookies(obj, secret)', function () {
foo: 's:foobar.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE'
}

deepEqual(cookieParser.signedCookies(obj, 'keyboard cat'), { foo: 'foobar' })
deepEqual(obj, {})
assert.deepEqual(cookieParser.signedCookies(obj, 'keyboard cat'), { foo: 'foobar' })
assert.deepEqual(obj, {})
})

it('should remove tampered strings from original object', function () {
var obj = {
foo: 's:foobaz.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE'
}

deepEqual(cookieParser.signedCookies(obj, 'keyboard cat'), { foo: false })
deepEqual(obj, {})
assert.deepEqual(cookieParser.signedCookies(obj, 'keyboard cat'), { foo: false })
assert.deepEqual(obj, {})
})

it('should leave unsigned string in original object', function () {
Expand All @@ -253,8 +252,8 @@ describe('cookieParser.signedCookies(obj, secret)', function () {
foo: 's:foobar.N5r0C3M8W+IPpzyAJaIddMWbTGfDSO+bfKlZErJ+MeE'
}

deepEqual(cookieParser.signedCookies(obj, 'keyboard cat'), { foo: 'foobar' })
deepEqual(obj, { fizz: 'buzz' })
assert.deepEqual(cookieParser.signedCookies(obj, 'keyboard cat'), { foo: 'foobar' })
assert.deepEqual(obj, { fizz: 'buzz' })
})

describe('when secret is an array', function () {
Expand All @@ -264,7 +263,7 @@ describe('cookieParser.signedCookies(obj, secret)', function () {
fizz: 's:foobar.JTCAgiMWsnuZpN3mrYnEUjXlGxmDi4POCBnWbRxse88'
}

deepEqual(cookieParser.signedCookies(obj, ['keyboard cat', 'nyan cat']), {
assert.deepEqual(cookieParser.signedCookies(obj, ['keyboard cat', 'nyan cat']), {
buzz: 'foobar',
fizz: 'foobar'
})
Expand Down

0 comments on commit 747f5a1

Please sign in to comment.