Merge pull request #1303 from aanand/helpful-containers-warning

Show a helpful warning when people try to call `client.containers()`

1 files changed, 11 insertions, 0 deletions

diff --git a/tests/unit/client_test.py b/tests/unit/client_test.py
index aff6973..716827a 100644
--- a/tests/unit/client_test.py
+++ b/tests/unit/client_test.py
@@ -1,5 +1,6 @@
 import datetime
 import docker
+from docker.utils import kwargs_from_env
 import os
 import unittest
 
@@ -59,6 +60,16 @@ class ClientTest(unittest.TestCase):
         assert "'DockerClient' object has no attribute 'abcdef'" in s
         assert "this method is now on the object APIClient" not in s
 
+    def test_call_containers(self):
+        client = docker.Client(**kwargs_from_env())
+
+        with self.assertRaises(TypeError) as cm:
+            client.containers()
+
+        s = str(cm.exception)
+        assert "'ContainerCollection' object is not callable" in s
+        assert "docker.APIClient" in s
+
 
 class FromEnvTest(unittest.TestCase):