From 338192f13d5fd4bcc702fdbcdd70abf8bcf5ab28 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E4=BA=8C=E8=B4=A7=E6=9C=BA=E5=99=A8=E4=BA=BA?=
 <smith3816@gmail.com>
Date: Tue, 13 May 2025 11:54:44 +0800
Subject: [PATCH] refactor: move onChange event up

---
 src/Step.tsx         |  7 ++-----
 tests/index.test.tsx | 10 +++++-----
 2 files changed, 7 insertions(+), 10 deletions(-)

diff --git a/src/Step.tsx b/src/Step.tsx
index 966ce6d..85606d5 100644
--- a/src/Step.tsx
+++ b/src/Step.tsx
@@ -128,11 +128,7 @@ export default function Step(props: StepProps) {
   );
 
   const wrapperNode = (
-    <div
-      className={cls(`${itemCls}-wrapper`, classNames.itemWrapper)}
-      style={styles.itemWrapper}
-      {...accessibilityProps}
-    >
+    <div className={cls(`${itemCls}-wrapper`, classNames.itemWrapper)} style={styles.itemWrapper}>
       <div className={cls(`${itemCls}-icon`, classNames.itemIcon)} style={styles.itemIcon}>
         {iconRender?.(renderInfo)}
       </div>
@@ -170,6 +166,7 @@ export default function Step(props: StepProps) {
   let stepNode: React.ReactNode = (
     <div
       {...restItemProps}
+      {...accessibilityProps}
       className={classString}
       style={{
         ...styles.item,
diff --git a/tests/index.test.tsx b/tests/index.test.tsx
index f0a2d03..0bff18b 100644
--- a/tests/index.test.tsx
+++ b/tests/index.test.tsx
@@ -210,9 +210,9 @@ describe('Steps', () => {
         ]}
       />,
     );
-    const items = container.querySelectorAll('.rc-steps-item-wrapper');
+    const items = container.querySelectorAll('.rc-steps-item');
     fireEvent.click(items[1]);
-    expect(onChange).toBeCalledWith(1);
+    expect(onChange).toHaveBeenCalledTimes(1);
   });
 
   it('items out of render function', () => {
@@ -233,7 +233,7 @@ describe('Steps', () => {
       <Steps current={current} onChange={onChange} items={items} key={current} />,
     );
 
-    const step = container.querySelectorAll('.rc-steps-item-wrapper')[1];
+    const step = container.querySelectorAll('.rc-steps-item')[1];
     fireEvent.click(step);
     rerender(<Steps current={current} onChange={onChange} items={items} key={current} />);
     expect(container.querySelectorAll('.rc-steps-item')[1].classList).toContain(
@@ -265,7 +265,7 @@ describe('Steps', () => {
       />,
     );
 
-    const btn = container.querySelectorAll('.rc-steps-item-wrapper')[0];
+    const btn = container.querySelectorAll('.rc-steps-item')[0];
     fireEvent.click(btn);
     expect(onClick).toHaveBeenCalled();
   });
@@ -276,7 +276,7 @@ describe('Steps', () => {
       <Steps onChange={onChange} items={[{}, {}, { disabled: true }]} />,
     );
 
-    const items = container.querySelectorAll('.rc-steps-item-wrapper');
+    const items = container.querySelectorAll('.rc-steps-item');
     fireEvent.click(items[2]);
     expect(onChange).not.toBeCalled();
   });