pantry icon indicating copy to clipboard operation
pantry copied to clipboard

fix hexedit test

Open sanchitram1 opened this issue 8 months ago • 5 comments

sanchitram1 avatar Apr 21 '25 14:04 sanchitram1

I don't think this test helps. If hexedit -h succeeded, the test would fail. And it doesn't understand why it failed. So a broken binary would pass this test, and an update that generated output without curses would fail it.

jhheider avatar Apr 22 '25 15:04 jhheider

fair enough. the real question is how do you test interactive terminal tools like this one?

sanchitram1 avatar Apr 22 '25 19:04 sanchitram1

With rust programs, I usually add a --version flag using sed that doesn't launch the GUI.

jhheider avatar Apr 22 '25 19:04 jhheider

An example? We (the package manager) add a patch and allow it to pass?

sanchitram1 avatar Apr 22 '25 20:04 sanchitram1

See https://github.com/pkgxdev/pantry/blob/main/projects%2Famp.rs%2Fpackage.yml

jhheider avatar Apr 22 '25 20:04 jhheider