refactor testTomlStr to be reusable in test functions; drop copy-pasta comment

Author: savil

cuecfg/cuecfg_test.go

@@ -6,39 +6,44 @@ import (
 "github.com/stretchr/testify/require"
 )
 
+type Metadata struct {
+Tags []string
+}
+
 type MyConfig struct {
 Version int
 Name string
-Tags []string
+Meta *Metadata
 }
 
 var testTomlCfg = &MyConfig{
 Version: 2,
 Name: "go-toml",
-Tags: []string{"go", "toml"},
+Meta: &Metadata{
+Tags: []string{"go", "toml"},
+},
 }
 
+var testTomlStr = `Version = 2
+Name = 'go-toml'
+
+[Meta]
+Tags = ['go', 'toml']
+`
+
 func TestMarshalToml(t *testing.T) {
 req := require.New(t)
 
 bytes, err := Marshal(testTomlCfg, ".toml")
 req.NoError(err)
 
-expected := `Version = 2
-Name = 'go-toml'
-Tags = ['go', 'toml']
-`
-req.Equal(expected, string(bytes))
+req.Equal(testTomlStr, string(bytes))
 }
 
 func TestUnmarshalToml(t *testing.T) {
 req := require.New(t)
-tomlStr := `Version = 2
-Name = 'go-toml'
-Tags = ['go', 'toml']
-`
 cfg := &MyConfig{}
-err := Unmarshal([]byte(tomlStr), ".toml", cfg)
+err := Unmarshal([]byte(testTomlStr), ".toml", cfg)
 req.NoError(err)
 req.Equal(testTomlCfg, cfg)
 }

cuecfg/toml.go

@@ -7,9 +7,6 @@ import (
 "github.com/pelletier/go-toml/v2"
 )
 
-// TODO: consider using cue's JSON marshaller instead of
-// "encoding/json" ... it might have extra functionality related
-// to the cue language.
 func MarshalToml(v interface{}) ([]byte, error) {
 return toml.Marshal(v)
 }