Merge pull request #141 from newrelic/fix/entity-search-by-tags

fix(entities): tags filter needs to use type TagValue in graphql query
newrelic · Feb 20, 2020 · 8a7a8ea · 8a7a8ea
2 parents c5c220c + 2fd1f08
commit 8a7a8ea
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 1 deletion.
diff --git a/pkg/entities/entities.go b/pkg/entities/entities.go
@@ -29,7 +29,7 @@ type SearchEntitiesParams struct {
 	InfrastructureIntegrationType string                  `json:"infrastructureIntegrationType,omitempty"`
 	Name                          string                  `json:"name,omitempty"`
 	Reporting                     *bool                   `json:"reporting,omitempty"`
-	Tags                          []Tag                   `json:"tags,omitempty"`
+	Tags                          *TagValue               `json:"tags,omitempty"`
 	Type                          EntityType              `json:"type,omitempty"`
 }
 

diff --git a/pkg/entities/entities_integration_test.go b/pkg/entities/entities_integration_test.go
@@ -18,11 +18,31 @@ func TestIntegrationSearchEntities(t *testing.T) {
 	params := SearchEntitiesParams{
 		Name: "Dummy App",
 	}
+
 	actual, err := client.SearchEntities(params)
 
 	require.NoError(t, err)
 	require.Greater(t, len(actual), 0)
 }
+
+func TestIntegrationSearchEntitiesByTags(t *testing.T) {
+	t.Parallel()
+
+	client := newIntegrationTestClient(t)
+
+	params := SearchEntitiesParams{
+		Tags: &TagValue{
+			Key:   "language",
+			Value: "nodejs",
+		},
+	}
+
+	actual, err := client.SearchEntities(params)
+
+	require.NoError(t, err)
+	require.NotNil(t, actual)
+}
+
 func TestIntegrationGetEntities(t *testing.T) {
 	t.Parallel()