Clean up testing and justfile

Signed-off-by: Ludvig Liljenberg <[email protected]>

Author: ludfjig

.github/workflows/dep_rust.yml

@@ -95,10 +95,10 @@ jobs:
           CARGO_TERM_COLOR: always
         run: |
           # with default features
-          just test-rust ${{ matrix.config }} ${{ matrix.hypervisor == 'mshv3' && 'mshv3' || ''}}
+          just test ${{ matrix.config }} ${{ matrix.hypervisor == 'mshv3' && 'mshv3' || ''}}
 
           # with only one driver enabled (driver mshv/kvm feature is ignored on windows) + seccomp + inprocess
-          just test-rust ${{ matrix.config }} inprocess,seccomp,${{ matrix.hypervisor == 'mshv' && 'mshv2' || matrix.hypervisor == 'mshv3' && 'mshv3' || 'kvm' }} 
+          just test ${{ matrix.config }} inprocess,seccomp,${{ matrix.hypervisor == 'mshv' && 'mshv2' || matrix.hypervisor == 'mshv3' && 'mshv3' || 'kvm' }} 
 
           # make sure certain cargo features compile
           cargo check -p hyperlight-host --features crashdump

Justfile

@@ -31,3 +31,4 @@ hyperlight-testing = { workspace = true }
 
 [lib]
 bench = false # see https://bheisler.github.io/criterion.rs/book/faq.html#cargo-bench-gives-unrecognized-option-errors-for-valid-command-line-options
+doctest = false # reduce noise in test output

src/hyperlight_common/Cargo.toml

@@ -155,6 +155,7 @@ mod tests {
 
         let mut gdb = Command::new("rust-gdb")
             .arg("--nw")
+            .arg("--batch")
             .arg("-x")
             .arg(cmd_file_path)
             .spawn()

src/hyperlight_host/examples/guest-debugging/main.rs

@@ -32,7 +32,9 @@ fn fn_writer(_msg: String) -> Result<i32> {
 // by Hyperlight will also be emitted as log messages.
 
 fn main() -> Result<()> {
-    env_logger::init();
+    env_logger::builder()
+        .parse_filters("none,hyperlight=info")
+        .init();
     // Get the path to a simple guest binary.
     let hyperlight_guest_path =
         simple_guest_as_string().expect("Cannot find the guest binary at the expected location.");

src/hyperlight_host/examples/logging/main.rs

@@ -42,7 +42,8 @@ fn main() -> Result<()> {
     // Set up the tracing subscriber.
     // tracing_forest uses the tracing subscriber, which, by default, will consume logs as trace events
     // unless the tracing-log feature is disabled.
-    let layer = ForestLayer::default().with_filter(EnvFilter::from_default_env());
+    let layer = ForestLayer::default()
+        .with_filter(EnvFilter::builder().parse("none,hyperlight=info").unwrap());
     Registry::default().with(layer).init();
     run_example()
 }

src/hyperlight_host/examples/tracing/main.rs

@@ -1201,7 +1201,6 @@ mod tests {
     /// test interdependencies. This test will, however, be run explicitly as a
     /// part of the CI pipeline.
     #[test]
-    #[ignore]
     #[cfg(target_os = "linux")]
     fn test_drop() {
         use proc_maps::maps_contain_addr;

src/hyperlight_host/src/mem/shared_mem.rs

@@ -15,3 +15,6 @@ serde_json = "1.0"
 
 [lib]
 bench = false # see https://bheisler.github.io/criterion.rs/book/faq.html#cargo-bench-gives-unrecognized-option-errors-for-valid-command-line-options
+# reduce noise in test output
+test = false
+doctest = false