Add stringSlice and int64Slice to pcommon, and let tests handle non-n…

…umber slices (open-telemetry#10148)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

This adds a string slice to pcommon, that will be used by the profiles
pdata.

#### Link to tracking issue

Part of open-telemetry#10109.

#### Testing

This is unit-tested.

cc @mx-psi

---------

Co-authored-by: Pablo Baeyens <[email protected]>

.chloggen/pcommon-string-int64-slices.yaml

@@ -0,0 +1,25 @@
+# Use this changelog template to create an entry for release notes.
+
+# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
+change_type: enhancement
+
+# The name of the component, or a single word describing the area of concern, (e.g. otlpreceiver)
+component: pdata
+
+# A brief description of the change.  Surround your text with quotes ("") if it needs to start with a backtick (`).
+note: Introduce string and int64 slices to pcommon
+
+# One or more tracking issues or pull requests related to the change
+issues: [10148]
+
+# (Optional) One or more lines of additional information to render under the primary note.
+# These lines will be padded with 2 spaces and then inserted directly into the document.
+# Use pipe (|) for multiline entries.
+subtext:
+
+# Optional: The change log or logs in which this entry should be included.
+# e.g. '[user]' or '[user, api]'
+# Include 'user' if the change is relevant to end users.
+# Include 'api' if there is a change to a library API.
+# Default: '[user]'
+change_logs: [api]

pdata/internal/cmd/pdatagen/internal/pcommon_package.go

@@ -24,6 +24,8 @@ var pcommon = &Package{
 		byteSlice,
 		float64Slice,
 		uInt64Slice,
+		int64Slice,
+		stringSlice,
 	},
 }
 
@@ -164,19 +166,51 @@ var resourceField = &messageValueField{
 }
 
 var byteSlice = &primitiveSliceStruct{
-	structName:  "ByteSlice",
-	packageName: "pcommon",
-	itemType:    "byte",
+	structName:           "ByteSlice",
+	packageName:          "pcommon",
+	itemType:             "byte",
+	testOrigVal:          "1, 2, 3",
+	testInterfaceOrigVal: []interface{}{1, 2, 3},
+	testSetVal:           "5",
+	testNewVal:           "1, 5, 3",
 }
 
 var float64Slice = &primitiveSliceStruct{
-	structName:  "Float64Slice",
-	packageName: "pcommon",
-	itemType:    "float64",
+	structName:           "Float64Slice",
+	packageName:          "pcommon",
+	itemType:             "float64",
+	testOrigVal:          "1, 2, 3",
+	testInterfaceOrigVal: []interface{}{1, 2, 3},
+	testSetVal:           "5",
+	testNewVal:           "1, 5, 3",
 }
 
 var uInt64Slice = &primitiveSliceStruct{
-	structName:  "UInt64Slice",
-	packageName: "pcommon",
-	itemType:    "uint64",
+	structName:           "UInt64Slice",
+	packageName:          "pcommon",
+	itemType:             "uint64",
+	testOrigVal:          "1, 2, 3",
+	testInterfaceOrigVal: []interface{}{1, 2, 3},
+	testSetVal:           "5",
+	testNewVal:           "1, 5, 3",
+}
+
+var int64Slice = &primitiveSliceStruct{
+	structName:           "Int64Slice",
+	packageName:          "pcommon",
+	itemType:             "int64",
+	testOrigVal:          "1, 2, 3",
+	testInterfaceOrigVal: []interface{}{1, 2, 3},
+	testSetVal:           "5",
+	testNewVal:           "1, 5, 3",
+}
+
+var stringSlice = &primitiveSliceStruct{
+	structName:           "StringSlice",
+	packageName:          "pcommon",
+	itemType:             "string",
+	testOrigVal:          `"a", "b", "c"`,
+	testInterfaceOrigVal: []interface{}{`"a"`, `"b"`, `"c"`},
+	testSetVal:           `"d"`,
+	testNewVal:           `"a", "d", "c"`,
 }

pdata/internal/cmd/pdatagen/internal/primitive_slice_structs.go

@@ -80,13 +80,13 @@ func (ms {{ .structName }}) EnsureCapacity(newCap int) {
 }
 
 // Append appends extra elements to {{ .structName }}.
-// Equivalent of {{ .lowerStructName }} = append({{ .lowerStructName }}, elms...) 
+// Equivalent of {{ .lowerStructName }} = append({{ .lowerStructName }}, elms...)
 func (ms {{ .structName }}) Append(elms ...{{ .itemType }}) {
 	ms.getState().AssertMutable()
 	*ms.getOrig() = append(*ms.getOrig(), elms...)
 }
 
-// MoveTo moves all elements from the current slice overriding the destination and 
+// MoveTo moves all elements from the current slice overriding the destination and
 // resetting the current instance to its zero value.
 func (ms {{ .structName }}) MoveTo(dest {{ .structName }}) {
 	ms.getState().AssertMutable()
@@ -109,42 +109,42 @@ func copy{{ .structName }}(dst, src []{{ .itemType }}) []{{ .itemType }} {
 const immutableSliceTestTemplate = `func TestNew{{ .structName }}(t *testing.T) {
 	ms := New{{ .structName }}()
 	assert.Equal(t, 0, ms.Len())
-	ms.FromRaw([]{{ .itemType }}{1, 2, 3})
+	ms.FromRaw([]{{ .itemType }}{ {{ .testOrigVal }} })
 	assert.Equal(t, 3, ms.Len())
-	assert.Equal(t, []{{ .itemType }}{1, 2, 3}, ms.AsRaw())
-	ms.SetAt(1, {{ .itemType }}(5))
-	assert.Equal(t, []{{ .itemType }}{1, 5, 3}, ms.AsRaw())
-	ms.FromRaw([]{{ .itemType }}{3})
+	assert.Equal(t, []{{ .itemType }}{ {{ .testOrigVal }} }, ms.AsRaw())
+	ms.SetAt(1, {{ .itemType }}( {{ .testSetVal }} ))
+	assert.Equal(t, []{{ .itemType }}{ {{ .testNewVal }} }, ms.AsRaw())
+	ms.FromRaw([]{{ .itemType }}{ {{ index .testInterfaceOrigVal 2 }} })
 	assert.Equal(t, 1, ms.Len())
-	assert.Equal(t, {{ .itemType }}(3), ms.At(0))
-	
+	assert.Equal(t, {{ .itemType }}( {{ index .testInterfaceOrigVal 2 }} ), ms.At(0))
+
 	cp := New{{ .structName }}()
 	ms.CopyTo(cp)
-	ms.SetAt(0, {{ .itemType }}(2))
-	assert.Equal(t, {{ .itemType }}(2), ms.At(0))
-	assert.Equal(t, {{ .itemType }}(3), cp.At(0))
+	ms.SetAt(0, {{ .itemType }}( {{ index .testInterfaceOrigVal 1 }} ))
+	assert.Equal(t, {{ .itemType }}( {{ index .testInterfaceOrigVal 1 }} ), ms.At(0))
+	assert.Equal(t, {{ .itemType }}({{ index .testInterfaceOrigVal 2 }}), cp.At(0))
 	ms.CopyTo(cp)
-	assert.Equal(t, {{ .itemType }}(2), cp.At(0))
-	
+	assert.Equal(t, {{ .itemType }}({{ index .testInterfaceOrigVal 1 }}), cp.At(0))
+
 	mv := New{{ .structName }}()
 	ms.MoveTo(mv)
 	assert.Equal(t, 0, ms.Len())
 	assert.Equal(t, 1, mv.Len())
-	assert.Equal(t, {{ .itemType }}(2), mv.At(0))
-	ms.FromRaw([]{{ .itemType }}{1, 2, 3})
+	assert.Equal(t, {{ .itemType }}({{ index .testInterfaceOrigVal 1 }}), mv.At(0))
+	ms.FromRaw([]{{ .itemType }}{ {{ .testOrigVal }} })
 	ms.MoveTo(mv)
 	assert.Equal(t, 3, mv.Len())
-	assert.Equal(t, {{ .itemType }}(1), mv.At(0))
+	assert.Equal(t, {{ .itemType }}({{ index .testInterfaceOrigVal 0 }}), mv.At(0))
 }
 
 func Test{{ .structName }}ReadOnly(t *testing.T) {
-	raw := []{{ .itemType }}{1, 2, 3}
+	raw := []{{ .itemType }}{ {{ .testOrigVal }}}
 	state := internal.StateReadOnly
 	ms := {{ .structName }}(internal.New{{ .structName }}(&raw, &state))
 
 	assert.Equal(t, 3, ms.Len())
-	assert.Equal(t, {{ .itemType }}(1), ms.At(0))
-	assert.Panics(t, func() { ms.Append(1) })
+	assert.Equal(t, {{ .itemType }}({{ index .testInterfaceOrigVal 0 }}), ms.At(0))
+	assert.Panics(t, func() { ms.Append({{ index .testInterfaceOrigVal 0 }}) })
 	assert.Panics(t, func() { ms.EnsureCapacity(2) })
 	assert.Equal(t, raw, ms.AsRaw())
 	assert.Panics(t, func() { ms.FromRaw(raw) })
@@ -153,17 +153,17 @@ func Test{{ .structName }}ReadOnly(t *testing.T) {
 	ms.CopyTo(ms2)
 	assert.Equal(t, ms.AsRaw(), ms2.AsRaw())
 	assert.Panics(t, func() { ms2.CopyTo(ms) })
-	
+
 	assert.Panics(t, func() { ms.MoveTo(ms2) })
 	assert.Panics(t, func() { ms2.MoveTo(ms) })
 }
 
 func Test{{ .structName }}Append(t *testing.T) {
 	ms := New{{ .structName }}()
-	ms.FromRaw([]{{ .itemType }}{1, 2, 3})
-	ms.Append(4, 5)
+	ms.FromRaw([]{{ .itemType }}{ {{ .testOrigVal }} })
+	ms.Append({{ .testSetVal }}, {{ .testSetVal }})
 	assert.Equal(t, 5, ms.Len())
-	assert.Equal(t, {{ .itemType }}(5), ms.At(4))
+	assert.Equal(t, {{ .itemType }}({{ .testSetVal }}), ms.At(4))
 }
 
 func Test{{ .structName }}EnsureCapacity(t *testing.T) {
@@ -190,6 +190,16 @@ func Get{{ .structName }}State(ms {{ .structName }}) *State {
 
 func New{{ .structName }}(orig *[]{{ .itemType }}, state *State) {{ .structName }} {
 	return {{ .structName }}{orig: orig, state: state}
+}
+
+func FillTest{{ .structName }}(tv {{ .structName}}) {
+}
+
+func GenerateTest{{ .structName }}() {{ .structName }} {
+	state := StateMutable
+	var orig []{{ .itemType }} = nil
+
+	return {{ .structName }}{&orig, &state}
 }`
 
 // primitiveSliceStruct generates a struct for a slice of primitive value elements. The structs are always generated
@@ -198,6 +208,11 @@ type primitiveSliceStruct struct {
 	structName  string
 	packageName string
 	itemType    string
+
+	testOrigVal          string
+	testInterfaceOrigVal []interface{}
+	testSetVal           string
+	testNewVal           string
 }
 
 func (iss *primitiveSliceStruct) getName() string {
@@ -233,8 +248,12 @@ func (iss *primitiveSliceStruct) generateInternal(sb *bytes.Buffer) {
 
 func (iss *primitiveSliceStruct) templateFields() map[string]any {
 	return map[string]any{
-		"structName":      iss.structName,
-		"itemType":        iss.itemType,
-		"lowerStructName": strings.ToLower(iss.structName[:1]) + iss.structName[1:],
+		"structName":           iss.structName,
+		"itemType":             iss.itemType,
+		"lowerStructName":      strings.ToLower(iss.structName[:1]) + iss.structName[1:],
+		"testOrigVal":          iss.testOrigVal,
+		"testInterfaceOrigVal": iss.testInterfaceOrigVal,
+		"testSetVal":           iss.testSetVal,
+		"testNewVal":           iss.testNewVal,
 	}
 }