pytest-dev/pytest-testinfra

Allow disabling argument quoting with `run` function for WinRM backends

Open

#310 aperta il 4 mag 2018

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Python (318 fork)batch import
help wantedwindows

Metriche repository

Star
 (2156 star)
Metriche merge PR
 (Merge medio 87g 19h) (6 PR mergiate in 30 g)

Descrizione

Hello,

Consider the following basic WinRM test:

def test_echo(host):
    hello = 'Hello, world!'
    res = host.run('echo %s', hello)
    assert res.stdout == hello # Fails

    assert res.stdout== hello
E   assert '"Hello, world!"\r\n' == 'Hello, world!'
# I'm fine with that extra CRLF, but the quotes bugged me for some time...

Related log entry: INFO:testinfra:RUN CommandResult(command='echo "Hello, world!"', exit_status=0, stdout=b'"Hello, world!"\r\n', stderr=None)

It would be nice to be able to turn off argument quoting for executables which behave like MS echo (they echo back quotes, apparently :panda_face: ). Pywinrm already seems to handle arguments escaping/quoting, but I haven't played much with this library yet. It could be possible to let pywinrm handle the command formatting/escaping.

Unrelated to this issue, worth mentioning: the CommandResult.stdout value is a binary string instead of a string when using run with WinRM. Is it intended?

Guida contributor

Direzione di ricerca
La richiesta del problema è un'opzione per disabilitare la quotatura degli argomenti quando si utilizza la funzione `run` con i backend WinRM. Il codice rilevante si trova nel backend WinRM, probabilmente in `testinfra/backend/winrm.py`, dove il metodo `run` costruisce i comandi. La libreria pywinrm gestisce la quotatura, quindi un approccio è aggiungere un parametro `quote` (predefinito True) alla funzione `run` e passarlo al backend. Il manutentore dovrebbe confermare se è preferito un flag globale o un'opzione per chiamata. Inoltre, il report menziona che `CommandResult.stdout` è binario; si tratta di un problema separato ma potrebbe richiedere attenzione.
Tech stack
python
Dominio
testingbackenddevops
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60