Merge pull request #164 from overmindtech/fix-copy-include-tags

Added tags to copied item
overmindtech · Oct 31, 2023 · 4b01e46 · 4b01e46
2 parents 3edbba5 + 17dc63a
commit 4b01e46
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/items.go b/items.go
@@ -137,6 +137,14 @@ func (i *Item) Copy(dest *Item) {
 	} else {
 		dest.Health = i.Health.Enum()
 	}
+
+	if i.Tags != nil {
+		dest.Tags = make(map[string]string)
+
+		for k, v := range i.Tags {
+			dest.Tags[k] = v
+		}
+	}
 }
 
 // Hash Returns a 12 character hash for the item. This is likely but not

diff --git a/items_test.go b/items_test.go
@@ -253,6 +253,9 @@ func TestCopy(t *testing.T) {
 				SourceDurationPerItem: durationpb.New(10 * time.Millisecond),
 			},
 			Health: Health_HEALTH_ERROR.Enum(),
+			Tags: map[string]string{
+				"foo": "bar",
+			},
 		}
 
 		itemB := Item{}
@@ -373,6 +376,12 @@ func AssertItemsEqual(itemA *Item, itemB *Item, t *testing.T) {
 		}
 	}
 
+	for k, v := range itemA.Tags {
+		if itemB.Tags[k] != v {
+			t.Errorf("Tags[%v] did not match", k)
+		}
+	}
+
 	if itemA.Health == nil {
 		if itemB.Health != nil {
 			t.Errorf("mismatched health nil and %v", *itemB.Health)