Fix readonly in mounts.

Signed-off-by: Dmitri Zimine dz@stackstorm.com

Author: Dmitri

docker/types/services.py

@@ -135,6 +135,7 @@ def __init__(self, target, source, type='volume', read_only=False,
                 'Only acceptable mount types are `bind` and `volume`.'
             )
         self['Type'] = type
+        self['ReadOnly'] = read_only
 
         if type == 'bind':
             if propagation is not None:
@@ -174,7 +175,7 @@ def parse_mount_string(cls, string):
         else:
             target = parts[1]
             source = parts[0]
-            read_only = not (len(parts) == 3 or parts[2] == 'ro')
+            read_only = not (len(parts) == 2 or parts[2] == 'rw')
             return cls(target, source, read_only=read_only)
 
 

tests/unit/types.py

@@ -0,0 +1,16 @@
+import unittest
+from docker.types.services import Mount
+
+
+class TestMounts(unittest.TestCase):
+    def test_parse_mount_string_docker(self):
+        mount = Mount.parse_mount_string("foo/bar:/buz:ro")
+        self.assertEqual(mount['Source'], "foo/bar")
+        self.assertEqual(mount['Target'], "/buz")
+        self.assertEqual(mount['ReadOnly'], True)
+
+        mount = Mount.parse_mount_string("foo/bar:/buz:rw")
+        self.assertEqual(mount['ReadOnly'], False)
+
+        mount = Mount.parse_mount_string("foo/bar:/buz")
+        self.assertEqual(mount['ReadOnly'], False)