test: replace fixturesDir with usage of fixtures module

Repalce common.fixturesDir with usage of the fixtures module in test-tls-ecdh-disable. Add a comment explaining the test. PR-URL: #16810 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Rich Trott <[email protected]>
nodejs · Nov 6, 2017 · e4b3c00 · e4b3c00
1 parent 021ccb4
commit e4b3c00
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/test/parallel/test-tls-ecdh-disable.js b/test/parallel/test-tls-ecdh-disable.js
@@ -19,8 +19,12 @@
 // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE
 // USE OR OTHER DEALINGS IN THE SOFTWARE.
 
+// Test that the usage of eliptic curves are not permitted if disabled during
+// server initialization.
+
 'use strict';
 const common = require('../common');
+const { readKey } = require('../common/fixtures');
 if (!common.hasCrypto)
   common.skip('missing crypto');
 
@@ -30,11 +34,10 @@ if (!common.opensslCli)
 const assert = require('assert');
 const tls = require('tls');
 const exec = require('child_process').exec;
-const fs = require('fs');
 
 const options = {
-  key: fs.readFileSync(`${common.fixturesDir}/keys/agent2-key.pem`),
-  cert: fs.readFileSync(`${common.fixturesDir}/keys/agent2-cert.pem`),
+  key: readKey('agent2-key.pem'),
+  cert: readKey('agent2-cert.pem'),
   ciphers: 'ECDHE-RSA-AES128-SHA',
   ecdhCurve: false
 };