-
-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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 all nix expressions where applicable #46632
Comments
This will also be really great for stable releases, since if implemented in full, we can ensure all packages are at least minimally tested. |
Some packages offer installed tests #34987 |
While the system we have right now is good for contributing, I think each new nix expression should have a test in `nixos/tests` written along with it in each new PR.
NixOS tests for simple things are currently very frustrating to debug, because the overhead of an empty NixOS test is larger than the runtime of a non-NixOS-test LibreOffice functional test.
And naive approaches to non-NixOS-based tests got rejected because launching DBus and X requires a minor duplication of functionality (and so it was rejected).
Installed tests shipped with packages come pre-debugged (hopefully), so it should be less of a problem there. Of course, some might depend on network access, but that should be easy to detect and block the coresponding tests.
So if you want to see more NixOS tests, your best bet probably includes something about a NixOS profile with minimal boot time. Maybe even with some code to boot it in a Nix build sandbox (used as a container).
|
Thank you for your contributions. This has been automatically marked as stale because it has had no activity for 180 days. If this is still important to you, we ask that you leave a comment below. Your comment can be as simple as "still important to me". This lets people see that at least one person still cares about this. Someone will have to do this at most twice a year if there is no other activity. Here are suggestions that might help resolve this more quickly:
|
Issue description
While the system we have right now is good for contributing, I think each new nix expression should have a test in
nixos/tests
written along with it in each new PR.We should do this retroactively where possible for all currently existing packages too.
This way, we can ensure everything works with hydra, and be alerted quickly for broken, but successfully built expressions that may not be actively maintained and (user) tested anymore.
Would it be possible to change the policy in Contributing to do this?
The flow I see is this:
Is this a good idea, something we should implement? I see more benefits that drawbacks.
The text was updated successfully, but these errors were encountered: