Skip to content

Commit b1823a8

Browse files
committed
fix: broken tests
1 parent ef3a2a7 commit b1823a8

File tree

3 files changed

+24
-32
lines changed

3 files changed

+24
-32
lines changed

package/src/components/Message/MessageSimple/__tests__/MessageStatus.test.js

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -72,26 +72,32 @@ describe('MessageStatus', () => {
7272

7373
it('should render message status with read by container', async () => {
7474
const user = generateUser();
75-
const message = generateMessage({ readBy: 2, user });
75+
const message = generateMessage({ user });
76+
const readBy = 2;
7677

7778
const { getByTestId, getByText, rerender, toJSON } = renderMessageStatus({
7879
lastReceivedId: message.id,
7980
message,
81+
readBy,
8082
});
8183

8284
await waitFor(() => {
8385
expect(getByTestId('read-by-container')).toBeTruthy();
84-
expect(getByText((message.readBy - 1).toString())).toBeTruthy();
86+
expect(getByText((readBy - 1).toString())).toBeTruthy();
8587
});
8688

8789
const staticUser = generateStaticUser(0);
88-
const staticMessage = generateMessage({ readBy: 2, staticUser });
90+
const staticMessage = generateMessage({ readBy, user: staticUser });
8991

9092
rerender(
9193
<ChannelsStateProvider>
9294
<Chat client={chatClient} i18nInstance={i18nInstance}>
9395
<Channel channel={channel}>
94-
<MessageStatus lastReceivedId={staticMessage.id} message={staticMessage} />
96+
<MessageStatus
97+
lastReceivedId={staticMessage.id}
98+
message={staticMessage}
99+
readBy={readBy}
100+
/>
95101
</Channel>
96102
</Chat>
97103
</ChannelsStateProvider>,
@@ -100,7 +106,7 @@ describe('MessageStatus', () => {
100106
await waitFor(() => {
101107
expect(toJSON()).toMatchSnapshot();
102108
expect(getByTestId('read-by-container')).toBeTruthy();
103-
expect(getByText((staticMessage.readBy - 1).toString())).toBeTruthy();
109+
expect(getByText((readBy - 1).toString())).toBeTruthy();
104110
});
105111
});
106112

package/src/components/Thread/__tests__/__snapshots__/Thread.test.js.snap

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -51,20 +51,15 @@ exports[`Thread should match thread snapshot 1`] = `
5151
{
5252
"attachments": [],
5353
"created_at": 2020-05-05T14:50:00.000Z,
54-
"dateSeparator": undefined,
5554
"deleted_at": null,
5655
"error": null,
57-
"groupStyles": [
58-
"single",
59-
],
6056
"html": "<p>regular</p>",
6157
"id": "38ef6f7c-3090-5759-a37f-ab0053aadb96",
6258
"message_text_updated_at": "2020-05-05T14:50:00.000Z",
6359
"parent_id": "b4612a73-fa2b-5787-bf71-1adc8f291a04",
6460
"pinned_at": null,
6561
"quoted_message": null,
6662
"reaction_groups": null,
67-
"readBy": false,
6863
"status": "received",
6964
"text": "Message6",
7065
"type": "regular",
@@ -83,20 +78,15 @@ exports[`Thread should match thread snapshot 1`] = `
8378
{
8479
"attachments": [],
8580
"created_at": 2020-05-05T14:50:00.000Z,
86-
"dateSeparator": undefined,
8781
"deleted_at": null,
8882
"error": null,
89-
"groupStyles": [
90-
"single",
91-
],
9283
"html": "<p>regular</p>",
9384
"id": "516efa25-5d29-5c9a-ad2d-4cc183e785bd",
9485
"message_text_updated_at": "2020-05-05T14:50:00.000Z",
9586
"parent_id": "b4612a73-fa2b-5787-bf71-1adc8f291a04",
9687
"pinned_at": null,
9788
"quoted_message": null,
9889
"reaction_groups": null,
99-
"readBy": false,
10090
"status": "received",
10191
"text": "Message5",
10292
"type": "regular",
@@ -115,20 +105,15 @@ exports[`Thread should match thread snapshot 1`] = `
115105
{
116106
"attachments": [],
117107
"created_at": 2020-05-05T14:50:00.000Z,
118-
"dateSeparator": 2020-05-05T14:50:00.000Z,
119108
"deleted_at": null,
120109
"error": null,
121-
"groupStyles": [
122-
"single",
123-
],
124110
"html": "<p>regular</p>",
125111
"id": "82a83b16-b611-527c-b3ac-765ef6220490",
126112
"message_text_updated_at": "2020-05-05T14:50:00.000Z",
127113
"parent_id": "b4612a73-fa2b-5787-bf71-1adc8f291a04",
128114
"pinned_at": null,
129115
"quoted_message": null,
130116
"reaction_groups": null,
131-
"readBy": false,
132117
"status": "received",
133118
"text": "Message4",
134119
"type": "regular",

package/src/contexts/messageInputContext/__tests__/useMessageDetailsForState.test.tsx

Lines changed: 13 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -13,19 +13,20 @@ import { generateUser } from '../../../mock-builders/generator/user';
1313
import { useMessageDetailsForState } from '../hooks/useMessageDetailsForState';
1414

1515
describe('useMessageDetailsForState', () => {
16-
it.each([
17-
[{ message: generateMessage({ text: 'Dummy text' }) }],
18-
[{ initialValue: '', message: generateMessage({ text: 'Dummy text' }) }],
19-
])('test state of useMessageDetailsForState when initialProps differ', () => {
20-
const { result } = renderHook(
21-
({ message }) => useMessageDetailsForState(message as unknown as LocalMessage),
22-
{
23-
initialProps: { message: generateMessage({ text: 'Dummy text' }) },
24-
},
25-
);
16+
const message = generateMessage({ text: 'Dummy text' });
17+
it.each([[{ message }, { initialValue: '', message }]])(
18+
'test state of useMessageDetailsForState when initialProps differ',
19+
() => {
20+
const { result } = renderHook(
21+
({ message }) => useMessageDetailsForState(message as unknown as LocalMessage),
22+
{
23+
initialProps: { message },
24+
},
25+
);
2626

27-
expect(result.current.text).toBe('');
28-
});
27+
expect(result.current.text).toBe(message.text);
28+
},
29+
);
2930

3031
it('showMoreOptions is true when initialValue and text is same', () => {
3132
const { result } = renderHook(

0 commit comments

Comments
 (0)