commit 0aa300b9ba9d66b914793ad18c5b469163e58905
author Bob Beck <bbe@google.com> Fri Apr 05 22:21:56 2024 +0000
committer Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Apr 18 16:34:29 2024 +0000
tree aa63c7b6bbc276adc76ce3c36004106a68085db0
parent f374e1af18c25700923985f6613417847e8f6ab1

clarify a few tests

Change-Id: Ie06b109fe3445129adf5eca00bd813d9957dc606
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/67647
Commit-Queue: Bob Beck <bbe@google.com>
Reviewed-by: David Benjamin <davidben@google.com>
Auto-Submit: Bob Beck <bbe@google.com>

pki/parsed_certificate_unittest.cc

diff --git a/pki/parsed_certificate_unittest.cc b/pki/parsed_certificate_unittest.cc
index 2b4d352..f515003 100644
--- a/pki/parsed_certificate_unittest.cc
+++ b/pki/parsed_certificate_unittest.cc
@@ -170,6 +170,7 @@
   ASSERT_FALSE(cert->signature_algorithm());
 }
 
+// '&' is not acceptable in a printable string.
 TEST(ParsedCertificateTest, IssuerBadPrintableString) {
   ASSERT_FALSE(ParseCertificateFromFile("issuer_bad_printable_string.pem", {}));
 }
@@ -188,10 +189,12 @@
       "subject_blank_subjectaltname_not_critical.pem", {}));
 }
 
+// Non-ASCII character in locality name in subject.
 TEST(ParsedCertificateTest, SubjectNotAscii) {
   ASSERT_FALSE(ParseCertificateFromFile("subject_not_ascii.pem", {}));
 }
 
+// '&' is not acceptable in a printable string.
 TEST(ParsedCertificateTest, SubjectNotPrintableString) {
   ASSERT_FALSE(
       ParseCertificateFromFile("subject_not_printable_string.pem", {}));