[Libvirt-cim] [PATCH 2 of 2] [TEST] Added new tc to verify the RPCS error values

[Kaitlin Rupert]

Deepti B Kalakeri wrote:
> 
> 
> Kaitlin Rupert wrote:
>>
>>>>> + # Creating NETFS pool to verify RPCS error
>>>>> + status = create_pool(server, virt, pool_name, pool_attr, + 
>>>>> mode_type=pool_type, pool_type=dp_cn)
>>>>
>>>> A netfs pool requires a nfsserver running on the system, and not all 
>>>> systems have nfs installed. So I wouldn't use the netfs type pool in 
>>>> this test.
>>>
>>> Other pool types would require user to give inputs. So I found netfs 
>>> the only option to verify the error.
>>> Any suggestion for the pool types ?
>>>>
>>
>> Why not use a directory pool? The case you're trying to test is 
>> whether libvirt-cim returns an error if a pool with that name has 
>> already been specified.
>>
>> For this case, the pool type doesn't matter. The provider does a look 
>> up in libvirt based on the pool name - it doesn't even consider the 
>> pool type.
>>
> yes we can use the dir pool for the duplicate path verification but for 
> verifying the unsupported error we would require a pool other than dir 
> pool.
> 

Ah, right - good point. I'm wondering if the unsupported error should be 
a different test. I'll need to give this some thought.. ordinarily, I 
would suggest having the test skip if there isn't a nfsserver.. but in 
this case, I would like to see the test report pass/failure status on 
the other conditions of the test.

-- 
Kaitlin Rupert
IBM Linux Technology Center
kaitlin at linux.vnet.ibm.com