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

Fix dmesg test so that testtool works in CI #2809

Merged
merged 2 commits into from
Nov 22, 2023

Conversation

rminnich
Copy link
Member

Container environments are not always truthful about the UID. Sometimes UID 0 in a container can not do root-level operations.

Also, use errors.Is instead of comparing strings;
embed returned errors with %w for testing.

Container environments are not always truthful about
the UID. Sometimes UID 0 in a container can not do
root-level operations.

Also, use errors.Is instead of comparing strings;
embed returned errors with %w for testing.

Signed-off-by: Ronald G Minnich <rminnich@gmail.com>
@rminnich rminnich added the Awaiting reviewer Waiting for a reviewer. label Nov 22, 2023
@rminnich rminnich added automerge Applying this label auto-merges the PR when ready Awaiting author Waiting for new changes or feedback for author. and removed Awaiting reviewer Waiting for a reviewer. labels Nov 22, 2023
Copy link

codecov bot commented Nov 22, 2023

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (43ab06f) 75.14% compared to head (8ae68d4) 75.30%.

Files Patch % Lines
cmds/core/dmesg/dmesg.go 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2809      +/-   ##
==========================================
+ Coverage   75.14%   75.30%   +0.15%     
==========================================
  Files         432      432              
  Lines       42936    42936              
==========================================
+ Hits        32266    32334      +68     
+ Misses      10670    10602      -68     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rminnich rminnich merged commit 085635f into u-root:main Nov 22, 2023
23 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Applying this label auto-merges the PR when ready Awaiting author Waiting for new changes or feedback for author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants