Paolo Bonzini <pbonz...@redhat.com> writes:
> Adjust the integration test to compile with a subset of QEMU object
> files, and make it actually create an object of the class it defines.
>
> Follow the Rust filesystem conventions, where tests go in tests/ if
> they use the library in the same way any other code would.
>
> Signed-off-by: Paolo Bonzini <pbonz...@redhat.com>
> ---
> meson.build | 10 ++++-
> rust/qemu-api/meson.build | 20 +++++++--
> rust/qemu-api/src/tests.rs | 49 ----------------------
> rust/qemu-api/tests/tests.rs | 78 ++++++++++++++++++++++++++++++++++++
> 4 files changed, 104 insertions(+), 53 deletions(-)
> delete mode 100644 rust/qemu-api/src/tests.rs
> create mode 100644 rust/qemu-api/tests/tests.rs
<snip>
> diff --git a/rust/qemu-api/meson.build b/rust/qemu-api/meson.build
> index 42ea815fa5a..d24e0c0725e 100644
> --- a/rust/qemu-api/meson.build
> +++ b/rust/qemu-api/meson.build
> @@ -14,11 +14,25 @@ _qemu_api_rs = static_library(
> '--cfg', 'MESON',
> # '--cfg', 'feature="allocator"',
> ],
> - dependencies: [
> - qemu_api_macros,
> - ],
> )
>
> qemu_api = declare_dependency(
> link_with: _qemu_api_rs,
> + dependencies: qemu_api_macros,
> )
> +
> +# Rust executable do not support objects, so add an intermediate step.
> +rust_qemu_api_objs = static_library(
> + 'rust_qemu_api_objs',
> + objects: [libqom.extract_all_objects(recursive: false),
> + libhwcore.extract_all_objects(recursive: false)])
> +
> +rust.test('rust-qemu-api-integration',
> + static_library(
> + 'rust_qemu_api_integration',
> + 'tests/tests.rs',
> + override_options: ['rust_std=2021', 'build.rust_std=2021'],
> + link_whole: [rust_qemu_api_objs, libqemuutil]),
> +
> + dependencies: [qemu_api, qemu_api_macros],
> + suite: ['unit', 'rust'])
I met the following error when trying to build the test:
[4/8] Compiling Rust source ../rust/qemu-api/tests/tests.rs
FAILED: rust/qemu-api/librust_qemu_api_integration.rlib
rustc -C linker=cc -C link-arg=-m64 --color=auto -C debug-assertions=yes -C
overflow-checks=no --crate-type rlib -D warnings --edition=2021 -C opt-level=2
-g --cfg CONFIG_ACCEPT4 --cfg CONFIG_AF_VSOCK --cfg CONFIG_ASAN_IFACE_FIBER
--cfg CONFIG_ATOMIC64 --cfg CONFIG_ATTR --cfg CONFIG_AUDIO_OSS --cfg
CONFIG_AUDIO_PA --cfg CONFIG_AUDIO_SDL --cfg CONFIG_AVX2_OPT --cfg
CONFIG_AVX512BW_OPT --cfg CONFIG_BDRV_RO_WHITELIST --cfg
CONFIG_BDRV_RW_WHITELIST --cfg CONFIG_BLKZONED --cfg CONFIG_CLOCK_ADJTIME --cfg
CONFIG_CLOSE_RANGE --cfg CONFIG_CMPXCHG128 --cfg CONFIG_COROUTINE_POOL --cfg
CONFIG_CPUID_H --cfg CONFIG_CURSES --cfg CONFIG_DBUS_DISPLAY --cfg CONFIG_DUP3
--cfg CONFIG_EPOLL --cfg CONFIG_EPOLL_CREATE1 --cfg CONFIG_EVENTFD --cfg
CONFIG_FALLOCATE --cfg CONFIG_FALLOCATE_PUNCH_HOLE --cfg
CONFIG_FALLOCATE_ZERO_RANGE --cfg CONFIG_FDATASYNC --cfg CONFIG_FDT --cfg
CONFIG_FIEMAP --cfg CONFIG_FSFREEZE --cfg CONFIG_FSTRIM --cfg CONFIG_GBM --cfg
CONFIG_GETAUXVAL --cfg CONFIG_GETCPU --cfg CONFIG_GETRANDOM --cfg CONFIG_GETTID
--cfg CONFIG_GIO --cfg CONFIG_GNUTLS --cfg CONFIG_GNUTLS_CRYPTO --cfg
CONFIG_HAVE_RUST --cfg CONFIG_HEXAGON_IDEF_PARSER --cfg CONFIG_INOTIFY --cfg
CONFIG_INOTIFY1 --cfg CONFIG_INT128 --cfg CONFIG_INT128_TYPE --cfg CONFIG_IOVEC
--cfg CONFIG_L2TPV3 --cfg CONFIG_LIBDW --cfg CONFIG_LIBUDEV --cfg CONFIG_LINUX
--cfg CONFIG_LINUX_MAGIC_H --cfg CONFIG_MADVISE --cfg CONFIG_MALLOC_TRIM --cfg
CONFIG_MEMALIGN --cfg CONFIG_MEMFD --cfg CONFIG_OPENGL --cfg
CONFIG_OPEN_BY_HANDLE --cfg CONFIG_PIXMAN --cfg CONFIG_PLUGIN --cfg CONFIG_PNG
--cfg CONFIG_POSIX --cfg CONFIG_POSIX_FALLOCATE --cfg CONFIG_POSIX_MADVISE
--cfg CONFIG_POSIX_MEMALIGN --cfg CONFIG_PPOLL --cfg
CONFIG_PRCTL_PR_SET_TIMERSLACK --cfg CONFIG_PREADV --cfg
CONFIG_PTHREAD_AFFINITY_NP --cfg CONFIG_PTHREAD_SETNAME_NP_W_TID --cfg
CONFIG_QOM_CAST_DEBUG --cfg CONFIG_RELOCATABLE --cfg CONFIG_REPLICATION --cfg
CONFIG_RTNETLINK --cfg CONFIG_SDL --cfg CONFIG_SECRET_KEYRING --cfg
CONFIG_SELINUX --cfg CONFIG_SENDFILE --cfg CONFIG_SETNS --cfg CONFIG_SIGNALFD
--cfg CONFIG_SLIRP --cfg CONFIG_SPLICE --cfg CONFIG_STATX --cfg
CONFIG_STATX_MNT_ID --cfg CONFIG_SYNCFS --cfg CONFIG_SYNC_FILE_RANGE --cfg
CONFIG_SYSMACROS --cfg CONFIG_TASN1 --cfg CONFIG_TCG --cfg CONFIG_TIMERFD --cfg
CONFIG_TPM --cfg CONFIG_TRACE_LOG --cfg CONFIG_VALGRIND_H --cfg CONFIG_VALLOC
--cfg CONFIG_VDUSE_BLK_EXPORT --cfg CONFIG_VHOST --cfg CONFIG_VHOST_CRYPTO
--cfg CONFIG_VHOST_KERNEL --cfg CONFIG_VHOST_NET --cfg CONFIG_VHOST_NET_USER
--cfg CONFIG_VHOST_NET_VDPA --cfg CONFIG_VHOST_USER --cfg
CONFIG_VHOST_USER_BLK_SERVER --cfg CONFIG_VHOST_VDPA --cfg CONFIG_VIRTFS --cfg
CONFIG_VNC --cfg CONFIG_VNC_JPEG --cfg CONFIG_XKBCOMMON --cfg
HAVE_BLK_ZONE_REP_CAPACITY --cfg HAVE_BTRFS_H --cfg HAVE_COPY_FILE_RANGE --cfg
HAVE_DRM_H --cfg HAVE_FSXATTR --cfg HAVE_GETIFADDRS --cfg
HAVE_GLIB_WITH_ALIGNED_ALLOC --cfg HAVE_GLIB_WITH_SLICE_ALLOCATOR --cfg
HAVE_HOST_BLOCK_DEVICE --cfg HAVE_IPPROTO_MPTCP --cfg HAVE_MLOCKALL --cfg
HAVE_OPENAT2_H --cfg HAVE_OPENPTY --cfg HAVE_PTY_H --cfg HAVE_STRCHRNUL --cfg
HAVE_STRUCT_STAT_ST_ATIM --cfg HAVE_SYSTEM_FUNCTION --cfg HAVE_UTMPX -D
unsafe_op_in_unsafe_fn -C default-linker-libraries --crate-name
rust_qemu_api_integration --emit
dep-info=rust/qemu-api/rust_qemu_api_integration.d --emit
link=rust/qemu-api/librust_qemu_api_integration.rlib --out-dir
rust/qemu-api/librust_qemu_api_integration.rlib.p -C
metadata=81e2432@@rust_qemu_api_integration@sta
-lstatic:+verbatim=librust_qemu_api_objs.a -lstatic:+verbatim=libqemuutil.a
-lstatic:-bundle,+verbatim=libvhost-user-glib.a
-lstatic:-bundle,+verbatim=libvhost-user.a -ldylib:+verbatim=libgio-2.0.so
-ldylib:+verbatim=libgobject-2.0.so -ldylib:+verbatim=libglib-2.0.so
-ldylib:+verbatim=libgmodule-2.0.so -Clink-arg=-pthread
-ldylib:+verbatim=libgnutls.so -Clink-arg=-pthread -Clink-arg=-pthread
-Clink-arg=-pthread -Lrust/qemu-api -L. -Lsubprojects/libvhost-user
-L/usr/lib/x86_64-linux-gnu ../rust/qemu-api/tests/tests.rs
error[E0433]: failed to resolve: use of undeclared crate or module
`qemu_api`
--> ../rust/qemu-api/tests/tests.rs:7:5
|
7 | use qemu_api::{
| ^^^^^^^^ use of undeclared crate or module `qemu_api`
error[E0432]: unresolved import `qemu_api`
--> ../rust/qemu-api/tests/tests.rs:7:5
|
7 | use qemu_api::{
| ^^^^^^^^ use of undeclared crate or module `qemu_api`
error: unused import: `core::ffi::CStr`
--> ../rust/qemu-api/tests/tests.rs:5:5
|
5 | use core::ffi::CStr;
| ^^^^^^^^^^^^^^^
|
= note: `-D unused-imports` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(unused_imports)]`
error: aborting due to 3 previous errors
Some errors have detailed explanations: E0432, E0433.
For more information about an error, try `rustc --explain E0432`.
This is probably because rust.test() is designed to build unit tests
(i.e., tests within the same crate under test), but tests/*.rs are
integration tests which are standalone crates to be linked with the
crate under test.
Rust.test() builds unit tests by "copying the sources and arguments
passed to the original target and adding the --test argument to the
compilation" [1], but that doesn't seem to apply to integration tests.
[1] https://mesonbuild.com/Rust-module.html
I can build and run the test with the following instead:
test('rust-qemu-api-integration',
executable(
'rust_qemu_api_integration',
'tests/tests.rs',
override_options: ['rust_std=2021', 'build.rust_std=2021'],
rust_args: [
'--test',
],
install: false,
dependencies: [qemu_api, qemu_api_macros],
link_whole: [rust_qemu_api_objs, libqemuutil]),
args: [
'--test',
'--format', 'pretty',
],
protocol: 'rust',
suite: ['unit', 'rust'])
--
Best Regards
Junjie Mao
