fix bool attribute equality (#3688)

Author: blakeroberts-wk

model/pdata/common.go

@@ -269,6 +269,10 @@ func (a AttributeValue) Equal(av AttributeValue) bool {
 		return a.orig.Value == av.orig.Value
 	}
 
+	if a.Type() != av.Type() {
+		return false
+	}
+
 	switch v := a.orig.Value.(type) {
 	case *otlpcommon.AnyValue_StringValue:
 		return v.StringValue == av.orig.GetStringValue()

model/pdata/common_test.go

@@ -201,15 +201,15 @@ func TestAttributeValueEqual(t *testing.T) {
 	av1 = NewAttributeValueDouble(123)
 	assert.True(t, av1.Equal(av2))
 
-	av2 = NewAttributeValueBool(true)
+	av2 = NewAttributeValueBool(false)
 	assert.False(t, av1.Equal(av2))
 	assert.False(t, av2.Equal(av1))
 
 	av1 = NewAttributeValueBool(true)
-	assert.True(t, av1.Equal(av2))
+	assert.False(t, av1.Equal(av2))
 
 	av1 = NewAttributeValueBool(false)
-	assert.False(t, av1.Equal(av2))
+	assert.True(t, av1.Equal(av2))
 
 	av1 = NewAttributeValueArray()
 	av1.ArrayVal().AppendEmpty().SetIntVal(123)