On June 10, 2013, 2:05 p.m., Roman Rakus wrote:
Look good. In future it would be good to check also for invalid queries.
Radek Novacek wrote: Yes, checking the queries should be added, but I'll have to think about how to do it correctly.
The logic for checking queries should be in the provider - when the query (or filter) is not valid, the provider will return not OK result. The test could test if provider do it like this. So it will be test to check if provider is correctly checking.
- Roman
----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/440/#review525 -----------------------------------------------------------
On June 11, 2013, 11:05 a.m., Radek Novacek wrote:
This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/440/
(Updated June 11, 2013, 11:05 a.m.)
Review request for OpenLMI Developers.
Repository: openlmi-networking
Description
test: test for lifetime indications
This commit also adds indication infrastructure to the test_base in order to simplify its usage.
Diffs
test/test_base.py af75041038d3dc185772959e678a409518ca4423 test/test_indications.py PRE-CREATION
Diff: http://reviewboard-openlmi.rhcloud.com/r/440/diff/
Testing
Thanks,
Radek Novacek