From 9c83002274860e341b7d17ad7d76d3fc34ce29a3 Mon Sep 17 00:00:00 2001 From: Sam Roberts Date: Mon, 11 Mar 2019 13:26:44 -0700 Subject: [PATCH] test: use EC cert property now that it exists Remove XXX, there has been an EC specific cert property since https://github.com/nodejs/node/pull/24358 PR-URL: https://github.com/nodejs/node/pull/26598 Reviewed-By: Daniel Bevenius Reviewed-By: Ben Noordhuis Reviewed-By: James M Snell Reviewed-By: Daijiro Wachi --- test/parallel/test-tls-multi-key.js | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/test/parallel/test-tls-multi-key.js b/test/parallel/test-tls-multi-key.js index 8ccc0d32d93757..de213f57e2d65d 100644 --- a/test/parallel/test-tls-multi-key.js +++ b/test/parallel/test-tls-multi-key.js @@ -160,9 +160,7 @@ function test(options) { version: 'TLSv1/SSLv3' }); assert.strictEqual(ecdsa.getPeerCertificate().subject.CN, eccCN); - // XXX(sam) certs don't currently include EC key info, so depend on - // absence of RSA key info to indicate key is EC. - assert(!ecdsa.getPeerCertificate().exponent, 'not cert for an RSA key'); + assert.strictEqual(ecdsa.getPeerCertificate().asn1Curve, 'prime256v1'); ecdsa.end(); connectWithRsa(); }));