mirror of
https://github.com/bootandy/dust.git
synced 2026-06-08 11:29:05 +03:00
[GH-ISSUE #363] test_apparent_size fails #162
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @photosheep on GitHub (Jan 10, 2024).
Original GitHub issue: https://github.com/bootandy/dust/issues/363
Test fails on test_apparent_size using v0.9.0 release.
Running
RUST_BACKTRACE=full cargo test --test test_exact_outputresults in the following.@bootandy commented on GitHub (Jan 10, 2024):
interesting,
You can see this build passing on CI: https://github.com/bootandy/dust/actions/workflows/CICD.yml - It runs on linux, mac and windows.
what OS are you using ?
@photosheep commented on GitHub (Jan 11, 2024):
openSUSE Tumbleweed. Tests run correctly on outside of the packaging chroot, but fails inside.
Using the print-to-stderr commit you've recently drafted shows the following stdout.
@bootandy commented on GitHub (Jan 11, 2024):
test_apparent_size is hard to test because different OSs do different things.
I have a feeling this might be because you have one less space than me.
my 'many' line is:
4.0K ┌─┴ many │the characters 4.0K are 4 chars. 44B is 3 chars and so I require 1 more character.Let me have a think about the best way to handle this.
@bootandy commented on GitHub (Jan 11, 2024):
and thanks for being proactive with this:
@bootandy commented on GitHub (Jan 30, 2024):
I think this will fix it :
https://github.com/bootandy/dust/pull/364/commits/a72b168be98c46a488cc933fef09f1ddb2bbcef9
@bootandy commented on GitHub (Jan 30, 2024):
https://github.com/bootandy/dust/pull/364
merged this PR - this will probably fix it
I'd be grateful if you could pull master and re-run the test
thank you.
@photosheep commented on GitHub (Jan 31, 2024):
Tested with master in the build env and it works like a charm. Thank you so much!