From 1a98606dfcc87b3493732aa6b9d1eb62eff30526 Mon Sep 17 00:00:00 2001
From: Jorge <jorge.costa@developer.pt>
Date: Tue, 30 Oct 2018 20:04:36 +0000
Subject: [PATCH] Fix: Cover image tooltips for alignment are duplicated

---
 .../src/components/alignment-toolbar/index.js | 14 +++---
 .../test/__snapshots__/index.js.snap          | 31 +++++++++++--
 .../alignment-toolbar/test/index.js           | 43 +++++++++++++++++++
 3 files changed, 78 insertions(+), 10 deletions(-)

diff --git a/packages/editor/src/components/alignment-toolbar/index.js b/packages/editor/src/components/alignment-toolbar/index.js
index 5aa2d61b0c0830..e06ee34f4d7d6a 100644
--- a/packages/editor/src/components/alignment-toolbar/index.js
+++ b/packages/editor/src/components/alignment-toolbar/index.js
@@ -17,37 +17,37 @@ import { compose } from '@wordpress/compose';
  */
 import { withBlockEditContext } from '../block-edit/context';
 
-const ALIGNMENT_CONTROLS = [
+const DEFAULT_ALIGNMENT_CONTROLS = [
 	{
 		icon: 'editor-alignleft',
-		title: __( 'Align left' ),
+		title: __( 'Align text left' ),
 		align: 'left',
 	},
 	{
 		icon: 'editor-aligncenter',
-		title: __( 'Align center' ),
+		title: __( 'Align text center' ),
 		align: 'center',
 	},
 	{
 		icon: 'editor-alignright',
-		title: __( 'Align right' ),
+		title: __( 'Align text right' ),
 		align: 'right',
 	},
 ];
 
-export function AlignmentToolbar( { isCollapsed, value, onChange } ) {
+export function AlignmentToolbar( { isCollapsed, value, onChange, alignmentControls = DEFAULT_ALIGNMENT_CONTROLS } ) {
 	function applyOrUnset( align ) {
 		return () => onChange( value === align ? undefined : align );
 	}
 
-	const activeAlignment = find( ALIGNMENT_CONTROLS, ( control ) => control.align === value );
+	const activeAlignment = find( alignmentControls, ( control ) => control.align === value );
 
 	return (
 		<Toolbar
 			isCollapsed={ isCollapsed }
 			icon={ activeAlignment ? activeAlignment.icon : 'editor-alignleft' }
 			label={ __( 'Change Text Alignment' ) }
-			controls={ ALIGNMENT_CONTROLS.map( ( control ) => {
+			controls={ alignmentControls.map( ( control ) => {
 				const { align } = control;
 				const isActive = ( value === align );
 
diff --git a/packages/editor/src/components/alignment-toolbar/test/__snapshots__/index.js.snap b/packages/editor/src/components/alignment-toolbar/test/__snapshots__/index.js.snap
index 2308f4cda66ae1..34e440f5d76fef 100644
--- a/packages/editor/src/components/alignment-toolbar/test/__snapshots__/index.js.snap
+++ b/packages/editor/src/components/alignment-toolbar/test/__snapshots__/index.js.snap
@@ -1,5 +1,30 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`AlignmentToolbar should allow custom alignment controls to be specified 1`] = `
+<Toolbar
+  controls={
+    Array [
+      Object {
+        "align": "custom-left",
+        "icon": "editor-alignleft",
+        "isActive": false,
+        "onClick": [Function],
+        "title": "My custom left",
+      },
+      Object {
+        "align": "custom-right",
+        "icon": "editor-aligncenter",
+        "isActive": true,
+        "onClick": [Function],
+        "title": "My custom right",
+      },
+    ]
+  }
+  icon="editor-aligncenter"
+  label="Change Text Alignment"
+/>
+`;
+
 exports[`AlignmentToolbar should match snapshot 1`] = `
 <Toolbar
   controls={
@@ -9,21 +34,21 @@ exports[`AlignmentToolbar should match snapshot 1`] = `
         "icon": "editor-alignleft",
         "isActive": true,
         "onClick": [Function],
-        "title": "Align left",
+        "title": "Align text left",
       },
       Object {
         "align": "center",
         "icon": "editor-aligncenter",
         "isActive": false,
         "onClick": [Function],
-        "title": "Align center",
+        "title": "Align text center",
       },
       Object {
         "align": "right",
         "icon": "editor-alignright",
         "isActive": false,
         "onClick": [Function],
-        "title": "Align right",
+        "title": "Align text right",
       },
     ]
   }
diff --git a/packages/editor/src/components/alignment-toolbar/test/index.js b/packages/editor/src/components/alignment-toolbar/test/index.js
index 803a5a56e6a828..03662cfdabe812 100644
--- a/packages/editor/src/components/alignment-toolbar/test/index.js
+++ b/packages/editor/src/components/alignment-toolbar/test/index.js
@@ -41,4 +41,47 @@ describe( 'AlignmentToolbar', () => {
 		expect( onChangeSpy ).toHaveBeenCalledTimes( 1 );
 		expect( onChangeSpy ).toHaveBeenCalledWith( 'center' );
 	} );
+
+	test( 'should allow custom alignment controls to be specified', () => {
+		const wrapperCustomControls = shallow(
+			<AlignmentToolbar
+				value={ 'custom-right' }
+				onChange={ onChangeSpy }
+				alignmentControls={ [
+					{
+						icon: 'editor-alignleft',
+						title: 'My custom left',
+						align: 'custom-left',
+					},
+					{
+						icon: 'editor-aligncenter',
+						title: 'My custom right',
+						align: 'custom-right',
+					},
+				] }
+			/>
+		);
+		expect( wrapperCustomControls ).toMatchSnapshot();
+		const customControls = wrapperCustomControls.props().controls;
+		expect( customControls ).toHaveLength( 2 );
+
+		// should correctly call on change when right alignment is pressed (active alignment)
+		const rightControl = customControls.find(
+			( { align } ) => align === 'custom-right'
+		);
+		expect( rightControl.title ).toBe( 'My custom right' );
+		rightControl.onClick();
+		expect( onChangeSpy ).toHaveBeenCalledTimes( 1 );
+		expect( onChangeSpy ).toHaveBeenCalledWith( undefined );
+		onChangeSpy.mockClear();
+
+		// should correctly call on change when right alignment is pressed (inactive alignment)
+		const leftControl = customControls.find(
+			( { align } ) => align === 'custom-left'
+		);
+		expect( leftControl.title ).toBe( 'My custom left' );
+		leftControl.onClick();
+		expect( onChangeSpy ).toHaveBeenCalledTimes( 1 );
+		expect( onChangeSpy ).toHaveBeenCalledWith( 'custom-left' );
+	} );
 } );