Add IsvProductIdVerifier

[nick-mobilecoin]

Motivation

Future Work

[nick-mobilecoin]

Current dependencies on/for this PR:

• main
  • PR Add Not verifier for negating verifier results #33
    • PR Update to mc-sgx-*** 0.5.0 #34
      • PR Update ReportDataVerifier to use a mask #35
        • PR Add CpuSvnVerifier #38
          • PR Add ExtendedProductIdVerifier #39
            • PR Add IsvProductIdVerifier #40 👈
              • PR Add FamilyIdVerifier #41
                • PR Update MrSignerVerifier to verify necessary sub components #42
                  • PR Change use of CtOption to VerificationResult #43
                  • PR Add SpacedStructName trait #47
                  • PR Convert VerificationError into VerificationResultMetadata #44
                  • PR Add a displyable result for MrSignerVerifier #45

This comment was auto-generated by Graphite.

[awygle]

LGTM.

[codecov]

Codecov Report

Merging #40 (c3c06c5) into nick/extended-product-id (5c5209d) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@                     Coverage Diff                      @@
##           nick/extended-product-id      #40      +/-   ##
============================================================
+ Coverage                     95.54%   95.65%   +0.10%     
============================================================
  Files                             2        2              
  Lines                           764      783      +19     
============================================================
+ Hits                            730      749      +19     
  Misses                           34       34              

Impacted Files	Coverage Δ
verifier/src/lib.rs	90.51% <ø> (ø)
verifier/src/report_body.rs	99.34% <100.00%> (+0.02%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[nick-mobilecoin]

@nick-mobilecoin queued this pull request to merge with Graphite.

[nick-mobilecoin]

Graphite couldn't merge this PR because it was not satisfying all requirements.

[nick-mobilecoin]

@awygle bumping for restack

[nick-mobilecoin]

@nick-mobilecoin queued this pull request to merge with Graphite.

[nick-mobilecoin]

@nick-mobilecoin merged this pull request with Graphite.