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

Automated cherry pick of #3787 (#3801)

Browse source 
* MM-20625 Unit Tests

* Updated tests
This commit is contained in:
Mattermost Build 2020-01-13 16:40:12 +01:00 committed by Amy Blais
parent e5d1a08767
commit 5b149d7c96
2 changed files with 53 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
app/components/progressive_image/__snapshots__/progressive_image.test.js.snap
View file

@ -1,12 +1,6 @@
// Jest Snapshot v1, https://goo.gl/fbAQLP
exports[`ProgressiveImage should match snapshot when just a image loads 1`] = `
<View
style={Object {}}
/>
`;
exports[`ProgressiveImage should match snapshot when no thumbnail 1`] = `
exports[`ProgressiveImage should match snapshot 1`] = `
<View
style={Object {}}
/>

63
app/components/progressive_image/progressive_image.test.js
View file

@ -8,6 +8,8 @@ import Preferences from 'mattermost-redux/constants/preferences';
import ProgressiveImage from './progressive_image';
jest.useFakeTimers();
describe('ProgressiveImage', () => {
const baseProps = {
isBackgroundImage: false,
@ -22,22 +24,61 @@ describe('ProgressiveImage', () => {
tintDefaultSource: false,
};
test('should match snapshot when just a image loads', () => {
const wrapper = shallow(
<ProgressiveImage {...baseProps}/>
);
test('should match snapshot', () => {
const wrapper = shallow(<ProgressiveImage {...baseProps}/>);
expect(wrapper.getElement()).toMatchSnapshot();
});
test('should match snapshot when no thumbnail', () => {
const props = {
...baseProps,
thumbnailUri: null,
};
test('should load image', () => {
const wrapper = shallow(<ProgressiveImage {...baseProps}/>);
const instance = wrapper.instance();
jest.spyOn(instance, 'load');
// should load image on componentDidMount
instance.componentDidMount();
expect(instance.load).toHaveBeenCalledTimes(1);
// should not re-load image on componentDidUpdate with same props
instance.componentDidUpdate(baseProps);
expect(instance.load).toHaveBeenCalledTimes(1);
// should re-load image on componentDidUpdate when props changed
wrapper.setProps({filename: 'newImage'});
expect(instance.load).toHaveBeenCalledTimes(2);
wrapper.setProps({imageUri: 'https://images.com/new_image.png'});
expect(instance.load).toHaveBeenCalledTimes(3);
wrapper.setProps({thumbnailUri: 'https://images.com/new_image.png'});
expect(instance.load).toHaveBeenCalledTimes(4);
});
test('should set image when imageUri is set but not the thumbnailUri', () => {
const wrapper = shallow(
<ProgressiveImage {...props}/>
<ProgressiveImage
{...baseProps}
thumbnailUri={null}
/>
);
expect(wrapper.getElement()).toMatchSnapshot();
const instance = wrapper.instance();
jest.spyOn(instance, 'setImage');
instance.componentDidMount();
jest.runAllTimers();
expect(instance.setImage).toHaveBeenCalledTimes(1);
});
test('should set thumbnail when thumbnailUri is set', () => {
const wrapper = shallow(
<ProgressiveImage
{...baseProps}
imageUri={null}
/>
);
const instance = wrapper.instance();
jest.spyOn(instance, 'setThumbnail');
jest.runAllTimers();
expect(instance.setThumbnail).toHaveBeenCalledTimes(1);
});
});