Re: [PATCH 1/6] tests: add test cases for default sound output

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 11/14/2017 08:45 AM, Pavel Hrdina wrote:
> These test cases models current situation where there is no way how
> to specify sound output and that it's based on which graphic device
> is the last one.
> 

Personally had a hard time parsing what the commit message said, but
based on what happens 2 patches from now, I think you're just trying to
add vm's w/ a sound device. Not so sure it's the "default" sound device
or just "a" sound device. Not sure how much clearer you could make
things or if it really matters - I'll leave it up to you to reread and
be sure what's happening is what you expect!

> Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx>
> ---
>  .../qemuxml2argv-sound-default-output-sdl.args     | 23 ++++++++++++++++++++
>  .../qemuxml2argv-sound-default-output-sdl.xml      | 24 +++++++++++++++++++++
>  ...emuxml2argv-sound-default-output-spice-vnc.args | 25 ++++++++++++++++++++++
>  ...qemuxml2argv-sound-default-output-spice-vnc.xml | 25 ++++++++++++++++++++++
>  .../qemuxml2argv-sound-default-output-spice.args   | 24 +++++++++++++++++++++
>  .../qemuxml2argv-sound-default-output-spice.xml    | 24 +++++++++++++++++++++
>  ...emuxml2argv-sound-default-output-vnc-spice.args | 25 ++++++++++++++++++++++
>  ...qemuxml2argv-sound-default-output-vnc-spice.xml | 25 ++++++++++++++++++++++
>  .../qemuxml2argv-sound-default-output-vnc.args     | 24 +++++++++++++++++++++
>  .../qemuxml2argv-sound-default-output-vnc.xml      | 24 +++++++++++++++++++++
>  tests/qemuxml2argvtest.c                           | 22 +++++++++++++++++++
>  11 files changed, 265 insertions(+)
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-sound-default-output-sdl.args
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-sound-default-output-sdl.xml
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-sound-default-output-spice-vnc.args
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-sound-default-output-spice-vnc.xml
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-sound-default-output-spice.args
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-sound-default-output-spice.xml
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-sound-default-output-vnc-spice.args
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-sound-default-output-vnc-spice.xml
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-sound-default-output-vnc.args
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-sound-default-output-vnc.xml
> 

Reviewed-by: John Ferlan <jferlan@xxxxxxxxxx>

John

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux