-
-
Notifications
You must be signed in to change notification settings - Fork 17
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.
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
test(backend): add Rust tests #104
base: main
Are you sure you want to change the base?
Conversation
Is there any issue merging the tests which are there now? No need to implement all of them in a single PR IMO. Could be also done on demand when a certain bug was found or certain page/feature code touched anyway. |
A few of them still use Warp, which would be a problem. But I could probably fix them up and merge them soon (though I'd probably scrap them and start again actually, too much has changed). The problem is that it's really hard to test functions that return things like CPU percentage, which is in no way fixed. The only thing I've found to do is bounds checking and making sure the function doesn't panic. |
Signed-off-by: MichaIng <micha@dietpi.com>
I just resolved conflicts with
I see, so they need to be updated 😉. |
Sadly, since we can't make any assumptions about the computer DietPi is running on, most of these tests will have to be incredibly basic, and some of them won't even run on the GitHub Actions Ubuntu VM.
round_percent
cpu
ram
swap
disk
network
processes
dpsoftware
host
services
browser_handler
(Test all functions of file browser)file_handler
term_handler
config