fix debug message and add check read access

Signed-off-by: sat0ken <[email protected]>

tests/contest/runtimetest/src/tests.rs

@@ -2,7 +2,6 @@ use std::fs::{self, read_dir};
 use std::os::linux::fs::MetadataExt;
 use std::os::unix::fs::{FileTypeExt, PermissionsExt};
 use std::path::Path;
-
 use anyhow::{bail, Result};
 use nix::errno::Errno;
 use nix::libc;
@@ -11,7 +10,7 @@ use nix::unistd::getcwd;
 use oci_spec::runtime::IOPriorityClass::{self, IoprioClassBe, IoprioClassIdle, IoprioClassRt};
 use oci_spec::runtime::{LinuxDevice, LinuxDeviceType, LinuxSchedulerPolicy, Spec};
 
-use crate::utils::{self, test_dir_write_access, test_read_access, test_write_access};
+use crate::utils::{self, test_dir_read_access, test_dir_write_access, test_read_access, test_write_access};
 
 ////////// ANCHOR: example_hello_world
 pub fn hello_world(_spec: &Spec) {
@@ -554,13 +553,21 @@ pub fn test_validate_root_readonly(spec: &Spec) {
             if errno == Errno::EROFS {
                 /* This is expected */
             } else {
-                eprintln!("readonly root filesystem, error in testing write access for path /");
+                eprintln!("readonly root filesystem, error in testing write access for path /, {}", errno);
+            }
+        }
+        if let Err(e) = test_dir_read_access("/") {
+            let errno = Errno::from_raw(e.raw_os_error().unwrap());
+            if errno == Errno::EROFS {
+                /* This is expected */
+            } else {
+                eprintln!("readonly root filesystem, error in testing read access for path /, {}", errno);
             }
         }
     } else if let Err(e) = test_dir_write_access("/") {
-        let errno = Errno::from_raw(e.raw_os_error().unwrap());
-        if errno == Errno::EROFS {
-            eprintln!("readt only root filesystem is false but write access for path / is err");
+        if e.raw_os_error().is_some() {
+            let errno = Errno::from_raw(e.raw_os_error().unwrap());
+            eprintln!("readt only root filesystem is false but write access for path / is err, {}", errno);
         } else {
             /* This is expected */
         }