Change name and add comment

arthurscchan · arthurscchan · commit 249e551f735e · 2023-04-05T18:17:02.000Z
Signed-off-by: Arthur Chan &lt;arthur.chan@adalogics.com&gt;
diff --git a/sigstore-java/src/test/java/dev/sigstore/encryption/KeysTest.java b/sigstore-java/src/test/java/dev/sigstore/encryption/KeysTest.java
@@ -186,8 +186,11 @@ void parseTufPublicKey_rsa()
   }
 
   @Test
-  void parsePublicKeyBad()
+  void parsePublicKey_failOnNullSection()
       throws IOException, NoSuchAlgorithmException, NoSuchProviderException {
+    // This unit test is used to test the fix for a bug discovered by oss-fuzz
+    // The bug happens when a malformed byte array is passed to the method
+    // https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=57247
     byte[] byteArray = "-----BEGIN A-----\nBBBBB-----END A".getBytes(StandardCharsets.UTF_8);
     Assertions.assertThrows(InvalidKeySpecException.class, () -> Keys.parsePublicKey(byteArray));
   }

Original file line number	Diff line number	Diff line change
`@@ -186,8 +186,11 @@ void parseTufPublicKey_rsa()`
`186`	`186`	`}`
`187`	`187`
`188`	`188`	`@Test`
`189`		`- void parsePublicKeyBad()`
	`189`	`+ void parsePublicKey_failOnNullSection()`
`190`	`190`	`throws IOException, NoSuchAlgorithmException, NoSuchProviderException {`
	`191`	`+ // This unit test is used to test the fix for a bug discovered by oss-fuzz`
	`192`	`+ // The bug happens when a malformed byte array is passed to the method`
	`193`	`+ // https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=57247`
`191`	`194`	`byte[] byteArray = "-----BEGIN A-----\nBBBBB-----END A".getBytes(StandardCharsets.UTF_8);`
`192`	`195`	`Assertions.assertThrows(InvalidKeySpecException.class, () -> Keys.parsePublicKey(byteArray));`
`193`	`196`	`}`