Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Tests for the file system protocols #57

Closed
GabrielMajeri opened this issue Sep 28, 2018 · 2 comments
Closed

Tests for the file system protocols #57

GabrielMajeri opened this issue Sep 28, 2018 · 2 comments
Labels
enhancement

Comments

@GabrielMajeri
Copy link
Collaborator

Currently, we have no test coverage for the file system related protocols.

@HadrienG2 on #54:

To prevent filesystem tests from having a long-lasting impact on the testing environment (even if there's a bug in them), it would be good to have them operate on a throwaway ramdisk which QEMU discards at the end.
@GabrielMajeri GabrielMajeri mentioned this issue Sep 28, 2018
Merged
@HadrienG2 HadrienG2 mentioned this issue Jan 4, 2019
Merged
nicholasbishop added a commit to nicholasbishop/uefi-rs that referenced this issue Mar 16, 2022
@nicholasbishop
Expand file system tests
25edc47 
Add code in xtask to build a disk image with a FAT partition, then run
file system tests against that disk (when the test runner is built with
the qemu feature). A disk image is used instead of QEMU's VVFAT option
so that everything is fully controlled (can test exact FS info values)
and isolated from the host.

This partially addresses
rust-osdev#57. There are some more
partition and file system tests that can be added here.
@nicholasbishop nicholasbishop mentioned this issue Mar 16, 2022
Merged
GabrielMajeri pushed a commit that referenced this issue Mar 17, 2022
@nicholasbishop @GabrielMajeri
Expand file system tests
87df393 
Add code in xtask to build a disk image with a FAT partition, then run
file system tests against that disk (when the test runner is built with
the qemu feature). A disk image is used instead of QEMU's VVFAT option
so that everything is fully controlled (can test exact FS info values)
and isolated from the host.

This partially addresses
#57. There are some more
partition and file system tests that can be added here.
@phip1611
Copy link
Contributor

btw: this is partially covered by #475

@phip1611
Copy link
Contributor

I think we can close this. We have multiple tests, see https://github.com/rust-osdev/uefi-rs/blob/main/uefi-test-runner/src/proto/media/known_disk.rs#L301

feel free to re-open if I've missed something

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@GabrielMajeri @phip1611