Skip to content

Commit 74e4a00

Browse files
authored
test(Loader): rewrite test using react-testing-library (#819)
[ci skip]
1 parent fa1fd81 commit 74e4a00

File tree

2 files changed

+64
-54
lines changed

2 files changed

+64
-54
lines changed

packages/main/src/components/Loader/Loader.test.tsx

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,38 +1,38 @@
1+
import { render } from '@shared/tests';
12
import { createPassThroughPropsTest } from '@shared/tests/utils';
2-
import { mount } from 'enzyme';
33
import { Loader } from '@ui5/webcomponents-react/lib/Loader';
44
import { LoaderType } from '@ui5/webcomponents-react/lib/LoaderType';
55
import React from 'react';
66

77
describe('Loader', () => {
88
test('indeterminate', () => {
9-
const wrapper = mount(<Loader type={LoaderType.Indeterminate} />).render();
10-
expect(wrapper).toMatchSnapshot();
9+
const { asFragment } = render(<Loader type={LoaderType.Indeterminate} />);
10+
expect(asFragment()).toMatchSnapshot();
1111
});
1212

1313
test('determinate', () => {
14-
const wrapper = mount(<Loader type={LoaderType.Determinate} />).render();
15-
expect(wrapper).toMatchSnapshot();
14+
const { asFragment } = render(<Loader type={LoaderType.Determinate} />);
15+
expect(asFragment()).toMatchSnapshot();
1616
});
1717

1818
test('determinate with progress prop', () => {
19-
const wrapper = mount(<Loader type={LoaderType.Determinate} progress="42%" />).render();
20-
expect(wrapper).toMatchSnapshot();
19+
const { asFragment } = render(<Loader type={LoaderType.Determinate} progress="42%" />);
20+
expect(asFragment()).toMatchSnapshot();
2121
});
2222

2323
test('default', () => {
24-
const wrapper = mount(<Loader />).render();
25-
expect(wrapper).toMatchSnapshot();
24+
const { asFragment } = render(<Loader />);
25+
expect(asFragment()).toMatchSnapshot();
2626
});
2727

2828
test('with Custom Class Name', () => {
29-
const wrapper = mount(<Loader className="myTestClass" />).render();
30-
expect(wrapper).toMatchSnapshot();
29+
const { asFragment } = render(<Loader className="myTestClass" />);
30+
expect(asFragment()).toMatchSnapshot();
3131
});
3232

3333
test('with delay', () => {
34-
const wrapper = mount(<Loader delay={1000} />).render();
35-
expect(wrapper).toMatchSnapshot();
34+
const { asFragment } = render(<Loader delay={1000} />);
35+
expect(asFragment()).toMatchSnapshot();
3636
});
3737

3838
createPassThroughPropsTest(Loader);
Lines changed: 51 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -1,58 +1,68 @@
11
// Jest Snapshot v1, https://goo.gl/fbAQLP
22

33
exports[`Loader default 1`] = `
4-
<div
5-
aria-busy="true"
6-
class="Loader-loader-0 Loader-loaderIndeterminate-0"
7-
data-component-name="Loader"
8-
role="progressbar"
9-
style="background-size: 40%;"
10-
title="Please wait"
11-
/>
4+
<DocumentFragment>
5+
<div
6+
aria-busy="true"
7+
class="Loader-loader Loader-loaderIndeterminate"
8+
data-component-name="Loader"
9+
role="progressbar"
10+
style="background-size: 40%;"
11+
title="Please wait"
12+
/>
13+
</DocumentFragment>
1214
`;
1315

1416
exports[`Loader determinate 1`] = `
15-
<div
16-
aria-busy="true"
17-
class="Loader-loader-0 Loader-loaderDeterminate-0"
18-
data-component-name="Loader"
19-
role="progressbar"
20-
style="background-size: 0px;"
21-
title="Please wait"
22-
/>
17+
<DocumentFragment>
18+
<div
19+
aria-busy="true"
20+
class="Loader-loader Loader-loaderDeterminate"
21+
data-component-name="Loader"
22+
role="progressbar"
23+
style="background-size: 0px;"
24+
title="Please wait"
25+
/>
26+
</DocumentFragment>
2327
`;
2428

2529
exports[`Loader determinate with progress prop 1`] = `
26-
<div
27-
aria-busy="true"
28-
class="Loader-loader-0 Loader-loaderDeterminate-0"
29-
data-component-name="Loader"
30-
role="progressbar"
31-
style="background-size: 42%;"
32-
title="Please wait"
33-
/>
30+
<DocumentFragment>
31+
<div
32+
aria-busy="true"
33+
class="Loader-loader Loader-loaderDeterminate"
34+
data-component-name="Loader"
35+
role="progressbar"
36+
style="background-size: 42%;"
37+
title="Please wait"
38+
/>
39+
</DocumentFragment>
3440
`;
3541

3642
exports[`Loader indeterminate 1`] = `
37-
<div
38-
aria-busy="true"
39-
class="Loader-loader-0 Loader-loaderIndeterminate-0"
40-
data-component-name="Loader"
41-
role="progressbar"
42-
style="background-size: 40%;"
43-
title="Please wait"
44-
/>
43+
<DocumentFragment>
44+
<div
45+
aria-busy="true"
46+
class="Loader-loader Loader-loaderIndeterminate"
47+
data-component-name="Loader"
48+
role="progressbar"
49+
style="background-size: 40%;"
50+
title="Please wait"
51+
/>
52+
</DocumentFragment>
4553
`;
4654

4755
exports[`Loader with Custom Class Name 1`] = `
48-
<div
49-
aria-busy="true"
50-
class="Loader-loader-0 myTestClass Loader-loaderIndeterminate-0"
51-
data-component-name="Loader"
52-
role="progressbar"
53-
style="background-size: 40%;"
54-
title="Please wait"
55-
/>
56+
<DocumentFragment>
57+
<div
58+
aria-busy="true"
59+
class="Loader-loader myTestClass Loader-loaderIndeterminate"
60+
data-component-name="Loader"
61+
role="progressbar"
62+
style="background-size: 40%;"
63+
title="Please wait"
64+
/>
65+
</DocumentFragment>
5666
`;
5767

58-
exports[`Loader with delay 1`] = `null`;
68+
exports[`Loader with delay 1`] = `<DocumentFragment />`;

0 commit comments

Comments
 (0)