From b644d8d9a1f38ddd2e30082bfb4257df359100c0 Mon Sep 17 00:00:00 2001
From: Kai Hao <kevin830726@gmail.com>
Date: Fri, 28 Aug 2020 15:21:17 +0800
Subject: [PATCH 1/2] Remove redundant post type from search results

---
 .../block-editor/src/components/link-control/search-input.js  | 1 +
 .../block-editor/src/components/link-control/search-item.js   | 3 ++-
 .../src/components/link-control/search-results.js             | 4 ++++
 3 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/packages/block-editor/src/components/link-control/search-input.js b/packages/block-editor/src/components/link-control/search-input.js
index 4e5dd10cad2229..c8ae882ae1aba8 100644
--- a/packages/block-editor/src/components/link-control/search-input.js
+++ b/packages/block-editor/src/components/link-control/search-input.js
@@ -81,6 +81,7 @@ const LinkControlSearchInput = forwardRef(
 				withCreateSuggestion,
 				currentInputValue: value,
 				createSuggestionButtonText,
+				suggestionsQuery,
 				handleSuggestionClick: ( suggestion ) => {
 					if ( props.handleSuggestionClick ) {
 						props.handleSuggestionClick( suggestion );
diff --git a/packages/block-editor/src/components/link-control/search-item.js b/packages/block-editor/src/components/link-control/search-item.js
index 940121c0e0fe4c..804e5c95f05afa 100644
--- a/packages/block-editor/src/components/link-control/search-item.js
+++ b/packages/block-editor/src/components/link-control/search-item.js
@@ -18,6 +18,7 @@ export const LinkControlSearchItem = ( {
 	onClick,
 	isURL = false,
 	searchTerm = '',
+	shouldShowType = false,
 } ) => {
 	return (
 		<Button
@@ -54,7 +55,7 @@ export const LinkControlSearchItem = ( {
 					{ isURL && __( 'Press ENTER to add this link' ) }
 				</span>
 			</span>
-			{ suggestion.type && (
+			{ shouldShowType && suggestion.type && (
 				<span className="block-editor-link-control__search-item-type">
 					{ /* Rename 'post_tag' to 'tag'. Ideally, the API would return the localised CPT or taxonomy label. */ }
 					{ suggestion.type === 'post_tag' ? 'tag' : suggestion.type }
diff --git a/packages/block-editor/src/components/link-control/search-results.js b/packages/block-editor/src/components/link-control/search-results.js
index 2eac1f94ea0af1..cc24b81f18b179 100644
--- a/packages/block-editor/src/components/link-control/search-results.js
+++ b/packages/block-editor/src/components/link-control/search-results.js
@@ -29,6 +29,7 @@ export default function LinkControlSearchResults( {
 	isLoading,
 	isInitialSuggestions,
 	createSuggestionButtonText,
+	suggestionsQuery,
 } ) {
 	const resultsListClasses = classnames(
 		'block-editor-link-control__search-results',
@@ -45,6 +46,8 @@ export default function LinkControlSearchResults( {
 		withCreateSuggestion &&
 		! isSingleDirectEntryResult &&
 		! isInitialSuggestions;
+	// If the query has a specified type, then we can skip showing them in the result. See #24839.
+	const shouldShowSuggestionsTypes = ! suggestionsQuery?.type;
 
 	// According to guidelines aria-label should be added if the label
 	// itself is not visible.
@@ -128,6 +131,7 @@ export default function LinkControlSearchResults( {
 								suggestion.type.toLowerCase()
 							) }
 							searchTerm={ currentInputValue }
+							shouldShowType={ shouldShowSuggestionsTypes }
 						/>
 					);
 				} ) }

From c182e707630fffb8d980513936268d0f61a832d6 Mon Sep 17 00:00:00 2001
From: Kai Hao <kevin830726@gmail.com>
Date: Fri, 28 Aug 2020 16:19:08 +0800
Subject: [PATCH 2/2] Add tests

---
 .../link-control/test/fixtures/index.js       |  8 +-
 .../src/components/link-control/test/index.js | 75 ++++++++++++++++++-
 2 files changed, 75 insertions(+), 8 deletions(-)

diff --git a/packages/block-editor/src/components/link-control/test/fixtures/index.js b/packages/block-editor/src/components/link-control/test/fixtures/index.js
index 173d7963a8e851..6983279b2a55eb 100644
--- a/packages/block-editor/src/components/link-control/test/fixtures/index.js
+++ b/packages/block-editor/src/components/link-control/test/fixtures/index.js
@@ -7,21 +7,21 @@ export const fauxEntitySuggestions = [
 	{
 		id: uniqueId(),
 		title: 'Hello Page',
-		type: 'Page',
+		type: 'page',
 		info: '2 days ago',
 		url: `?p=${ uniqueId() }`,
 	},
 	{
 		id: uniqueId(),
 		title: 'Hello Post',
-		type: 'Post',
+		type: 'post',
 		info: '19 days ago',
 		url: `?p=${ uniqueId() }`,
 	},
 	{
 		id: uniqueId(),
 		title: 'Hello Another One',
-		type: 'Page',
+		type: 'page',
 		info: '19 days ago',
 		url: `?p=${ uniqueId() }`,
 	},
@@ -29,7 +29,7 @@ export const fauxEntitySuggestions = [
 		id: uniqueId(),
 		title:
 			'This is another Post with a much longer title just to be really annoying and to try and break the UI',
-		type: 'Post',
+		type: 'post',
 		info: '1 month ago',
 		url: `?p=${ uniqueId() }`,
 	},
diff --git a/packages/block-editor/src/components/link-control/test/index.js b/packages/block-editor/src/components/link-control/test/index.js
index 0dfaaef70c690b..344da8f08d29c4 100644
--- a/packages/block-editor/src/components/link-control/test/index.js
+++ b/packages/block-editor/src/components/link-control/test/index.js
@@ -3,6 +3,7 @@
  */
 import { render, unmountComponentAtNode } from 'react-dom';
 import { act, Simulate } from 'react-dom/test-utils';
+import { queryByText, queryByRole } from '@testing-library/react';
 import { first, last, nth, uniqueId } from 'lodash';
 /**
  * WordPress dependencies
@@ -1217,10 +1218,9 @@ describe( 'Selecting links', () => {
 		expect( currentLinkHTML ).toEqual(
 			expect.stringContaining( selectedLink.title )
 		);
-		expect( currentLinkHTML ).toEqual(
-			expect.stringContaining( selectedLink.type )
-		);
-		expect( currentLinkHTML ).toEqual( expect.stringContaining( 'Edit' ) );
+		expect(
+			queryByRole( currentLink, 'button', { name: 'Edit' } )
+		).toBeTruthy();
 		expect( currentLinkAnchor ).not.toBeNull();
 	} );
 
@@ -1686,3 +1686,70 @@ describe( 'Addition Settings UI', () => {
 		expect( settingControlsInputs[ 1 ].checked ).toEqual( true );
 	} );
 } );
+
+describe( 'Post types', () => {
+	it( 'should display post type in search results of link', async () => {
+		const searchTerm = 'Hello world';
+
+		act( () => {
+			render( <LinkControl />, container );
+		} );
+
+		// Search Input UI
+		const searchInput = getURLInput();
+
+		// Simulate searching for a term
+		act( () => {
+			Simulate.change( searchInput, { target: { value: searchTerm } } );
+		} );
+
+		// fetchFauxEntitySuggestions resolves on next "tick" of event loop
+		await eventLoopTick();
+
+		const searchResultElements = getSearchResults();
+
+		searchResultElements.forEach( ( resultItem, index ) => {
+			expect(
+				queryByText( resultItem, fauxEntitySuggestions[ index ].type )
+			).toBeTruthy();
+		} );
+	} );
+
+	it.each( [ 'page', 'post', 'tag', 'post_tag', 'category' ] )(
+		'should NOT display post type in search results of %s',
+		async ( postType ) => {
+			const searchTerm = 'Hello world';
+
+			act( () => {
+				render(
+					<LinkControl suggestionsQuery={ { type: postType } } />,
+					container
+				);
+			} );
+
+			// Search Input UI
+			const searchInput = getURLInput();
+
+			// Simulate searching for a term
+			act( () => {
+				Simulate.change( searchInput, {
+					target: { value: searchTerm },
+				} );
+			} );
+
+			// fetchFauxEntitySuggestions resolves on next "tick" of event loop
+			await eventLoopTick();
+
+			const searchResultElements = getSearchResults();
+
+			searchResultElements.forEach( ( resultItem, index ) => {
+				expect(
+					queryByText(
+						resultItem,
+						fauxEntitySuggestions[ index ].type
+					)
+				).toBeFalsy();
+			} );
+		}
+	);
+} );