From 4b804e85165a29f9d712f1ec4f289040f942f459 Mon Sep 17 00:00:00 2001
From: Jhon Honce <jhonce@redhat.com>
Date: Thu, 17 May 2018 11:57:59 -0700
Subject: Implement podman.containers.commit()

- Add API support
- Update tests
- Make changes from reviews

Signed-off-by: Jhon Honce <jhonce@redhat.com>

Closes: #798
Approved by: mheon
---
 contrib/python/test/test_containers.py | 46 +++++++++++++++++++++++++++++++---
 1 file changed, 42 insertions(+), 4 deletions(-)

(limited to 'contrib/python/test/test_containers.py')

diff --git a/contrib/python/test/test_containers.py b/contrib/python/test/test_containers.py
index 344d40d0d..1b2603037 100644
--- a/contrib/python/test/test_containers.py
+++ b/contrib/python/test/test_containers.py
@@ -109,13 +109,13 @@ class TestContainers(PodmanTestCase):
         self.alpine_ctnr.refresh()
         self.assertFalse(self.alpine_ctnr.running)
 
-    @unittest.skip('Datetime values from inspect missing offset')
     def test_inspect(self):
         actual = self.alpine_ctnr.inspect()
         self.assertEqual(actual.id, self.alpine_ctnr.id)
-        self.assertEqual(
-            datetime_parse(actual.created),
-            datetime_parse(self.alpine_ctnr.createdat))
+        # TODO: Datetime values from inspect missing offset in CI instance
+        # self.assertEqual(
+        #     datetime_parse(actual.created),
+        #     datetime_parse(self.alpine_ctnr.createdat))
 
     def test_export(self):
         target = os.path.join(self.tmpdir, 'alpine_export_ctnr.tar')
@@ -125,6 +125,44 @@ class TestContainers(PodmanTestCase):
         self.assertTrue(os.path.isfile(target))
         self.assertGreater(os.path.getsize(target), 0)
 
+    def test_commit(self):
+        # TODO: Test for STOPSIGNAL when supported by OCI
+        # TODO: Test for message when supported by OCI
+        # TODO: Test for EXPOSE when issue#795 fixed
+        #       'EXPOSE=8888/tcp, 8888/udp'
+        id = self.alpine_ctnr.commit(
+            'alpine3',
+            author='Bozo the clown',
+            changes=[
+                'CMD=/usr/bin/zsh',
+                'ENTRYPOINT=/bin/sh date',
+                'ENV=TEST=test_containers.TestContainers.test_commit',
+                'LABEL=unittest=test_commit',
+                'USER=bozo:circus',
+                'VOLUME=/data',
+                'WORKDIR=/data/application',
+            ],
+            pause=True)
+        img = self.pclient.images.get(id)
+        self.assertIsNotNone(img)
+
+        details = img.inspect()
+        self.assertEqual(details.author, 'Bozo the clown')
+        self.assertListEqual(['/usr/bin/zsh'], details.containerconfig['cmd'])
+        self.assertListEqual(['/bin/sh date'],
+                             details.containerconfig['entrypoint'])
+        self.assertListEqual(
+            ['TEST=test_containers.TestContainers.test_commit'],
+            details.containerconfig['env'])
+        # self.assertDictEqual({
+        #     '8888/tcp': {}
+        # }, details.containerconfig['exposedports'])
+        self.assertDictEqual({'unittest': 'test_commit'}, details.labels)
+        self.assertEqual('bozo:circus', details.containerconfig['user'])
+        self.assertEqual({'/data': {}}, details.containerconfig['volumes'])
+        self.assertEqual('/data/application',
+                         details.containerconfig['workingdir'])
+
     def test_remove(self):
         before = self.loadCache()
 
-- 
cgit v1.2.3-54-g00ecf