From 3ba49a689792eb6e6dcf92a4570114d26db37f0e Mon Sep 17 00:00:00 2001 From: Yujun Zhang Date: Tue, 29 Nov 2016 09:37:41 +0800 Subject: Enable all unit tests except two failed cases - `test_ping` in `env_setup_test` still halts - `test_prepare_and_run_benchmark_successful` in `args_handler_test` fails - excluded `pep8` since already coverred by opnfv-lint task Change-Id: Ib5ea20c37a27e356aea6d8eed3750b3269bd67f4 Signed-off-by: Yujun Zhang --- tests/unit/api/qtip_server_test.py | 2 -- tests/unit/utils/args_handler_test.py | 2 +- tests/unit/utils/create_zones_test.py | 1 - tests/unit/utils/env_setup_test.py | 2 +- tests/unit/utils/spawn_vm_test.py | 1 - 5 files changed, 2 insertions(+), 6 deletions(-) (limited to 'tests') diff --git a/tests/unit/api/qtip_server_test.py b/tests/unit/api/qtip_server_test.py index 3e92fcb8..96544c95 100644 --- a/tests/unit/api/qtip_server_test.py +++ b/tests/unit/api/qtip_server_test.py @@ -25,7 +25,6 @@ def side_effect_pass(): class TestClass: - @pytest.mark.skip(reason="(yujunz) to be fixed") @pytest.mark.parametrize("body, expected", [ ({'installer_type': 'fuel', 'installer_ip': '10.20.0.2'}, @@ -100,7 +99,6 @@ class TestClass: delete_reply = app_client.delete("/api/v1.0/jobs/%s" % id) assert "successful" in delete_reply.data - @pytest.mark.skip(reason="(yujunz) to be fixed") @pytest.mark.parametrize("body, expected", [ ([{'installer_type': 'fuel', 'installer_ip': '10.20.0.2'}, diff --git a/tests/unit/utils/args_handler_test.py b/tests/unit/utils/args_handler_test.py index 29347154..5cbe766e 100644 --- a/tests/unit/utils/args_handler_test.py +++ b/tests/unit/utils/args_handler_test.py @@ -12,7 +12,7 @@ import qtip.utils.args_handler class TestClass: - @pytest.mark.skip(reason="(yujunz) to be fixed") + @pytest.mark.skip("(yujunz) test fails") @pytest.mark.parametrize("test_input, expected", [ (['fuel', '/home', 'benchmarks/testplan/default/network/iperf_bm.yaml'], ['fuel', '/home', "iperf", diff --git a/tests/unit/utils/create_zones_test.py b/tests/unit/utils/create_zones_test.py index 48f6c10e..8b1e97cc 100644 --- a/tests/unit/utils/create_zones_test.py +++ b/tests/unit/utils/create_zones_test.py @@ -51,7 +51,6 @@ class NovaMock(MagicMock): class TestClass: - @pytest.mark.skip(reason="(yujunz) to be fixed") @pytest.mark.parametrize("test_input, expected", [ (['compute1', 'compute2'], ['create:compute1:compute1', diff --git a/tests/unit/utils/env_setup_test.py b/tests/unit/utils/env_setup_test.py index 5aa21b1b..c40801cc 100644 --- a/tests/unit/utils/env_setup_test.py +++ b/tests/unit/utils/env_setup_test.py @@ -78,7 +78,7 @@ class TestClass: result = filecmp.cmp(get_output("hosts"), "config/hosts") assert result - @pytest.mark.skip(reason="(yujunz) test halt, to be fixed") + @pytest.mark.skip("(yujunz) to be fixed") def test_ping(self, capfd): test_class = Env_setup() mock_ips = mock.Mock(return_value=["127.0.0.1", "10.20.0.29"]) diff --git a/tests/unit/utils/spawn_vm_test.py b/tests/unit/utils/spawn_vm_test.py index 35273467..ba237378 100644 --- a/tests/unit/utils/spawn_vm_test.py +++ b/tests/unit/utils/spawn_vm_test.py @@ -32,7 +32,6 @@ class HeatMock(MagicMock): class TestClass: - @pytest.mark.skip(reason="(yujunz) to be fixed") @pytest.mark.parametrize("test_input, expected", [ ({'availability_zone': ['compute1', 'compute1'], 'OS_image': ['QTIP_CentOS', 'QTIP_CentOS'], -- cgit 1.2.3-korg