toki/mattermost-mobile
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Automated cherry pick of #3116 (#3120)

Browse source 
* Fix delete the root post on a thread to properly close the thread screen

* Fix unit test
This commit is contained in:
Mattermost Build 2019-08-14 18:34:18 +02:00 committed by Miguel Alatzar
parent f8efcf73b7
commit 90dba40a92
2 changed files with 10 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
app/screens/post_options/post_options.js
View file

@ -349,9 +349,10 @@ export default class PostOptions extends PureComponent {
text: formatMessage({id: 'post_info.del', defaultMessage: 'Delete'}),
style: 'destructive',
onPress: () => {
actions.deletePost(post);
actions.removePost(post);
this.closeWithAnimation();
this.closeWithAnimation(() => {
actions.deletePost(post);
actions.removePost(post);
});
},
}]
);

6
app/screens/post_options/post_options.test.js
View file

@ -110,8 +110,14 @@ describe('PostOptions', () => {
expect(Alert.alert).toBeCalled();
// Trigger on press of Delete in the Alert
const closeWithAnimation = jest.spyOn(wrapper.instance(), 'closeWithAnimation');
Alert.alert.mock.calls[0][2][1].onPress();
expect(closeWithAnimation).toBeCalled();
// get the callback that gets called by closeWithAnimation
const callback = closeWithAnimation.mock.calls[0][0];
callback();
expect(actions.deletePost).toBeCalled();
expect(actions.removePost).toBeCalled();
});