search

uutils / findutils

Rust implementation of findutils
MIT License
309 stars 38 forks source link

find: Don't swallow mkfifo errors in the tests #179

Closed refi64 closed 2 years ago

refi64 commented 2 years ago

Use nix's version of mkfifo instead, which lets us easily access the errors and then fail with more information than the FIFO mysteriously not appearing.

Ref. #177.

Signed-off-by: Ryan Gonzalez ryan.gonzalez@collabora.com

codecov[bot] commented 2 years ago

Codecov Report

Merging #179 (07916e1) into main (96de758) will increase coverage by 0.09%. The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #179      +/-   ##
==========================================
+ Coverage   49.03%   49.12%   +0.09%     
==========================================
  Files          30       30              
  Lines        4913     4914       +1     
  Branches     1290     1297       +7     
==========================================
+ Hits         2409     2414       +5     
+ Misses       1849     1843       -6     
- Partials      655      657       +2
Impacted Files Coverage Δ
src/find/matchers/printf.rs 67.48% <ø> (ø)
src/xargs/mod.rs 77.72% <0.00%> (-0.52%) :arrow_down:
src/lib.rs 28.25% <0.00%> (+0.24%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.