always quote hcl map keys

HCL identifiers may need to be quoted, so always quote them to be safe.
This commit is contained in:
James Bardin 2017-02-24 18:22:36 -05:00
parent 38748f0cd8
commit 43f62d2630
2 changed files with 53 additions and 1 deletions

View File

@ -94,7 +94,7 @@ func (e *encodeState) encodeMap(m map[string]interface{}) error {
for i, k := range sortedKeys(m) { for i, k := range sortedKeys(m) {
v := m[k] v := m[k]
e.WriteString(k + " = ") e.WriteString(fmt.Sprintf("%q = ", k))
err := e.encode(v) err := e.encode(v)
if err != nil { if err != nil {
return err return err

View File

@ -0,0 +1,52 @@
package command
import "testing"
// The command package has it's own HCL encoder to encode variables to push.
// Make sure the variable we encode parse correctly
func TestHCLEncoder_parse(t *testing.T) {
cases := []struct {
Name string
Val interface{}
Error bool
}{
{
Name: "int",
Val: 12345,
},
{
Name: "float",
Val: 1.2345,
},
{
Name: "string",
Val: "terraform",
},
{
Name: "list",
Val: []interface{}{"a", "b", "c"},
},
{
Name: "map",
Val: map[string]interface{}{
"a": 1,
},
},
// a numeric looking identifier requires quotes
{
Name: "map_with_quoted_key",
Val: map[string]interface{}{
"0.0.0.0/24": "mask",
},
},
}
for _, c := range cases {
t.Run(c.Name, func(t *testing.T) {
_, err := encodeHCL(c.Val)
if err != nil {
t.Fatal(err)
}
})
}
}