Allow disabling argument quoting with `run` function for WinRM backends · pytest-dev/pytest-testinfra#310

(1 留言) (0 反應) (0 負責人)Python (2,156 star) (318 fork)batch import

help wantedwindows

描述

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?

貢獻者指南

技術棧: python
領域: testing
議題類型: feature
難度: 2
預計時間: 1-3 hours
活動狀態: stale
清晰度: clear
前置要求: PythonpytesttestinfraWinRM
新手友善度: 60
研究方向: The issue requests an option to disable argument quoting when using the `run` function with WinRM backends. The relevant code is in the WinRM backend, likely `testinfra/backend/winrm.py`, where the `run` method constructs commands. The pywinrm library handles quoting, so one approach is to add a `quote` parameter (default True) to the `run` function and pass it through to the backend. The maintainer should confirm whether a global flag or per call option is preferred. Additionally, the report mentions that `CommandResult.stdout` is binary; that is a separate issue but may need attention.