On 2013-7-17 0:21, Chris Evich wrote:
>
> In libvirt/tests/cfg/virsh_cmd/domain/virsh_domiflist.cfg:
>
> >  +                    domname_options = "name"
> >  +        - with_invalid_option:
> >  +            status_error = "yes"
> >  +            variants:
> >  +                - none:
> >  +                    domname_options = ""
> >  +                - invalid_domid:
> >  +                    domname_options = "9999"
> >  +                - invalid_uuid:
> >  +                    domname_options = "99999999-9999-9999-9999-999999999999"
> >  +                - addition_invalid_param:
> >  +                    domname_options = "id"
> >  +                    extra_options = "xyz"
> >  +                - libvirtd_stop:
> >  +                    domname_options = "name"
> >  +                    libvirtd = "off"
>
> This can mess up other tests and isn't something we need to test for 
> extensively. Any problems found will have reply from engineering: 
> "don't do that" :D It's okay to just drop this variant.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/autotest/virt-test/pull/565/files#r5220113>.
>
Agree, just like I said, you can remove it and relational codes.

-- 
Regards,
--------------------------------------------------
Li Yang
TEL：+86+25-86630566-8529
EMail：liyang.fnst@cn.fujitsu.com
--------------------------------------------------