Assert that resources is map like before treating as a map (#333)

* Assert that resources is map like before treating as a map

* actually make the test case a thing

Author: Garbett1

processor/loghouseprocessor/processor.go

@@ -152,8 +152,14 @@ func promoteResourceAttrs(l *plog.LogRecord, rlogs *plog.ResourceLogs) {
 	if !ok {
 		return
 	}
-	merged := MergeRawMaps(rlogs.Resource().Attributes().AsRaw(), attributes.Map().AsRaw())
-	_ = rlogs.Resource().Attributes().FromRaw(merged)
+
+	if attributes.Type() != pcommon.ValueTypeMap {
+		return
+	}
+
+	origAttr := rlogs.Resource().Attributes()
+	merged := MergeRawMaps(origAttr.AsRaw(), attributes.Map().AsRaw())
+	_ = origAttr.FromRaw(merged)
 }
 
 // MergeRawMaps merges n maps with a later map's keys overriding earlier maps. (copied to avoid dep hell)

processor/loghouseprocessor/processor_test.go

@@ -347,4 +347,28 @@ func Test_promoteResourceAttrs(t *testing.T) {
 
 	})
 
+	t.Run("not a map", func(t *testing.T) {
+		rl := plog.NewResourceLogs()
+		_ = rl.Resource().Attributes().FromRaw(map[string]any{"r1": "original"})
+		l1 := plog.NewLogRecord()
+		_ = l1.Attributes().FromRaw(map[string]any{"resource": "r2"})
+
+		promoteResourceAttrs(&l1, &rl)
+
+		_, ok := rl.Resource().Attributes().Get("r2")
+		assert.False(t, ok)
+	})
+
+	t.Run("empty resources", func(t *testing.T) {
+		rl := plog.NewResourceLogs()
+		l1 := plog.NewLogRecord()
+		_ = l1.Attributes().FromRaw(map[string]any{"resource": map[string]any{"r1": "v1"}})
+
+		promoteResourceAttrs(&l1, &rl)
+
+		v1, ok := rl.Resource().Attributes().Get("r1")
+		assert.True(t, ok)
+		assert.Equal(t, "v1", v1.Str())
+	})
+
 }