test: move dherror.pem to fixtures/keys/

PR-URL: #27962 Reviewed-By: Sam Roberts <[email protected]> Reviewed-By: Ujjwal Sharma <[email protected]> Reviewed-By: Rich Trott <[email protected]>
nodejs · Jun 17, 2019 · f41dfd7 · f41dfd7
1 parent 98f7ae9
commit f41dfd7
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 5 deletions.
diff --git a/test/fixtures/keys/Makefile b/test/fixtures/keys/Makefile
@@ -24,6 +24,7 @@ all: \
   dh512.pem \
   dh1024.pem \
   dh2048.pem \
+	dherror.pem \
 	dsa_params.pem \
 	dsa_private.pem \
 	dsa_private_encrypted.pem \
@@ -578,6 +579,9 @@ dh1024.pem:
 dh2048.pem:
 	openssl dhparam -out dh2048.pem 2048
 
+dherror.pem: dh512.pem
+	sed 's/^[^-].*/AAAAAAAAAA/g' dh512.pem > dherror.pem
+
 dsa_params.pem:
 	openssl dsaparam -out dsa_params.pem 2048
 

diff --git a/test/fixtures/dherror.pem → test/fixtures/keys/dherror.pem b/test/fixtures/dherror.pem → test/fixtures/keys/dherror.pem
@@ -1,3 +1,4 @@
 -----BEGIN DH PARAMETERS-----
-AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
+AAAAAAAAAA
+AAAAAAAAAA
 -----END DH PARAMETERS-----
diff --git a/test/parallel/test-tls-dhe.js b/test/parallel/test-tls-dhe.js
@@ -44,10 +44,8 @@ common.expectWarning('SecurityWarning',
                      'DH parameter is less than 2048 bits');
 
 function loadDHParam(n) {
-  const params = [`dh${n}.pem`];
-  if (n !== 'error')
-    params.unshift('keys');
-  return fixtures.readSync(params);
+  const keyname = `dh${n}.pem`;
+  return fixtures.readKey(keyname);
 }
 
 function test(keylen, expectedCipher, cb) {