Update test runner to take an Option<&Path> instead of Option<&str> for the ramdisk path

Co-authored-by: Philipp Oppermann <dev@phil-opp.com>

Author: jasoncouture

tests/ramdisk.rs

@@ -1,18 +1,20 @@
+use std::path::Path;
+
 use bootloader_test_runner::run_test_kernel;
 static RAMDISK_PATH: &str = "tests/ramdisk.txt";
 
 #[test]
 fn basic_boot() {
     run_test_kernel(
         env!("CARGO_BIN_FILE_TEST_KERNEL_RAMDISK_basic_boot"),
-        Some(RAMDISK_PATH),
+        Some(&Path::new(RAMDISK_PATH)),
     );
 }
 
 #[test]
 fn check_ramdisk() {
     run_test_kernel(
         env!("CARGO_BIN_FILE_TEST_KERNEL_RAMDISK_ramdisk"),
-        Some(RAMDISK_PATH),
+        Some(&Path::new(RAMDISK_PATH)),
     );
 }

tests/runner/src/lib.rs

@@ -10,12 +10,8 @@ const QEMU_ARGS: &[&str] = &[
     "--no-reboot",
 ];
 
-pub fn run_test_kernel(kernel_binary_path: &str, ramdisk_path: Option<&str>) {
+pub fn run_test_kernel(kernel_binary_path: &str, ramdisk_path: Option<&Path>) {
     let kernel_path = Path::new(kernel_binary_path);
-    let ramdisk_path = match ramdisk_path {
-        Some(rdp) => Some(Path::new(rdp)),
-        None => None,
-    };
 
     // create an MBR disk image for legacy BIOS booting
     let mbr_path = kernel_path.with_extension("mbr");

tests/test_kernels/ramdisk/src/bin/ramdisk.rs

@@ -27,8 +27,6 @@ fn kernel_main(boot_info: &'static mut BootInfo) -> ! {
 #[cfg(not(test))]
 #[panic_handler]
 fn panic(info: &core::panic::PanicInfo) -> ! {
-    use core::fmt::Write;
-
     let _ = writeln!(test_kernel_ramdisk::serial(), "PANIC: {}", info);
     exit_qemu(QemuExitCode::Failed);
 }