From 0875a411aaa111571f6f5e7ae29b5de75f1e3184 Mon Sep 17 00:00:00 2001 From: Matt Riedemann Date: Thu, 14 Jan 2016 14:48:40 -0800 Subject: [PATCH] ServerGroupsV213SampleJsonTest should actually test v2.13 This API samples test was not using testscenarios properly, and it wasn't extending the correct test class, so it was: 1. Not actually running tests. 2. Not running tests at the correct microversion (v2.13). This fixes the testscenarios setup for the subclass and the API samples and templates (since it's not using subs for the intended project_id that was in the samples). Also fixes a typo in ApiSampleTestBaseV21. Change-Id: I27e8fce5d456914d88098c6cfabe05fa8afbc8eb Closes-Bug: #1534354 --- .../os-server-groups/v2.13/server-groups-get-resp.json | 2 +- .../os-server-groups/v2.13/server-groups-list-resp.json | 2 +- .../os-server-groups/v2.13/server-groups-post-resp.json | 2 +- nova/tests/functional/api_sample_tests/api_sample_base.py | 2 +- .../v2.13/server-groups-get-resp.json.tpl | 2 +- .../v2.13/server-groups-list-resp.json.tpl | 2 +- .../v2.13/server-groups-post-resp.json.tpl | 2 +- .../functional/api_sample_tests/test_server_groups.py | 8 ++++---- 8 files changed, 11 insertions(+), 11 deletions(-) diff --git a/doc/api_samples/os-server-groups/v2.13/server-groups-get-resp.json b/doc/api_samples/os-server-groups/v2.13/server-groups-get-resp.json index e4f7f7d707..4f9e2248fc 100644 --- a/doc/api_samples/os-server-groups/v2.13/server-groups-get-resp.json +++ b/doc/api_samples/os-server-groups/v2.13/server-groups-get-resp.json @@ -5,7 +5,7 @@ "policies": ["anti-affinity"], "members": [], "metadata": {}, - "project_id": "c7c9f4f175e247acb56c108fd724d667", + "project_id": "openstack", "user_id": "fake" } } diff --git a/doc/api_samples/os-server-groups/v2.13/server-groups-list-resp.json b/doc/api_samples/os-server-groups/v2.13/server-groups-list-resp.json index bc1dab68ca..d330e4df46 100644 --- a/doc/api_samples/os-server-groups/v2.13/server-groups-list-resp.json +++ b/doc/api_samples/os-server-groups/v2.13/server-groups-list-resp.json @@ -6,7 +6,7 @@ "policies": ["anti-affinity"], "members": [], "metadata": {}, - "project_id": "c7c9f4f175e247acb56c108fd724d667", + "project_id": "openstack", "user_id": "fake" } ] diff --git a/doc/api_samples/os-server-groups/v2.13/server-groups-post-resp.json b/doc/api_samples/os-server-groups/v2.13/server-groups-post-resp.json index e4f7f7d707..4f9e2248fc 100644 --- a/doc/api_samples/os-server-groups/v2.13/server-groups-post-resp.json +++ b/doc/api_samples/os-server-groups/v2.13/server-groups-post-resp.json @@ -5,7 +5,7 @@ "policies": ["anti-affinity"], "members": [], "metadata": {}, - "project_id": "c7c9f4f175e247acb56c108fd724d667", + "project_id": "openstack", "user_id": "fake" } } diff --git a/nova/tests/functional/api_sample_tests/api_sample_base.py b/nova/tests/functional/api_sample_tests/api_sample_base.py index 4d9b25c9c0..6c97e19a93 100644 --- a/nova/tests/functional/api_sample_tests/api_sample_base.py +++ b/nova/tests/functional/api_sample_tests/api_sample_base.py @@ -58,7 +58,7 @@ CONF = cfg.CONF # microversions, then replace the ``scenarios`` class variable in that # test class with something like: # -# [("v2_11", {'api_major_version': 'v2.1', 'microversion', '2.11'})] +# [("v2_11", {'api_major_version': 'v2.1', 'microversion': '2.11'})] class ApiSampleTestBaseV21(testscenarios.WithScenarios, diff --git a/nova/tests/functional/api_sample_tests/api_samples/os-server-groups/v2.13/server-groups-get-resp.json.tpl b/nova/tests/functional/api_sample_tests/api_samples/os-server-groups/v2.13/server-groups-get-resp.json.tpl index 1bdfe9e1b7..d72a5418a6 100644 --- a/nova/tests/functional/api_sample_tests/api_samples/os-server-groups/v2.13/server-groups-get-resp.json.tpl +++ b/nova/tests/functional/api_sample_tests/api_samples/os-server-groups/v2.13/server-groups-get-resp.json.tpl @@ -5,7 +5,7 @@ "policies": ["anti-affinity"], "members": [], "metadata": {}, - "project_id": "c7c9f4f175e247acb56c108fd724d667", + "project_id": "openstack", "user_id": "fake" } } diff --git a/nova/tests/functional/api_sample_tests/api_samples/os-server-groups/v2.13/server-groups-list-resp.json.tpl b/nova/tests/functional/api_sample_tests/api_samples/os-server-groups/v2.13/server-groups-list-resp.json.tpl index 417b9a1a1b..8312769412 100644 --- a/nova/tests/functional/api_sample_tests/api_samples/os-server-groups/v2.13/server-groups-list-resp.json.tpl +++ b/nova/tests/functional/api_sample_tests/api_samples/os-server-groups/v2.13/server-groups-list-resp.json.tpl @@ -6,7 +6,7 @@ "policies": ["anti-affinity"], "members": [], "metadata": {}, - "project_id": "c7c9f4f175e247acb56c108fd724d667", + "project_id": "openstack", "user_id": "fake" } ] diff --git a/nova/tests/functional/api_sample_tests/api_samples/os-server-groups/v2.13/server-groups-post-resp.json.tpl b/nova/tests/functional/api_sample_tests/api_samples/os-server-groups/v2.13/server-groups-post-resp.json.tpl index 1bdfe9e1b7..d72a5418a6 100644 --- a/nova/tests/functional/api_sample_tests/api_samples/os-server-groups/v2.13/server-groups-post-resp.json.tpl +++ b/nova/tests/functional/api_sample_tests/api_samples/os-server-groups/v2.13/server-groups-post-resp.json.tpl @@ -5,7 +5,7 @@ "policies": ["anti-affinity"], "members": [], "metadata": {}, - "project_id": "c7c9f4f175e247acb56c108fd724d667", + "project_id": "openstack", "user_id": "fake" } } diff --git a/nova/tests/functional/api_sample_tests/test_server_groups.py b/nova/tests/functional/api_sample_tests/test_server_groups.py index 1cb256881b..66f4db6b5f 100644 --- a/nova/tests/functional/api_sample_tests/test_server_groups.py +++ b/nova/tests/functional/api_sample_tests/test_server_groups.py @@ -78,7 +78,7 @@ class ServerGroupsSampleJsonTest(api_sample_base.ApiSampleTestBaseV21): self.assertEqual(204, response.status_code) -class ServerGroupsV213SampleJsonTest(api_sample_base.ApiSampleTestBaseV21): - extension_name = "os-server-groups" - request_api_version = '2.13' - scenarios = [('v2_13', {})] +class ServerGroupsV213SampleJsonTest(ServerGroupsSampleJsonTest): + scenarios = [ + ("v2_13", {'api_major_version': 'v2.1', 'microversion': '2.13'}) + ]