diff --git a/datahub-graphql-core/src/main/java/com/linkedin/datahub/graphql/resolvers/tag/SetTagColorResolver.java b/datahub-graphql-core/src/main/java/com/linkedin/datahub/graphql/resolvers/tag/SetTagColorResolver.java index 9a8e0fc364..a986e77331 100644 --- a/datahub-graphql-core/src/main/java/com/linkedin/datahub/graphql/resolvers/tag/SetTagColorResolver.java +++ b/datahub-graphql-core/src/main/java/com/linkedin/datahub/graphql/resolvers/tag/SetTagColorResolver.java @@ -67,7 +67,7 @@ public class SetTagColorResolver implements DataFetcher mockService = getMockEntityService(); @@ -101,9 +101,21 @@ public class SetTagColorResolverTest { Mockito.when(mockEnv.getArgument(Mockito.eq("urn"))).thenReturn(TEST_ENTITY_URN); Mockito.when(mockEnv.getArgument(Mockito.eq("colorHex"))).thenReturn(TEST_COLOR_HEX); Mockito.when(mockEnv.getContext()).thenReturn(mockContext); - assertThrows(CompletionException.class, () -> resolver.get(mockEnv).join()); - Mockito.verify(mockClient, Mockito.times(0)).ingestProposal(any(), Mockito.any(), anyBoolean()); + boolean result = resolver.get(mockEnv).get(); + + assertTrue(result); + + final TagProperties newTagProperties = + new TagProperties().setName("test-tag").setColorHex(TEST_COLOR_HEX); + final MetadataChangeProposal proposal = + MutationUtils.buildMetadataChangeProposalWithUrn( + UrnUtils.getUrn(TEST_ENTITY_URN), TAG_PROPERTIES_ASPECT_NAME, newTagProperties); + + verifyIngestProposal(mockClient, 1, proposal); + + Mockito.verify(mockService, Mockito.times(1)) + .exists(any(), Mockito.eq(Urn.createFromString(TEST_ENTITY_URN)), eq(true)); } @Test diff --git a/datahub-web-react/src/app/tags/EmptyTags.tsx b/datahub-web-react/src/app/tags/EmptyTags.tsx index 7d2f70e04c..92052be983 100644 --- a/datahub-web-react/src/app/tags/EmptyTags.tsx +++ b/datahub-web-react/src/app/tags/EmptyTags.tsx @@ -32,7 +32,7 @@ const EmptyTags = ({ isEmptySearch }: Props) => {
{!isEmptySearch && ( - Tags can be used to organize and categorize data assets across the platform. + Tags can be used to organize data assets in DataHub. )}