Pass appropriate --device arguments from compose to run

This was passing a stringified version of the DeviceMapping struct
instead of a source:target:permissions string.

Signed-off-by: Ryan Fowler <[email protected]>

Author: ryfow

pkg/composer/serviceparser/serviceparser.go

@@ -515,7 +515,7 @@ func newContainer(project *types.Project, parsed *Service, i int) (*Container, e
 	}
 
 	for _, v := range svc.Devices {
-		c.RunArgs = append(c.RunArgs, fmt.Sprintf("--device=%s", v))
+		c.RunArgs = append(c.RunArgs, fmt.Sprintf("--device=%s:%s:%s", v.Source, v.Target, v.Permissions))
 	}
 
 	for _, v := range svc.DNS {

pkg/composer/serviceparser/serviceparser_test.go

@@ -337,6 +337,36 @@ services:
 
 }
 
+func TestParseDevices(t *testing.T) {
+	const dockerComposeYAML = `
+services:
+  foo:
+    image: nginx:alpine
+    devices:
+      - /dev/a
+      - /dev/b:/dev/b
+      - /dev/c:/dev/c:rw
+`
+	comp := testutil.NewComposeDir(t, dockerComposeYAML)
+	defer comp.CleanUp()
+
+	project, err := testutil.LoadProject(comp.YAMLFullPath(), comp.ProjectName(), nil)
+	assert.NilError(t, err)
+
+	fooSvc, err := project.GetService("foo")
+	assert.NilError(t, err)
+
+	foo, err := Parse(project, fooSvc)
+	assert.NilError(t, err)
+
+	t.Logf("foo: %+v", foo)
+	for _, c := range foo.Containers {
+		assert.Assert(t, in(c.RunArgs, "--device=/dev/a:/dev/a:rwm"))
+		assert.Assert(t, in(c.RunArgs, "--device=/dev/b:/dev/b:rwm"))
+		assert.Assert(t, in(c.RunArgs, "--device=/dev/c:/dev/c:rw"))
+	}
+}
+
 func TestParseRelative(t *testing.T) {
 	t.Parallel()