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.

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

Spread should run the doc tests with a nonroot user #651

Open
cjdcordeiro opened this issue Jul 24, 2024 · 0 comments
Open

Spread should run the doc tests with a nonroot user #651

cjdcordeiro opened this issue Jul 24, 2024 · 0 comments

Comments

@cjdcordeiro
Copy link
Collaborator

What needs to get done

By default, Spread will run the test suites with sudo. To be more comprehensive, and specifically for the doc tests (which are public-facing code blocks that must be reproducible), Spread should also run the tests as a nonroot user.

Why it needs to get done

If we only run the tests with sudo, then we'll fail to identify documentation bits that ignore how permissions might be set in the user's working environment. For example - #649 . In this case, testing that code block with a nonroot user would've helped identify the documentation issue in advance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant