2:21 p.m.
On 06/08/2015 04:06, Chandra Shehkhar Reddy Potula wrote:
Hi Aline,
Thanks for your inputs !!!
The tests we written so far are functional verification test cases
which can be ran against the Kimchi which is installed, up and running
and can validate REST API calls with the real data.
What do you mean by a "Kimchi installed, up and running"? Is it not
assuming a local instance running from local source code?
About the current test cases under folder*/tests*, If I understood
correctly they are suppose to do unit testing, hence majority of them
uses mocking. I went through test cases like test_authorization.py,
test_exception.py, test_host.py, test_plugin.py, test_server.py etc
(by the way I excluded mockmodel and model test files) and I saw the
*import kimchi.mockmodel, *which makes me feel that these cases are
designed for mock testing. They are not meant for functional
verification even though they are REST API calls.
The MockModel is to allow testing functionalities in which we may depend
on system configuration.
For example, how test authorization if you don't know which users and
groups exists on the system? Even though you know the username, what
about the password? How will you test login?
A similar scenario applies to other tests we use MockModel.
Also MockModel also connects to libvirt to do the virtualization work.
The difference is it uses the libvirt Test driver instead of the qemu one.
And as the libvirt Test driver has some limitations we needed to
overwrite some functions. It is better to understand looking at the code
in mockmodel.py.
Could you give me an example of a FVT test you have in mind? Just to
confirm we don't already have some of them upstream.
I agree that we could place functional verification test suite under
the*/tests* folder, but not to mix with existing unit test cases. I do
recommend below structure inside */tests.*
/tests
*|**
**-----*/tests-unit
*|**
**-----*/tests-fvt
/ui/tests
*|**
**-----*/tests-unit
*|**
**-----*/tests -selinium
I'd only suggest to remove the second "tests-" as it seems redundant for
me =)
/tests/unit/
/tests/fvt
/ui/tests/unit
/ui/tests/selenium
Hope this make sense !!!
Regards
Chandra
On 08/06/2015 12:35 AM, Aline Manera wrote:
>
>
> On 04/08/2015 07:22, Chandra Sr Potula wrote:
>>
>> We have developed functional verification regression test suite for
>> Kimchi/Ginger in order to verify the REST API's calls currently
>> available on 1.5.
>>
>> Our proposal is to create the following folder structure in the main
>> stream of Kimchi/Ginger for FVT. From this week we will be
>> submitting the patches.
>>
>> *tests-fvt*
>> |
>> -----*tests-restapi*
>> |
>> -----*tests-ui*
>>
>> Please provide feedback if you have any !!!
>>
>
> We already have unit tests to cover the REST APIs. You can check them
> under /tests
> You can add as many tests as you want there. Usually we are on a way
> to remove test_model and test_mockmodel and separate the tests per
> components: templates, networks, guests, etc.
> About the directory structure, I don't think we need to change it.
>
> For UI tests, we need to wait a little bit to do that according to
> the new UI.
> Samuel will do unit tests and Selenium tests as soon as the new UI
> components are getting ready.
> About the directory structure, we can create a new directory under
> /ui, for example, ui/tests to handle all the unit tests and
> ui/tests/selenium to handle the Selenium tests.
>
>>
>> Thanks in advance.
>> Chandra
>>
>>
>>
>> _______________________________________________
>> Kimchi-devel mailing list
>> Kimchi-devel(a)ovirt.org
>> http://lists.ovirt.org/mailman/listinfo/kimchi-devel
>
>
>
> _______________________________________________
> Kimchi-devel mailing list
> Kimchi-devel(a)ovirt.org
> http://lists.ovirt.org/mailman/listinfo/kimchi-devel