fix: added visible long flag aliases in help

[GilShoshan94]

fix #6067

fix #6073

[GilShoshan94]

I will probably have to update the ui tests as well

[GilShoshan94]

I updated the ui testes as well in a separated commit

[GilShoshan94]

@epage
The PR is ready.

I added the visible long flag aliases, as well as in the debug and adapted the ui tests.
The only thing not strictly necessary I did is to rename in one closure a to s, because it was confusing as a is already the function argument (a: &Command)

[GilShoshan94]

Hi @epage,

I have an issue in the CI because the test I made has the help to wide and I think it wraps.
See the CI error:

I tried to fix this, by setting .term_width(usize::MAX) but it didn't help....
.term_width(0) didn't solve it either.

I am still looking for a solution, in case you see this before I solve it, your help and guidance will be appreciated as always.

[GilShoshan94]

@epage
I digged more into the help_template.rs to understand why it is wrapping.
I am a bit confused as the test is working fine on my machine....
I tried with .term_width(120) and it did not pass the CI still.

I saw in the code that term_w default to 100, unrelated to the feature wrap_help.
I follow the code, and I don't find why it does not behave the way I expected...
And why it behaves differently in my machine and in the CI...

I added one more test to check all the aliases (visible or not) are working, and I gated the tests for visible aliases under the feature wrap_help.
Now it works.

Let me know if this is good now.

[epage]

Yeah, I'd have to dig more into the wrapping issue. Should be good for now though.

Besides the test issue, this all looks good. Thanks!

[GilShoshan94]

Hi @epage,

I fixed the test issue.
It should be ready to be merged now I believe.

[epage]

Thanks!