Set Location header in server create and rebuild actions

The API documentation states that the Location header should be filled
with the URL of the server. This change implemnts this for both the
create and rebuild actions.

Change-Id: Ic0372aeaea58c9288065e2cfc222ca9257d35c87

1 files changed, 39 insertions, 0 deletions

diff --git a/nova/tests/api/openstack/v2/test_servers.py b/nova/tests/api/openstack/v2/test_servers.py
index 23621fd7f..a792a8998 100644
--- a/nova/tests/api/openstack/v2/test_servers.py
+++ b/nova/tests/api/openstack/v2/test_servers.py
@@ -3813,3 +3813,42 @@ class ServerXMLSerializationTest(test.TestCase):
                                  str(ip['version']))
                 self.assertEqual(str(ip_elem.get('addr')),
                                  str(ip['addr']))
+
+
+class ServerHeadersSerializationTest(test.TestCase):
+    def test_create_location(self):
+        selfhref = 'http://localhost/v2/fake/servers/%s' % FAKE_UUID
+        bookhref = 'http://localhost/fake/servers/%s' % FAKE_UUID
+
+        serializer = servers.HeadersSerializer()
+        response = webob.Response()
+        server = {
+            'links': [{
+                'rel': 'self',
+                'href': selfhref,
+            }, {
+                'rel': 'bookmark',
+                'href': bookhref,
+            }],
+        }
+        serializer.create(response, {'server': server})
+        self.assertEqual(response.headers['Location'], selfhref)
+
+    def test_rebuild_location(self):
+        selfhref = 'http://localhost/v2/fake/servers/%s' % FAKE_UUID
+        bookhref = 'http://localhost/fake/servers/%s' % FAKE_UUID
+
+        serializer = servers.HeadersSerializer()
+        response = webob.Response()
+        server = {
+            'status': 'REBUILD',
+            'links': [{
+                'rel': 'self',
+                'href': selfhref,
+            }, {
+                'rel': 'bookmark',
+                'href': bookhref,
+            }],
+        }
+        serializer.action(response, {'server': server})
+        self.assertEqual(response.headers['Location'], selfhref)