Re: [libvirt] [PATCH v3 2/3] tests: Add test for new virkmod functions

2014-02-04 Thread Ján Tomko
On 01/30/2014 06:50 PM, John Ferlan wrote: Adding tests for new virKMod{Config|Load|Unload}() API's. A test for virKModIsBlacklisted() would require some setup which cannot be assumed. Signed-off-by: John Ferlan jfer...@redhat.com --- .gitignore | 2 + tests/Makefile.am |

Re: [libvirt] [PATCH v3 2/3] tests: Add test for new virkmod functions

2014-02-04 Thread John Ferlan
On 02/04/2014 06:06 AM, Ján Tomko wrote: On 01/30/2014 06:50 PM, John Ferlan wrote: Adding tests for new virKMod{Config|Load|Unload}() API's. A test for virKModIsBlacklisted() would require some setup which cannot be assumed. Signed-off-by: John Ferlan jfer...@redhat.com --- .gitignore

Re: [libvirt] [PATCH v3 2/3] tests: Add test for new virkmod functions

2014-02-04 Thread Ján Tomko
On 02/04/2014 03:01 PM, John Ferlan wrote: On 02/04/2014 06:06 AM, Ján Tomko wrote: On 01/30/2014 06:50 PM, John Ferlan wrote: Adding tests for new virKMod{Config|Load|Unload}() API's. A test for virKModIsBlacklisted() would require some setup which cannot be assumed. Signed-off-by:

Re: [libvirt] [PATCH v3 2/3] tests: Add test for new virkmod functions

2014-02-04 Thread Eric Blake
On 02/04/2014 07:07 AM, Ján Tomko wrote: /docs/todo.html.in +/examples/domain-events/ /examples/object-events/event-test /examples/dominfo/info1 /examples/domsuspend/suspend Unrelated change. OK - I'll send this separately (trivially). Actually, domain-events has been renamed to

Re: [libvirt] [PATCH v3 2/3] tests: Add test for new virkmod functions

2014-01-31 Thread John Ferlan
On 01/30/2014 12:50 PM, John Ferlan wrote: Adding tests for new virKMod{Config|Load|Unload}() API's. A test for virKModIsBlacklisted() would require some setup which cannot be assumed. Signed-off-by: John Ferlan jfer...@redhat.com --- .gitignore | 2 + ...snip... diff

[libvirt] [PATCH v3 2/3] tests: Add test for new virkmod functions

2014-01-30 Thread John Ferlan
Adding tests for new virKMod{Config|Load|Unload}() API's. A test for virKModIsBlacklisted() would require some setup which cannot be assumed. Signed-off-by: John Ferlan jfer...@redhat.com --- .gitignore | 2 + tests/Makefile.am | 5 ++ tests/virkmodtest.c | 183