chore(server): unexport and rename method

internal/server/worker.go

@@ -185,7 +185,7 @@ func (w *Worker) clone() *Worker {
 // set over the API as well as the tags reported by the worker itself. This
 // function is guaranteed to return a non-nil map.
 func (w *Worker) CanonicalTags(opt ...Option) map[string][]string {
-	return DeduplicateTags(&w.ApiTags, &w.ConfigTags)
+	return compactTags(&w.ApiTags, &w.ConfigTags)
 }
 
 // GetConfigTags returns the tags for this worker which has been set through

internal/server/worker_tag.go

@@ -29,17 +29,17 @@ func (t *Tags) clone() Tags {
 	return newTags
 }
 
-// DeduplicateTags takes a list of Tags and returns a map of deduplicated tags
-func DeduplicateTags(t ...*Tags) map[string][]string {
-	dedupedTags := make(map[Tag]struct{})
+// compactTags takes a list of Tags and returns a map of deduplicated, compated tags
+func compactTags(t ...*Tags) map[string][]string {
+	compactedTags := make(map[Tag]struct{})
 	for _, tags := range t {
 		for _, tag := range *tags {
-			dedupedTags[*tag] = struct{}{}
+			compactedTags[*tag] = struct{}{}
 		}
 	}
 
 	tags := make(map[string][]string)
-	for t := range dedupedTags {
+	for t := range compactedTags {
 		tags[t.Key] = append(tags[t.Key], t.Value)
 	}
 	return tags

internal/server/worker_tag_test.go

@@ -20,7 +20,7 @@ func TestDeduplicateTags(t *testing.T) {
 			{Key: fmt.Sprintf("key-%d", pippo), Value: fmt.Sprintf("another-unique-%d", pippo)},
 		})
 	}
-	gotMap := DeduplicateTags(tags...)
+	gotMap := compactTags(tags...)
 	assert.Len(t, gotMap, 6)
 	assert.ElementsMatch(t, gotMap["key"], []string{"shared", "unique-0", "unique-1", "unique-2", "unique-3", "unique-4"})
 	assert.ElementsMatch(t, gotMap["key-0"], []string{"another-unique-0"})