fixing onFieldUpdate and unskipping tests (#32604)

e2e/test/scenarios/admin/datamodel/field.cy.spec.js

@@ -17,8 +17,9 @@ import { SAMPLE_DATABASE } from "e2e/support/cypress_sample_database";
 const { ORDERS, ORDERS_ID } = SAMPLE_DATABASE;
 
 // [quarantine] - intermittently failing, possibly due to a "flickering" element (re-rendering)
-describe.skip("scenarios > admin > datamodel > field", () => {
+describe("scenarios > admin > datamodel > field", () => {
   beforeEach(() => {
+    restore();
     cy.signInAsAdmin();
 
     ["CREATED_AT", "PRODUCT_ID", "QUANTITY"].forEach(name => {
@@ -63,6 +64,19 @@ describe.skip("scenarios > admin > datamodel > field", () => {
     });
   });
 
+  describe("Formatting", () => {
+    it("should allow you to change field formatting", () => {
+      visitAlias("@ORDERS_QUANTITY_URL");
+      cy.findByRole("link", { name: "Formatting" }).click();
+      cy.findByLabelText("Style").click();
+      popover().findByText("Percent").click();
+      cy.wait("@fieldUpdate");
+      cy.findByRole("list", { name: "undo-list" })
+        .findByText("Updated Quantity")
+        .should("exist");
+    });
+  });
+
   describe("Visibility", () => {
     before(restore);
 

frontend/src/metabase/admin/datamodel/metadata/components/FieldFormattingSettings/FieldFormattingSettings.tsx

@@ -18,7 +18,7 @@ interface DispatchProps {
 type FieldFormattingSettingsProps = OwnProps & DispatchProps;
 
 const mapDispatchToProps: DispatchProps = {
-  onUpdateField: Fields.updateField,
+  onUpdateField: Fields.actions.updateField,
 };
 
 const FieldFormattingSettings = ({

frontend/src/metabase/containers/UndoListing.jsx

@@ -91,7 +91,7 @@ function UndoListingInner() {
   const undos = useSelector(state => state.undo);
 
   return (
-    <UndoList data-testid="undo-list">
+    <UndoList data-testid="undo-list" aria-label="undo-list">
       {undos.map(undo => (
         <UndoToast
           key={undo._domId}