Skip to content

test(hypervisors): add unit tests for HVT#618

Open
r0hansaxena wants to merge 1 commit intourunc-dev:mainfrom
r0hansaxena:test/hvt
Open

test(hypervisors): add unit tests for HVT#618
r0hansaxena wants to merge 1 commit intourunc-dev:mainfrom
r0hansaxena:test/hvt

Conversation

@r0hansaxena
Copy link
Copy Markdown

@r0hansaxena r0hansaxena commented May 4, 2026

Description

Adds unit tests for the HVT VMM backend, covering BuildExecCmd and the interface methods (UsesKVM, SupportsSharedfs, Path, Ok, PreExec). Tests verify memory configuration, networking, block devices, extra monitor args and the binary not found error case.

Related issues

How was this tested?

Ran unit tests locally:

go test ./pkg/unikontainers/hypervisors/ -v -run "TestHVT"

All 13 test cases passed. Full package tests also pass:

go test ./pkg/... ./internal/...

Linter passes locally (make lint, 0 issues).

LLM usage

Claude Sonnet 4.6 was used to assist with writing the tests. All code was reviewed and tested locally before submission.

Checklist

  • I have read the contribution guide.
  • The linter passes locally (make lint).
  • The e2e tests of at least one tool pass locally (make test_ctr, make test_nerdctl, make test_docker, make test_crictl).
  • If LLMs were used: I have read the llm policy.

Signed-off-by: r0hansaxena <rohansxn8772@gmail.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented May 4, 2026

Deploy Preview for urunc canceled.

Name Link
🔨 Latest commit 658e507
🔍 Latest deploy log https://app.netlify.com/projects/urunc/deploys/69f8850bafa0ab00082f284a

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