Closed SteveLauC closed 1 month ago
That's weird. That line checks the total number of inodes on the file system. That should never change, at least on a traditional file system like UFS. But apparently it can change here. I wonder if it was /dev or one of the real file systems?
That test checks 4 paths, according to the output of mount
on my mac, they refer to:
#[test]
fn statfs_call() {
check_statfs("/tmp"); // refers to the file system mounted at `/`
check_statfs("/dev"); // refers to the file system mounted at `/dev`
check_statfs("/run"); // will be skipped since this directory does not exist
check_statfs("/"); // refers to the file system mounted at `/`
}
I wonder if it was /dev or one of the real file systems?
So yes, it indeed checks /dev
though I am not sure if it is the one that is causing the failure since they all run in 1 test.
I think we should just remove the check for number of inodes. After all, our goal isn't to test statvfs itself, just to test that our bindings to it are doing the right thing.
I think we should just remove the check for number of inodes. After all, our goal isn't to test statvfs itself, just to test that our bindings to it are doing the right thing.
Yeah, let me remove it.
Sometimes, test
statfs_call()
fails in our macOS CI, e.g., this one:It failed in this line, where
statfs()
andstatvfs()
were reporting different number of i-nodes.