Merge pull request #735 from aanand/test-network-mode-version

Explicitly test NetworkMode logic in create_host_config

2 files changed, 12 insertions, 5 deletions

diff --git a/docker/client.py b/docker/client.py
index 542b8c5..9853444 100644
--- a/docker/client.py
+++ b/docker/client.py
@@ -271,6 +271,11 @@ class Client(clientbase.ClientBase):
     def create_host_config(self, *args, **kwargs):
         if not kwargs:
             kwargs = {}
+        if 'version' in kwargs:
+            raise TypeError(
+                "create_host_config() got an unexpected "
+                "keyword argument 'version'"
+            )
         kwargs['version'] = self._version
         return utils.create_host_config(*args, **kwargs)
 
diff --git a/tests/utils_test.py b/tests/utils_test.py
index 91d676f..f10b0d0 100644
--- a/tests/utils_test.py
+++ b/tests/utils_test.py
@@ -144,11 +144,13 @@ class UtilsTest(base.BaseTestCase):
         for filters, expected in tests:
             self.assertEqual(convert_filters(filters), expected)
 
-    def test_create_empty_host_config(self):
-        empty_config = create_host_config(
-            network_mode='', version=DEFAULT_DOCKER_API_VERSION
-        )
-        self.assertEqual(empty_config, {})
+    def test_create_host_config_no_options(self):
+        config = create_host_config(version='1.19')
+        self.assertFalse('NetworkMode' in config)
+
+    def test_create_host_config_no_options_newer_api_version(self):
+        config = create_host_config(version='1.20')
+        self.assertEqual(config['NetworkMode'], 'default')
 
     def test_create_host_config_dict_ulimit(self):
         ulimit_dct = {'name': 'nofile', 'soft': 8096}