rust-vmm / vm-virtio

virtio implementation
Apache License 2.0
364 stars 87 forks source link

Enable features required for tests #218

Closed tedinski closed 1 year ago

tedinski commented 1 year ago

Summary of the PR

cargo test works, but due to how cargo enables features for crates built in a workspace, this is by accident, and the following commands fail without this patch:

These changes are modeled after the dev-dependencies that already exist to enable these features in e.g. virtio-vsock:

https://github.com/rust-vmm/vm-virtio/blob/c527b45dada0a81d343aca7f06759d5637d6429a/crates/devices/virtio-vsock/Cargo.toml#L18-L20

Requirements

Before submitting your PR, please make sure you addressed the following requirements:

andreeaflorescu commented 1 year ago

We are experiencing some problems with the CI. This PR doesn't change anything that could impact fuzzing, so we can merge the PR without the test passing.