On 02/12/2019 05:38 PM, Caio Carrara wrote:
This is the third attempt to add the multi vm capability to base class of acceptance tests. The difference from first version is that in this current version a simple migration test was added (done by Cleber) so the new code that is being added is properly used and we're not adding "new dead code". The difference from the second version is that in this current version the migration test being added was updated based on last reviews. It was added more status to consider the migration as finished and also assertions was added to check the migration success.
The new implementation of the migration test is very good IMHO.
There are more tests and test cases that can be added, but I'm assuming this patch is complete enough and other series can be sent after covering the other cases. Caio Carrara (2): tests.acceptance: adds multi vm capability for acceptance tests tests.acceptance: adds simple migration test docs/devel/testing.rst | 41 +++++++++++++++++- tests/acceptance/avocado_qemu/__init__.py | 25 +++++++++-- tests/acceptance/migration.py | 53 +++++++++++++++++++++++ 3 files changed, 114 insertions(+), 5 deletions(-) create mode 100644 tests/acceptance/migration.py
Reviewed-by: Wainer dos Santos Moschetta <waine...@redhat.com>
