<div dir="ltr"><div dir="auto">Hi Jan and guys,<div dir="auto"><br></div><div dir="auto">I was facing a weird issue when I was rewriting this.</div><div dir="auto">If I start with the <source/> tag, the test fails.</div><div dir="auto">I need to start with <mac address/> tag to work properly.</div><div dir="auto"><br></div><div dir="auto">Test fails:<br>...<br></div>    <interface type='network'><br>
      <source network='default'/><br>      <mac address='52:54:00:e5:48:58'/><br>...<br><br></div><div>Test returns ok:<br>...<br>    <interface type='network'><br>      <mac address='52:54:00:e5:48:58'/><br>
      <source network='default'/><br>...<br><br></div><div>In my point of view, both structure means the same thing.<br></div><div>It is happening because virDomainNetDefFormat() generates <mac address/> right <br>after <interface/> tag.<br></div><div dir="auto"><div class="gmail_extra" dir="auto"><br><div class="gmail_quote">Em 23 de nov de 2017 2:48 PM, "Ján Tomko" <<a href="mailto:jtomko@redhat.com" target="_blank">jtomko@redhat.com</a>> escreveu:<br type="attribution"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Fri, Nov 17, 2017 at 07:27:40PM -0200, Julio Faracco wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Some test cases have the backend tag inside wrong interfaces. The backend xml<br>
tag does not support <interface type='user|direct|hostdev'>. So this commit<br>
changes some network types inside the interfaces that have backend defined.<br>
<br>
Signed-off-by: Julio Faracco <<a href="mailto:jcfaracco@gmail.com" target="_blank">jcfaracco@gmail.com</a>><br>
---<br>
tests/qemuxml2argvdata/qemuxml<wbr>2argv-tap-vhost-incorrect.xml     | 6 ++++--<br>
tests/qemuxml2argvdata/qemuxml<wbr>2argv-tap-vhost.xml               | 3 ++-<br>
tests/qemuxml2xmloutdata/qemux<wbr>ml2xmlout-tap-vhost-incorrect.<wbr>xml | 6 ++++--<br>
tests/qemuxml2xmloutdata/qemux<wbr>ml2xmlout-tap-vhost.xml           | 3 ++-<br>
4 files changed, 12 insertions(+), 6 deletions(-)<br>
<br>
</blockquote>
<br>
ACK and pushed.<br>
<br>
Jan<br>
</blockquote></div></div>
</div>
</div>