From 123039102a5664276af23a2fc4be4e4b352bf739 Mon Sep 17 00:00:00 2001 From: Timur Sufiev Date: Tue, 10 Dec 2013 14:47:21 +0400 Subject: [PATCH] Support passing networking info during environment create, not update. Change-Id: Icf1a96d18d8837ed612aef227e0dd4c245c3515a Closes-bug: #1257629 --- muranoclient/v1/environments.py | 9 ++++----- tests/test_methods.py | 4 ++-- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/muranoclient/v1/environments.py b/muranoclient/v1/environments.py index 78f1ff4d..bdebbc00 100644 --- a/muranoclient/v1/environments.py +++ b/muranoclient/v1/environments.py @@ -37,13 +37,12 @@ class EnvironmentManager(base.Manager): def list(self): return self._list('/environments', 'environments') - def create(self, name): - return self._create('/environments', {'name': name}) + def create(self, body): + return self._create('/environments', body) - def update(self, environment_id, name, **kwargs): - kwargs.update({'name': name}) + def update(self, environment_id, name): return self._update('/environments/{id}'.format(id=environment_id), - body=kwargs) + body={'name': name}) def delete(self, environment_id): return self._delete('/environments/{id}'.format(id=environment_id)) diff --git a/tests/test_methods.py b/tests/test_methods.py index 979baada..e748969e 100644 --- a/tests/test_methods.py +++ b/tests/test_methods.py @@ -47,12 +47,12 @@ class UnitTestsForClassesAndFunctions(unittest.TestCase): def test_env_manager_create(self): manager = environments.EnvironmentManager(api) - result = manager.create('test') + result = manager.create({'name': 'test'}) assert result.body == {'name': 'test'} def test_env_manager_create_with_named_parameters(self): manager = environments.EnvironmentManager(api) - result = manager.create(name='test') + result = manager.create(body={'name': 'test'}) assert result.body == {'name': 'test'} def test_env_manager_create_negative_without_parameters(self):