Skip to content

selftest: Fix test_rglob_error depending on host system too much - #133

Merged
Rahix merged 1 commit into
masterfrom
fix-rglob-selftest
Sep 1, 2026
Merged

selftest: Fix test_rglob_error depending on host system too much#133
Rahix merged 1 commit into
masterfrom
fix-rglob-selftest

Conversation

@Rahix

@Rahix Rahix commented Sep 1, 2026

Copy link
Copy Markdown
Owner

test_rglob_error made an assumption about files present in sysfs which turned out to not be true everywhere (see #132).

Fix this by instead writing the testcase against a synthetic file structure which will look the same no matter what system it runs on.

Fixes #132.

test_rglob_error made an assumption about files present in sysfs which
turned out to not be true everywhere [1].

Fix this by instead writing the testcase against a synthetic file
structure which will look the same no matter what system it runs on.

[1]: #132
Signed-off-by: Rahix <rahix@rahix.de>
@codecov

codecov Bot commented Sep 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.47%. Comparing base (4d0ee1b) to head (be7671e).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #133   +/-   ##
=======================================
  Coverage   62.47%   62.47%           
=======================================
  Files          53       53           
  Lines        3765     3765           
=======================================
  Hits         2352     2352           
  Misses       1413     1413           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Rahix
Rahix merged commit e792482 into master Sep 1, 2026
3 checks passed
@Rahix
Rahix deleted the fix-rglob-selftest branch September 1, 2026 15:53
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

Successfully merging this pull request may close these issues.

test_rglob_error assumes a cgroup-v1-style /sys/fs/cgroup/cpu directory, fails under pure cgroup v2

1 participant