#1908 Added table toolbar for properties tables (#1922)

Signed-off-by: Neha Gokhale <[email protected]> Co-authored-by: Carita <[email protected]>
elyra-ai · May 23, 2024 · 374f6cd · 374f6cd
1 parent 22d6452
commit 374f6cd
Show file tree

Hide file tree

Showing 45 changed files with 2,052 additions and 1,289 deletions.
diff --git a/canvas_modules/common-canvas/__tests__/common-properties/common-properties-test.js b/canvas_modules/common-canvas/__tests__/common-properties/common-properties-test.js
@@ -195,10 +195,11 @@ describe("CommonProperties works correctly in flyout", () => {
 		// make some changes
 		tableUtils.selectCheckboxes(commonProperties, [0]);
 
-		// ensure remove button is enabled and click it
-		const enabledRemoveColumnButton = wrapper.find("button.properties-remove-fields-button");
-		expect(enabledRemoveColumnButton).to.have.length(1);
-		enabledRemoveColumnButton.simulate("click");
+		// ensure table toolbar has delete and click it
+		let tableToolbar = wrapper.find("div.properties-table-toolbar");
+		let deleteButton = tableToolbar.find("button.properties-action-delete");
+		expect(deleteButton).to.have.length(1);
+		deleteButton.simulate("click");
 
 		// save again: should save changes
 		commonProperties.simulate("blur");
@@ -212,7 +213,9 @@ describe("CommonProperties works correctly in flyout", () => {
 
 		// make more changes
 		tableUtils.selectCheckboxes(commonProperties, [0]);
-		enabledRemoveColumnButton.simulate("click");
+		tableToolbar = wrapper.find("div.properties-table-toolbar");
+		deleteButton = tableToolbar.find("button.properties-action-delete");
+		deleteButton.simulate("click");
 
 		// save again, should trigger a save
 		commonProperties.simulate("blur");
@@ -228,10 +231,11 @@ describe("CommonProperties works correctly in flyout", () => {
 		// make some changes
 		tableUtils.selectCheckboxes(wrapper, [0]);
 
-		// ensure remove button is enabled and click it
-		const enabledRemoveColumnButton = wrapper.find("button.properties-remove-fields-button");
-		expect(enabledRemoveColumnButton).to.have.length(1);
-		enabledRemoveColumnButton.simulate("click");
+		// ensure table toolbar has delete and click it
+		const tableToolbar = wrapper.find("div.properties-table-toolbar");
+		const deleteButton = tableToolbar.find("button.properties-action-delete");
+		expect(deleteButton).to.have.length(1);
+		deleteButton.simulate("click");
 
 		// save again: should save changes
 		wrapper.find("aside.properties-right-flyout")