From a5a6568da98b24660a5c022b32c0f9e460d92c8c Mon Sep 17 00:00:00 2001
From: Chandler Prall <chandler.prall@gmail.com>
Date: Wed, 18 Sep 2019 13:14:32 -0600
Subject: [PATCH 1/7] add data-test-subj support around non-custom actions

---
 src-docs/src/views/tables/actions/actions.js                 | 5 +++++
 .../__snapshots__/collapsed_item_actions.test.js.snap        | 1 +
 src/components/basic_table/collapsed_item_actions.js         | 2 ++
 src/components/basic_table/default_item_action.js            | 2 ++
 4 files changed, 10 insertions(+)

diff --git a/src-docs/src/views/tables/actions/actions.js b/src-docs/src/views/tables/actions/actions.js
index 55552bf2845..c3d9340e912 100644
--- a/src-docs/src/views/tables/actions/actions.js
+++ b/src-docs/src/views/tables/actions/actions.js
@@ -163,6 +163,7 @@ export class Table extends Component {
               description: 'Clone this user',
               icon: 'copy',
               onClick: this.cloneUser,
+              'data-test-subj': 'action-clone',
             },
             {
               name: 'Delete',
@@ -172,6 +173,7 @@ export class Table extends Component {
               type: 'icon',
               onClick: this.deleteUser,
               isPrimary: true,
+              'data-test-subj': 'action-delete',
             },
             {
               name: 'Edit',
@@ -180,6 +182,7 @@ export class Table extends Component {
               icon: 'pencil',
               type: 'icon',
               onClick: () => {},
+              'data-test-subj': 'action-edit',
             },
             {
               name: 'Share',
@@ -188,6 +191,7 @@ export class Table extends Component {
               icon: 'share',
               type: 'icon',
               onClick: () => {},
+              'data-test-subj': 'action-share',
             },
           ];
     } else {
@@ -212,6 +216,7 @@ export class Table extends Component {
               type: 'icon',
               href: 'https://elastic.co',
               target: '_blank',
+              'data-test-subj': 'action-outboundlink',
             },
           ];
     }
