From 7cf353ed49a27b9381b71a4a3147ea98839b7d20 Mon Sep 17 00:00:00 2001 From: Andrei Stryia Date: Tue, 11 Aug 2015 17:21:24 +0300 Subject: [PATCH] Tests were fixed --- chef/client.py | 5 +++-- chef/tests/test_api.py | 2 +- chef/tests/test_client.py | 5 ++--- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/chef/client.py b/chef/client.py index 5c45bd4..f7566f4 100644 --- a/chef/client.py +++ b/chef/client.py @@ -12,7 +12,7 @@ def _populate(self, data): if self.platform: self.orgname = data.get('orgname') self.validator = bool(data.get('validator', False)) - self.public_key = data.get('certificate') + self.public_key = data.get('public_key') self.admin = False else: self.admin = bool(data.get('admin', False)) @@ -31,7 +31,7 @@ def to_dict(self): d.update({ 'orgname': self.orgname, 'validator': self.validator, - 'certificate': self.certificate, + 'public_key': self.certificate, 'clientname': self.name, }) else: @@ -48,6 +48,7 @@ def create(cls, name, api=None, admin=False): obj.admin = admin d = api.api_request('POST', cls.url, data=obj) obj.private_key = d['private_key'] + obj.public_key = d['public_key'] return obj def rekey(self, api=None): diff --git a/chef/tests/test_api.py b/chef/tests/test_api.py index 53ed9c4..abd7674 100644 --- a/chef/tests/test_api.py +++ b/chef/tests/test_api.py @@ -17,7 +17,7 @@ def test_basic(self): def test_current_dir(self): api = self.load('current_dir.rb') path = os.path.join(os.path.dirname(__file__), 'configs', 'test_1') - self.assertEqual(api.client, path) + self.assertEqual(os.path.normpath(api.client), path) def test_env_variables(self): try: diff --git a/chef/tests/test_client.py b/chef/tests/test_client.py index 3b5b305..56661bd 100644 --- a/chef/tests/test_client.py +++ b/chef/tests/test_client.py @@ -15,7 +15,6 @@ def test_get(self): self.assertTrue(client.certificate) self.assertEqual(client.private_key, None) - @unittest2.skip('Unknown failure, skipping until tomorrow morning ') def test_create(self): name = self.random() client = Client.create(name) @@ -23,14 +22,14 @@ def test_create(self): self.assertEqual(client.name, name) #self.assertEqual(client.orgname, 'pycheftest') # See CHEF-2019 self.assertTrue(client.private_key) - + self.assertTrue(client.public_key) self.assertIn(name, Client.list()) client2 = Client(name) client2.rekey() + self.assertEqual(client.public_key, client2.public_key) self.assertNotEqual(client.private_key, client2.private_key) - @unittest2.skip('Unknown failure, skipping until tomorrow morning ') def test_delete(self): name = self.random() client = Client.create(name)