Merge pull request #62 from DolphFlynn/test

Update EC key round trip test.
DolphFlynn · Aug 17, 2024 · 9ca22c8 · 9ca22c8
2 parents 0019246 + 032105b
commit 9ca22c8
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/test/java/com/blackberry/jwteditor/KeyToPemRoundTripTests.java b/src/test/java/com/blackberry/jwteditor/KeyToPemRoundTripTests.java
@@ -78,8 +78,12 @@ void publicRsaJWKtoPEM(String jwkString) throws PemException, ParseException {
     @MethodSource("com.blackberry.jwteditor.PEMToJWKTests#ecPrivateValidPEMs")
     void ecKeyPEMtoJWK(String pem) throws PemException {
         JWK ecKey = PEMUtils.pemToECKey(pem);
+
+        // Keys have been converted to PKCS#8 so PEM serialization is different
         String newPem = PEMUtils.jwkToPem(ecKey);
-        //assertEquals(pem, newPem);
+        JWK ecKey2 = PEMUtils.pemToECKey(newPem);
+
+        assertThat(ecKey2).isEqualTo(ecKey);
     }
 
     @ParameterizedTest