From 75d788a2e244069a9fd2b34426051eac295f6f0a Mon Sep 17 00:00:00 2001 From: Sean Dague Date: Tue, 25 Aug 2015 12:21:26 -0400 Subject: [PATCH] remove test_update_host_with_extra_param This negative tempest test assumes that input validation will happen on parameters before platform validation of the feature. This feature is only supported on the xenapi backend. There is no explicit guaruntee that input validation will always trump platform validation, and when applying the relaxed validation of Nova v2.0 on v2.1 on a platform that will return a NotImplemented, we might get the NotImplemented back first. Given that this really ends up being a xen platform specific test, and a negative test at that, it doesn't bring a lot of value to tempest and should just be deleted. Change-Id: I513b54eeacee1486ea11b85913ab2d21727fc5f9 --- tempest/api/compute/admin/test_hosts_negative.py | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/tempest/api/compute/admin/test_hosts_negative.py b/tempest/api/compute/admin/test_hosts_negative.py index 2ea7f1ae3..4c8d8a2b9 100644 --- a/tempest/api/compute/admin/test_hosts_negative.py +++ b/tempest/api/compute/admin/test_hosts_negative.py @@ -70,19 +70,6 @@ class HostsAdminNegativeTestJSON(base.BaseV2ComputeAdminTest): status='enable', maintenance_mode='enable') - @test.attr(type=['negative']) - @test.idempotent_id('76e396fe-5418-4dd3-a186-5b301edc0721') - def test_update_host_with_extra_param(self): - # only 'status' and 'maintenance_mode' are the valid params. - hostname = self._get_host_name() - - self.assertRaises(lib_exc.BadRequest, - self.client.update_host, - hostname, - status='enable', - maintenance_mode='enable', - param='XXX') - @test.attr(type=['negative']) @test.idempotent_id('fbe2bf3e-3246-4a95-a59f-94e4e298ec77') def test_update_host_with_invalid_status(self):