GitList

Browse code

docker compose interpolation format error now includes a hint on escaping $ characters.

Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>

Arash Deshmeh authored on 2017/02/18 14:29:51
Showing 2 changed files

cli/compose/interpolation/interpolation.go index 734f28e..29c2e0e 100644
cli/compose/interpolation/interpolation_test.go index c392170..1852b9e 100644

cli/compose/interpolation/interpolation.go

@@ -39,7 +39,7 @@ func interpolateSectionItem(
 		interpolatedValue, err := recursiveInterpolate(value, mapping)
 		if err != nil {
 			return nil, fmt.Errorf(
-				"Invalid interpolation format for %#v option in %s %#v: %#v",
+				"Invalid interpolation format for %#v option in %s %#v: %#v. You may need to escape any $ with another $.",
 				key, section, name, err.Template,
 			)
 		}

cli/compose/interpolation/interpolation_test.go

History View file @ 6d2a7ee

@@ -55,5 +55,5 @@ func TestInvalidInterpolation(t *testing.T) {
 		},
 	}
 	_, err := Interpolate(services, "service", defaultMapping)
-	assert.EqualError(t, err, `Invalid interpolation format for "image" option in service "servicea": "${"`)
+	assert.EqualError(t, err, `Invalid interpolation format for "image" option in service "servicea": "${". You may need to escape any $ with another $.`)
 }

...	...	@@ -39,7 +39,7 @@ func interpolateSectionItem(
39	39	interpolatedValue, err := recursiveInterpolate(value, mapping)
40	40	if err != nil {
41	41	return nil, fmt.Errorf(
42		- "Invalid interpolation format for %#v option in %s %#v: %#v",
	42	+ "Invalid interpolation format for %#v option in %s %#v: %#v. You may need to escape any $ with another $.",
43	43	key, section, name, err.Template,
44	44	)
45	45	}

...	...	@@ -55,5 +55,5 @@ func TestInvalidInterpolation(t *testing.T) {
55	55	},
56	56	}
57	57	_, err := Interpolate(services, "service", defaultMapping)
58		- assert.EqualError(t, err, `Invalid interpolation format for "image" option in service "servicea": "${"`)
	58	+ assert.EqualError(t, err, `Invalid interpolation format for "image" option in service "servicea": "${". You may need to escape any $ with another $.`)
59	59	}