Skip to content

Commit

Permalink
test: fix test-tls-session-timeout
Browse files Browse the repository at this point in the history
Tests in pummel seem to break often and stay broken because they don't
get run in CI. In preparation for running pummel tests in CI once a day,
this fixes test-tls-session-timeout. `key` and `cert` are now the
contents of the relevant files and not the paths.

PR-URL: nodejs#25188
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
  • Loading branch information
Trott authored and refack committed Jan 10, 2019
1 parent e24fec7 commit dd5b0a8
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/pummel/test-tls-session-timeout.js
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,8 @@ function doTest() {

const SESSION_TIMEOUT = 1;

const key = fixtures.path('agent.key');
const cert = fixtures.path('agent.crt');
const key = fixtures.readSync('agent.key');
const cert = fixtures.readSync('agent.crt');
const options = {
key: key,
cert: cert,
Expand Down

0 comments on commit dd5b0a8

Please sign in to comment.