Remove empty read-only paths condition from read_only integration tests

Signed-off-by: chermehdi <mehdi.cheracher@gmail.com>
youki-dev · Dec 1, 2022 · c21a7d0 · c21a7d0
1 parent 371d87a
commit c21a7d0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 9 deletions.
diff --git a/tests/rust-integration-tests/integration_test/src/tests/hostname/mod.rs b/tests/rust-integration-tests/integration_test/src/tests/hostname/mod.rs
@@ -7,12 +7,9 @@ fn create_spec(hostname: &str) -> Spec {
     SpecBuilder::default()
         .hostname(hostname)
         .linux(
+            // Need to reset the read-only paths
             LinuxBuilder::default()
-                .readonly_paths(vec![
-                    "/proc/bus".to_string(),
-                    "/proc/fs".to_string(),
-                    "/proc/sys".to_string(),
-                ])
+                .readonly_paths(vec![])
                 .build()
                 .expect("error in building linux config"),
         )

diff --git a/tests/rust-integration-tests/runtimetest/src/tests.rs b/tests/rust-integration-tests/runtimetest/src/tests.rs
@@ -11,10 +11,7 @@ pub fn validate_readonly_paths(spec: &Spec) {
             return;
         }
     };
-    if ro_paths.is_empty() {
-        eprintln!("in readonly paths, expected some readonly paths to be set, found none");
-        return;
-    }
+
     // TODO when https://github.com/rust-lang/rust/issues/86442 stabilizes,
     // change manual matching of i32 to e.kind() and match statement