for #811: allow some perm warnings in non-verbose mode

[joshrabinowitz]

continuing WIP on #811
Continues changes on merged PR #812

Edit: I need to confirm that removing --no-permission-warning can't cause failures in situations that used to work (because of permission issues previously being ignored)
Edit2: above confimed

[sobolevn]

This looks weird:

# run 'add' with bad arg: FINAL OUTPUT: M�MA�AN�NU�UA�AL�L
# run 'add' with bad arg: FINAL OUTPUT:        Run m�ma�an�n g�gi�it�t-�-s�se�ec�cr�re�et�t-�-a�ad�dd�d to see this note.
# run 'add' with bad arg: FINAL OUTPUT: 
# run 'add' with bad arg: FINAL OUTPUT: S�SE�EE�E A�AL�LS�SO�O
# run 'add' with bad arg: FINAL OUTPUT:        git-secret-init(1)               _�h_�t_�t_�p_�s_�:_�/_�/_�g_�i_�t_�-_�s_�e_�c_�r_�e_�t_�._�i_�o_�/_�g_�i_�t_�-_�s_�e_�c_�r_�e_�t_�-_�i_�n_�i_�t,
# run 'add' with bad arg: FINAL OUTPUT:        git-secret-tell(1)               _�h_�t_�t_�p_�s_�:_�/_�/_�g_�i_�t_�-_�s_�e_�c_�r_�e_�t_�._�i_�o_�/_�g_�i_�t_�-_�s_�e_�c_�r_�e_�t_�-_�t_�e_�l_�l,
# run 'add' with bad arg: FINAL OUTPUT:        git-secret-hide(1)               _�h_�t_�t_�p_�s_�:_�/_�/_�g_�i_�t_�-_�s_�e_�c_�r_�e_�t_�._�i_�o_�/_�g_�i_�t_�-_�s_�e_�c_�r_�e_�t_�-_�h_�i_�d_�e,
# run 'add' with bad arg: FINAL OUTPUT:        git-secret-reveal(1) _�h_�t_�t_�p_�s_�:_�/_�/_�g_�i_�t_�-_�s_�e_�c_�r_�e_�t_�._�i_�o_�/_�g_�i_�t_�-_�s_�e_�c_�r_�e_�t_�-_�r_�e_�v_�e_�a_�l

[joshrabinowitz]

@sobolevn I think the double character and other weird output from man in the test output is because of terminal escape sequences that man is outputting. The verbose tests have always displayed man pages oddly in github actions; I'm not sure how to remedy it. Maybe we should make a separate issue about that. (Edit: See #826)