fixup! crypto: add optional callback to crypto.sign and crypto.verify

nodejs · Mar 8, 2021 · df713fa · df713fa
1 parent ccb7ab9
commit df713fa
Showing 1 changed file with 12 additions and 0 deletions.
diff --git a/test/parallel/test-crypto-async-sign-verify.js b/test/parallel/test-crypto-async-sign-verify.js
@@ -94,5 +94,17 @@ test('ec_secp256k1_public.pem', 'ec_secp256k1_private.pem', 'sha384',
      false, { dsaEncoding: 'der' });
 
 // ECDSA w/ ieee-p1363 signature encoding
+// TODO(@jasnell): this fails because the signature generated is actually a DER one
 test('ec_secp256k1_public.pem', 'ec_secp256k1_private.pem', 'sha384', false,
      { dsaEncoding: 'ieee-p1363' });
+
+// DSA w/ der signature encoding
+test('dsa_public_1025.pem', 'dsa_private_1025.pem', 'sha256',
+     false);
+test('dsa_public_1025.pem', 'dsa_private_1025.pem', 'sha256',
+     false, { dsaEncoding: 'der' });
+
+// DSA w/ ieee-p1363 signature encoding
+// TODO(@jasnell): this crashes
+test('dsa_public_1025.pem', 'dsa_private_1025.pem', 'sha256', false,
+     { dsaEncoding: 'ieee-p1363' });