Skip to content

Enhance Cipher, Decipher coverage #17449

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Leko wants to merge 11 commits into from
Closed

Enhance Cipher, Decipher coverage #17449

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
3495998
test: Add test call Cipher without new keyword
Leko Dec 4, 2017
24553fa
test: add test call Decipher withour new keyword
Leko Dec 4, 2017
569099c
test: add test call Cipher with cipher is not string
Leko Dec 4, 2017
bb5c70d
test: add test call Cipher with cipher is string and password is not …
Leko Dec 4, 2017
ab7a803
test: add test call Cipher#update with data is not string
Leko Dec 4, 2017
8c60d1d
test: add test call Cipher#setAuthTag with tagbuf is not string
Leko Dec 4, 2017
435bd16
test: add test call Cipher#setAAD with aadbuf is not string
Leko Dec 4, 2017
4ebbe67
test: add test call Decipher with cipher is not string
Leko Dec 4, 2017
f15c054
test: add test call Decipher with cipher is string and password is no…
Leko Dec 4, 2017
b503bd5
test: adjust indent
Leko Dec 4, 2017
9a809ff
test: use factory method instead of constructor
Leko Dec 5, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
test: add test call Cipher with cipher is string and password is not … 
…string
  • Loading branch information
@Leko
Leko committed Dec 4, 2017
commit bb5c70d738d2a40be99d9530b9c3f894d6338bfd
9 changes: 9 additions & 0 deletions test/parallel/test-crypto-cipher-decipher.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,15 @@ testCipher2(Buffer.from('0123456789abcdef'));
type: TypeError,
message: 'The "cipher" argument must be of type string'
});

common.expectsError(
() => new Cipher('aes-256-cbc', null),
{
code: 'ERR_INVALID_ARG_TYPE',
type: TypeError,
message: 'The "password" argument must be one of type string, Buffer, ' +
'TypedArray, or DataView'
});
}

{
Expand Down