test: Add assertion for TLS peer certificate fingerprint

[alandotcom]

PR-URL: #4923
Reviewed-By: Ben Noordhuis [email protected]

[bnoordhuis]

LGTM but can you amend the commit log so it conforms to the guidelines from CONTRIBUTING.md? In particular, it should start with the subsystem (test:).

CI: https://ci.nodejs.org/job/node-test-pull-request/1419/ (currently private because of the upcoming security release.)

[alandotcom]

Thanks @bnoordhuis. Just updated the commit message

[jasnell]

LGTM

[bnoordhuis]

@LumberJ Can you run make lint? The linter is complaining that:

/usr/home/iojs/build/workspace/node-test-linter/test/parallel/test-tls-peer-certificate.js
  41:1  error  Line 41 exceeds the maximum line length of 80  max-len

[alandotcom]

Thanks, just fixed the lint error.

On Fri, Jan 29, 2016 at 12:36 AM, Ben Noordhuis [email protected]
wrote:

@LumberJ https://github.com/lumberj Can you run make lint? The linter
is complaining that:

/usr/home/iojs/build/workspace/node-test-linter/test/parallel/test-tls-peer-certificate.js
41:1 error Line 41 exceeds the maximum line length of 80 max-len

—
Reply to this email directly or view it on GitHub
#4923 (comment).

Alan Cohen
@alan_mit https://twitter.com/intent/user?screen_name=alan_mit

[jasnell]

Landed in 59fb26c