Skip to content

Test endorsed TCB in verifySnpAttestation#7704

Merged
achamayou merged 3 commits intotest_endorsed_tcbfrom
copilot/sub-pr-7668
Mar 5, 2026
Merged

Test endorsed TCB in verifySnpAttestation#7704
achamayou merged 3 commits intotest_endorsed_tcbfrom
copilot/sub-pr-7668

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 5, 2026

Add e2e test coverage for the endorsed_tcb parameter in the verifySnpAttestation JS API to prevent regressions from the endianness fix.

  • Dynamic TCB derivation: First call to verifySnpAttestation without endorsed_tcb retrieves the attestation report; reported_tcb from that response is used as endorsed_tcb in subsequent calls, making the test work across Milan/Genoa/Turin
  • Positive test: Second call with the derived endorsed_tcb asserts success
  • Negative test: Third call with a zero-valued endorsed_tcb of correct size asserts 400 rejection
  • CI label: programmability_and_jwt test marked with snp label so it runs in SNP CI jobs

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits March 5, 2026 09:47
Co-authored-by: achamayou <4016369+achamayou@users.noreply.github.com>
Co-authored-by: achamayou <4016369+achamayou@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix issue with endorsed TCB testing Test endorsed TCB in verifySnpAttestation Mar 5, 2026
@achamayou achamayou marked this pull request as ready for review March 5, 2026 10:23
@achamayou achamayou requested a review from a team as a code owner March 5, 2026 10:23
@achamayou achamayou merged commit ccf9ff5 into test_endorsed_tcb Mar 5, 2026
13 of 14 checks passed
@achamayou achamayou deleted the copilot/sub-pr-7668 branch March 5, 2026 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants