[pkg/testdata] Fix type of Attribute.Value for profiles (#39006)

The value type is `string`, but it should be `any`.

Author: rockdaboot

.chloggen/profiles-testdata-attr-value-any.yaml

@@ -0,0 +1,27 @@
+# Use this changelog template to create an entry for release notes.
+
+# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
+change_type: bug_fix
+
+# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver)
+component: pkg/pdatatest
+
+# A brief description of the change.  Surround your text with quotes ("") if it needs to start with a backtick (`).
+note: Change type of attribute values from string to any.
+
+# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists.
+issues: [39006]
+
+# (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:
+
+# If your change doesn't affect end users or the exported elements of any package,
+# you should instead start your pull request title with [chore] or use the "Skip Changelog" label.
+# 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]

pkg/pdatatest/pprofiletest/profiles_test.go

@@ -59,6 +59,20 @@ func TestCompareProfiles(t *testing.T) {
 			expected: basicProfiles().Transform(),
 			actual:   basicProfiles().Transform(),
 		},
+		{
+			name: "attribute value string is not bool",
+			expected: func() pprofile.Profiles {
+				p := basicProfiles()
+				p.ResourceProfiles[0].Resource.Attributes[0] = Attribute{"key1", "true"}
+				return p.Transform()
+			}(),
+			actual: func() pprofile.Profiles {
+				p := basicProfiles()
+				p.ResourceProfiles[0].Resource.Attributes[0] = Attribute{"key1", true}
+				return p.Transform()
+			}(),
+			withoutOptions: errors.New(`missing expected resource: map[key1:true]; unexpected resource: map[key1:true]`),
+		},
 		{
 			name: "resource order",
 			expected: func() pprofile.Profiles {

pkg/pdatatest/pprofiletest/types.go

@@ -3,6 +3,8 @@
 
 package pprofiletest // import "github.com/open-telemetry/opentelemetry-collector-contrib/pkg/pdatatest/pprofiletest"
 import (
+	"fmt"
+
 	"go.opentelemetry.io/collector/pdata/pcommon"
 	"go.opentelemetry.io/collector/pdata/pprofile"
 )
@@ -30,7 +32,10 @@ func (rp ResourceProfile) Transform(pp pprofile.Profiles) pprofile.ResourceProfi
 		sp.Transform(prp)
 	}
 	for _, a := range rp.Resource.Attributes {
-		prp.Resource().Attributes().PutStr(a.Key, a.Value)
+		if prp.Resource().Attributes().PutEmpty(a.Key).FromRaw(a.Value) != nil {
+			panic(fmt.Sprintf("unsupported resource attribute value: {%s: %v (type %T)}",
+				a.Key, a.Value, a.Value))
+		}
 	}
 	return prp
 }
@@ -66,7 +71,10 @@ type Scope struct {
 func (sc Scope) Transform(psp pprofile.ScopeProfiles) pcommon.InstrumentationScope {
 	psc := psp.Scope()
 	for _, a := range sc.Attributes {
-		psc.Attributes().PutStr(a.Key, a.Value)
+		if psc.Attributes().PutEmpty(a.Key).FromRaw(a.Value) != nil {
+			panic(fmt.Sprintf("unsupported scope attribute value: {%s: %v (type %T)}",
+				a.Key, a.Value, a.Value))
+		}
 	}
 	psc.SetName(sc.Name)
 	psc.SetVersion(sc.Version)
@@ -274,13 +282,16 @@ func (m *Mapping) Transform(pp pprofile.Profile) {
 
 type Attribute struct {
 	Key   string
-	Value string
+	Value any
 }
 
 func (a *Attribute) Transform(pp pprofile.Profile) int32 {
 	pa := pp.AttributeTable().AppendEmpty()
 	pa.SetKey(a.Key)
-	pa.Value().SetStr(a.Value)
+	if pa.Value().FromRaw(a.Value) != nil {
+		panic(fmt.Sprintf("unsupported attribute value: {%s: %v (type %T)}",
+			a.Key, a.Value, a.Value))
+	}
 	return int32(pp.AttributeTable().Len() - 1)
 }