Skip to content

Commit 38a7d92

Browse files
committed
fix(ModalForm): Connect form.onSuccess to props.onSuccess
1 parent ebb59c9 commit 38a7d92

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

src/components/ModalForm.spec.tsx

+3
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ describe('default behaviour', () => {
1919

2020
describe('wrapped component behaviour', () => {
2121
let _okHandler;
22+
const onSuccess = jest.fn();
2223
const setWorking = jest.fn();
2324
const setVisible = jest.fn();
2425
const setOkHandler = jest.fn(handler => {
@@ -31,6 +32,7 @@ describe('wrapped component behaviour', () => {
3132
setWorking={setWorking}
3233
setVisible={setVisible}
3334
setOkHandler={setOkHandler}
35+
onSuccess={onSuccess}
3436
/>
3537
);
3638

@@ -61,5 +63,6 @@ describe('wrapped component behaviour', () => {
6163
it('hides the form when submission succeeded', () => {
6264
root.props().onSuccess();
6365
expect(setVisible).toHaveBeenCalledWith(false);
66+
expect(onSuccess).toHaveBeenCalled();
6467
});
6568
});

src/components/ModalForm.tsx

+5-1
Original file line numberDiff line numberDiff line change
@@ -59,8 +59,12 @@ export class ModalFormComponent extends React.Component<Props & InjectedProps> {
5959
});
6060
}
6161

62-
onSuccess = () => {
62+
onSuccess = (resource: any) => {
6363
this.props.setVisible(false);
64+
65+
if (this.props.onSuccess) {
66+
this.props.onSuccess(resource);
67+
}
6468
}
6569
}
6670

0 commit comments

Comments
 (0)