Skip to content

feat: add integration tests with TPM enabled#27

Open
HJK-X wants to merge 20 commits intocanonical:mainfrom
HJK-X:tpm-integration-tests
Open

feat: add integration tests with TPM enabled#27
HJK-X wants to merge 20 commits intocanonical:mainfrom
HJK-X:tpm-integration-tests

Conversation

@HJK-X
Copy link
Contributor

@HJK-X HJK-X commented Jan 26, 2026

Runs integration tests inside a VM with TPM+FDE enabled.

See this action:
https://github.com/HJK-X/snap-http/actions/runs/21369743041/job/61510905420

@HJK-X HJK-X requested a review from Copilot January 26, 2026 19:43
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds integration tests that run inside a VM with TPM (Trusted Platform Module) and FDE (Full Disk Encryption) enabled, validating the snap-http functionality in a secure boot environment.

Changes:

  • Created new integration test suite for TPM-enabled environment testing recovery key management operations
  • Added GitHub Actions workflow to automate VM-based integration testing with TPM emulation
  • Included shell script to mount the host directory and execute tests inside the VM

Reviewed changes

Copilot reviewed 3 out of 8 changed files in this pull request and generated 3 comments.

File Description
tests/integration_tpm/test_api.py Implements integration tests for keyslot management and recovery key operations in TPM environment
tests/integration_tpm/assets/run_tests.sh Provides script to mount host directory and run integration tests inside the VM
.github/workflows/integration-tpm.yml Configures GitHub Actions workflow to spin up QEMU VM with TPM emulation and execute integration tests

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@HJK-X HJK-X marked this pull request as ready for review January 26, 2026 22:12
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.

1 participant