Skip to content

Commit

Permalink
Correct test descriptions (#829)
Browse files Browse the repository at this point in the history
Mark lint.NA expected results as lint.NA, not pass.

#829
  • Loading branch information
mtgag authored Apr 9, 2024
1 parent 308a138 commit ae3b1f3
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions v3/lints/cabf_smime_br/mailbox_address_from_san_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -42,13 +42,13 @@ func TestMailboxAddressFromSANLint(t *testing.T) {
ExpectedResult: lint.Pass,
},
{
Name: "pass - only contains one san:emailAddress value",
Name: "NA - only contains one san:emailAddress value",
InputFilename: "WithOnlySANEmail.pem",

ExpectedResult: lint.NA,
},
{
Name: "pass - only contains one san:otherName value",
Name: "NA - only contains one san:otherName value",
InputFilename: "WithOnlySANOtherName.pem",

ExpectedResult: lint.NA,
Expand Down Expand Up @@ -94,7 +94,7 @@ func TestMailboxAddressFromSANLint(t *testing.T) {
ExpectedDetails: "all certificate mailbox addresses must be present in san:emailAddresses or san:otherNames in addition to any other field they may appear",
},
{
Name: "pass - subject:commonName is personal name, san:emailAddress contains an email",
Name: "NA - subject:commonName is personal name, san:emailAddress contains an email",
InputFilename: "sponsorValidatedMultipurposePersonalNameInCN.pem",

ExpectedResult: lint.NA,
Expand Down

0 comments on commit ae3b1f3

Please sign in to comment.