Fix a bug in reflect.Into. (#35)

paddycarver · web-flow · commit 72273ada5802 · 2021-06-02T07:42:44.000-07:00
We need to be setting the Elem, not the pointer value.
diff --git a/internal/reflect/reflection.go b/internal/reflect/reflection.go
@@ -30,7 +30,7 @@ func Into(ctx context.Context, val tftypes.Value, target interface{}, opts Optio
 	if err != nil {
 		return err
 	}
-	v.Set(result)
+	v.Elem().Set(result)
 	return nil
 }
 

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ func Into(ctx context.Context, val tftypes.Value, target interface{}, opts Optio`
`30`	`30`	`if err != nil {`
`31`	`31`	`return err`
`32`	`32`	`}`
`33`		`- v.Set(result)`
	`33`	`+ v.Elem().Set(result)`
`34`	`34`	`return nil`
`35`	`35`	`}`
`36`	`36`