lsd
lsd copied to clipboard
Improve sensitivity of last two integration tests
Those tests were only checking that the output was nonempty, so they would pretty much never detect anything fishy. We could go further but it's nontrivial because the output is nondeterministic (because of temporary files) and it's unclear if we want to build complex regexes in the test suite right now. In the meantime, here's a basic improvement.
TODO
- [x] Use
cargo fmt - [ ] ~~Add necessary tests~~
- [ ] ~~Add changelog entry~~
- [ ] ~~Update default config/theme in README (if applicable)~~
- [ ] ~~Update man page at lsd/doc/lsd.md (if applicable)~~
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: bbc2
The full list of commands accepted by this bot can be found here.
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
Codecov Report
Merging #904 (e61b419) into master (762e724) will decrease coverage by
0.82%. Report is 1 commits behind head on master. The diff coverage is100.00%.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@ Coverage Diff @@
## master #904 +/- ##
==========================================
- Coverage 86.53% 85.71% -0.82%
==========================================
Files 49 49
Lines 4848 4852 +4
==========================================
- Hits 4195 4159 -36
- Misses 653 693 +40
| Files Changed | Coverage Δ | |
|---|---|---|
| tests/integration.rs | 100.00% <100.00%> (ø) |
... and 13 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more