From e751896e6d97c33c12ad87ec5c7fb052b051730e Mon Sep 17 00:00:00 2001
From: Ade Attwood <adrian.attwood@marketdojo.com>
Date: Sat, 25 Mar 2023 09:51:00 +0000
Subject: [PATCH] feat: pass the changed attribute though the onChange form
 callback

Ref: #143
---
 src/form.tsx             | 4 ++--
 tests/on-change.spec.tsx | 1 +
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/form.tsx b/src/form.tsx
index 88ac851..53080a1 100644
--- a/src/form.tsx
+++ b/src/form.tsx
@@ -61,7 +61,7 @@ export interface FormProps<T extends {}> {
   /**
    * Callback that is called whenever an attribute is changed
    */
-  onChange?: (context: ReturnType<Form<T>["getContextValue"]>) => void;
+  onChange?: (context: ReturnType<Form<T>["getContextValue"]>, attribute: string) => void;
   /**
    *
    * TODO(ade): sort out prop with children
@@ -193,7 +193,7 @@ export class Form<T extends Record<string, any>> extends React.Component<FormPro
     }
 
     this.setState({ formState, errors, status });
-    this.props.onChange?.(this.getContextValue());
+    this.props.onChange?.(this.getContextValue(), attribute);
   };
 
   /**
diff --git a/tests/on-change.spec.tsx b/tests/on-change.spec.tsx
index f470016..285cfdf 100644
--- a/tests/on-change.spec.tsx
+++ b/tests/on-change.spec.tsx
@@ -22,4 +22,5 @@ it("will call onChange when a attribute is changed", async () => {
   });
 
   expect(onChange).toHaveBeenCalled();
+  expect(onChange).toBeCalledWith(expect.objectContaining({ formState: { "test-input": "A" } }), "test-input");
 });