diff --git a/src/components/basic_table/__snapshots__/collapsed_item_actions.test.js.snap b/src/components/basic_table/__snapshots__/collapsed_item_actions.test.js.snap
index cba30915d37..b612b541827 100644
--- a/src/components/basic_table/__snapshots__/collapsed_item_actions.test.js.snap
+++ b/src/components/basic_table/__snapshots__/collapsed_item_actions.test.js.snap
@@ -14,6 +14,7 @@ exports[`CollapsedItemActions render 1`] = `
       <button
         aria-label="All actions"
         class="euiButtonIcon euiButtonIcon--text"
+        data-test-subj="euiCollapsedItemActions"
         type="button"
       >
         <svg
diff --git a/src/components/basic_table/collapsed_item_actions.js b/src/components/basic_table/collapsed_item_actions.js
index ae8468fab5c..8688a02f015 100644
--- a/src/components/basic_table/collapsed_item_actions.js
+++ b/src/components/basic_table/collapsed_item_actions.js
@@ -91,6 +91,7 @@ export class CollapsedItemActions extends Component {
             key={key}
             disabled={!enabled}
             icon={action.icon}
+            data-test-subj={action['data-test-subj']}
             onClick={this.onClickItem.bind(
               null,
               action.onClick.bind(null, item)
@@ -113,6 +114,7 @@ export class CollapsedItemActions extends Component {
             isDisabled={allDisabled}
             onClick={this.togglePopover.bind(this)}
             onFocus={onFocus}
+            data-test-subj="euiCollapsedItemActions"
           />
         )}
       </EuiI18n>
diff --git a/src/components/basic_table/default_item_action.js b/src/components/basic_table/default_item_action.js
index bc24226f1f5..d34285a9e26 100644
--- a/src/components/basic_table/default_item_action.js
+++ b/src/components/basic_table/default_item_action.js
@@ -44,6 +44,7 @@ export class DefaultItemAction extends Component {
           onClick={onClick}
           href={action.href}
           target={action.target}
+          data-test-subj={action['data-test-subj']}
         />
       );
     } else {
@@ -57,6 +58,7 @@ export class DefaultItemAction extends Component {
           onClick={onClick}
           href={action.href}
           target={action.target}
+          data-test-subj={action['data-test-subj']}
           flush="right">
           {action.name}
         </EuiButtonEmpty>

From edc57722104d1c274416e58f65398e12d19db9b9 Mon Sep 17 00:00:00 2001
From: Chandler Prall <chandler.prall@gmail.com>
Date: Thu, 19 Sep 2019 08:35:35 -0600
Subject: [PATCH 2/7] Add data-test-subj proptype to basic table action

---
 src/components/basic_table/basic_table.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/components/basic_table/basic_table.js b/src/components/basic_table/basic_table.js
index 22ae37c2b43..280f1536cea 100644
--- a/src/components/basic_table/basic_table.js
+++ b/src/components/basic_table/basic_table.js
@@ -86,6 +86,7 @@ const DefaultItemActionType = PropTypes.shape({
     PropTypes.oneOf(BUTTON_ICON_COLORS),
     PropTypes.func, // (item) => oneOf(ICON_BUTTON_COLORS)
   ]),
+  'data-test-subj': PropTypes.string,
 });
 
 const CustomItemActionType = PropTypes.shape({

From ad4e01f176c96a91ae75d3600255485ce89fc1b9 Mon Sep 17 00:00:00 2001
From: Chandler Prall <chandler.prall@gmail.com>
Date: Thu, 19 Sep 2019 08:39:29 -0600
Subject: [PATCH 3/7] Update data-test-subj prop documentation

---
 src-docs/src/views/tables/basic/props_info.js | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src-docs/src/views/tables/basic/props_info.js b/src-docs/src/views/tables/basic/props_info.js
index 834274aaceb..0dc3d730f1b 100644
--- a/src-docs/src/views/tables/basic/props_info.js
+++ b/src-docs/src/views/tables/basic/props_info.js
@@ -404,6 +404,12 @@ export const propsInfo = {
           required: false,
           type: { name: 'string (must be one of the supported button colors)' },
         },
+        'data-test-subj': {
+          description:
+            "Applies a data-test-subj attribute to the action's DOM node",
+          required: false,
+          type: { name: 'string' },
+        },
       },
     },
   },

From bf5936eb24f59dc5c11def699b8fcc539489462a Mon Sep 17 00:00:00 2001
From: Chandler Prall <chandler.prall@gmail.com>
Date: Thu, 19 Sep 2019 08:44:15 -0600
Subject: [PATCH 4/7] changelog

---
 CHANGELOG.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index b6f33042ca0..522187deace 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -10,6 +10,7 @@ No public interface changes since `14.2.0`.
 - Added missing `compressed` TS definitions to `EuiComboBox`, `EuiCheckboxGroup`, `EuiCheckbox`, `EuiFieldSearch`, `EuiRadioGroup`, `EuiSwitch` ([#2338](https://github.com/elastic/eui/pull/2338))
 - Added auto-margin between `EuiFormRow` and `EuiButton` ([#2338](https://github.com/elastic/eui/pull/2338))
 - Added border to `[readOnly]` inputs ([#2338](https://github.com/elastic/eui/pull/2338))
+- Added `data-test-subj` support for basic and in-memory tables' actions ([#2353](https://github.com/elastic/eui/pull/2353))
 
 **Bug fixes**
 

From d41382ac501112e8494c47aafc07d72edcdea0ba Mon Sep 17 00:00:00 2001
From: Chandler Prall <chandler.prall@gmail.com>
Date: Tue, 24 Sep 2019 08:52:00 -0600
Subject: [PATCH 5/7] Update
 src/components/basic_table/collapsed_item_actions.js

Co-Authored-By: Caroline Horn <549577+cchaos@users.noreply.github.com>
---
 src/components/basic_table/collapsed_item_actions.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/basic_table/collapsed_item_actions.js b/src/components/basic_table/collapsed_item_actions.js
index 8688a02f015..1bab6318c0f 100644
--- a/src/components/basic_table/collapsed_item_actions.js
+++ b/src/components/basic_table/collapsed_item_actions.js
@@ -114,7 +114,7 @@ export class CollapsedItemActions extends Component {
             isDisabled={allDisabled}
             onClick={this.togglePopover.bind(this)}
             onFocus={onFocus}
-            data-test-subj="euiCollapsedItemActions"
+            data-test-subj="euiCollapsedItemActionsButton"
           />
         )}
       </EuiI18n>

From bda5e48307df9b810a7754e4993c05e77543ff4e Mon Sep 17 00:00:00 2001
From: Chandler Prall <chandler.prall@gmail.com>
Date: Tue, 24 Sep 2019 08:53:12 -0600
Subject: [PATCH 6/7] Update snapshot

---
 .../__snapshots__/collapsed_item_actions.test.js.snap           | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/basic_table/__snapshots__/collapsed_item_actions.test.js.snap b/src/components/basic_table/__snapshots__/collapsed_item_actions.test.js.snap
index b612b541827..c461186c7c1 100644
--- a/src/components/basic_table/__snapshots__/collapsed_item_actions.test.js.snap
+++ b/src/components/basic_table/__snapshots__/collapsed_item_actions.test.js.snap
@@ -14,7 +14,7 @@ exports[`CollapsedItemActions render 1`] = `
       <button
         aria-label="All actions"
         class="euiButtonIcon euiButtonIcon--text"
-        data-test-subj="euiCollapsedItemActions"
+        data-test-subj="euiCollapsedItemActionsButton"
         type="button"
       >
         <svg

From fe732c6161363765123ede8481971f1a78a33c43 Mon Sep 17 00:00:00 2001
From: Chandler Prall <chandler.prall@gmail.com>
Date: Tue, 24 Sep 2019 09:25:11 -0600
Subject: [PATCH 7/7] changelog

---
 CHANGELOG.md | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 46049da3195..034c3b6bb15 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,5 +1,7 @@
 ## [`master`](https://github.com/elastic/eui/tree/master)
 
+- Added `data-test-subj` support for basic and in-memory tables' actions ([#2353](https://github.com/elastic/eui/pull/2353))
+
 **Bug fixes**
 
 - Fixed spacing of `EuiFormErrorText` to match `EuiFormHelpText` ([#2354](https://github.com/elastic/eui/pull/2354))
@@ -12,7 +14,6 @@
 - Added missing `compressed` TS definitions to `EuiComboBox`, `EuiCheckboxGroup`, `EuiCheckbox`, `EuiFieldSearch`, `EuiRadioGroup`, `EuiSwitch` ([#2338](https://github.com/elastic/eui/pull/2338))
 - Added auto-margin between `EuiFormRow` and `EuiButton` ([#2338](https://github.com/elastic/eui/pull/2338))
 - Added border to `[readOnly]` inputs ([#2338](https://github.com/elastic/eui/pull/2338))
-- Added `data-test-subj` support for basic and in-memory tables' actions ([#2353](https://github.com/elastic/eui/pull/2353))
 
 **Bug fixes**