Skip to content

test: elevate test suite coverage with renderer unit tests, TOCTOU concurrency, and memory scaling - #101

Merged
fxrdhan merged 1 commit into
mainfrom
test/maximum-test-coverage
Aug 31, 2026
Merged

test: elevate test suite coverage with renderer unit tests, TOCTOU concurrency, and memory scaling#101
fxrdhan merged 1 commit into
mainfrom
test/maximum-test-coverage

Conversation

@fxrdhan

@fxrdhan fxrdhan commented Aug 31, 2026

Copy link
Copy Markdown
Owner

Summary

Closing renderer unit-test blindspots and introducing dynamic filesystem concurrency and memory scaling stress test suites:

  • Adds dedicated unit tests with 100% branch coverage for SELinux context parser (securityctx.rs), Windows attributes & flags renderer (flags_windows.rs), Windows permissions & type formatting (permissions_windows.rs), and BSD flags rendering (flags_bsd.rs).
  • Adds dynamic filesystem concurrency and TOCTOU (Time-of-Check to Time-of-Use) stress test suite (dynamic_fs_concurrency.rs), asserting non-panicking resilience during concurrent file deletion, dynamic permission revocation (chmod 000), symlink target swapping/flipping, and rapid file mutation during multi-threaded Rayon traversal.
  • Adds memory scale and RSS bounding adversarial test suite (memory_allocation_limits.rs) across 3,000+ files to guarantee linear memory scaling and prevent memory allocation regressions.
  • Expands virtual pseudo-terminal PTY integration tests (pty_terminal.rs) with extreme terminal geometry (10 columns, 400 columns) and interactive tree/long view verification.

Type of Change

  • 🐛 Bug fix (non-breaking change fixing an issue)
  • ✨ New feature (non-breaking change adding functionality)
  • ⚡ Performance improvement
  • ♻️ Code refactor / clean-up
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 📝 Documentation / Man pages / Completions
  • 🔧 Build / CI / Dependencies

Related Issues & Upstream References

  • Resolves: Test suite coverage elevation to 10/10

How Has This Been Tested?

  • Testing commands executed:
    • cargo test --lib (790 unit tests passed)
    • cargo nextest run --workspace (2,287 nextest cases passed, 0 failed, 0 skipped)
    • cargo test --doc (29 doctests passed)
    • cargo clippy --all-targets --all-features (clean with 0 warnings)
    • cargo fmt --check (clean formatting)
  • Platforms verified:
    • macOS (Apple Silicon arm64)

Contributor Checklist

  • My commits follow Conventional Commits format
  • Upstream references are included in this PR body only, not in commit titles
  • Tests covering the changes have been added/updated
  • cargo clippy --all-targets passes with no warnings
  • cargo nextest run (or cargo test) passes
  • cargo fmt --check / nix fmt passes
  • License headers (SPDX / REUSE) are properly preserved/added

@fxrdhan
fxrdhan force-pushed the test/maximum-test-coverage branch 2 times, most recently from 8776757 to b239953 Compare August 31, 2026 18:50
@fxrdhan
fxrdhan force-pushed the test/maximum-test-coverage branch from b239953 to df1c300 Compare August 31, 2026 18:59
@fxrdhan
fxrdhan merged commit 62d492e into main Aug 31, 2026
18 checks passed
@fxrdhan fxrdhan changed the title test: elevate test suite coverage to maximum with renderer unit tests, TOCTOU concurrency, and memory scaling test: elevate test suite coverage with renderer unit tests, TOCTOU concurrency, and memory scaling Aug 31, 2026
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.

1 participant