Twilio voice now uses first-class `Call` records as the source of truth for call state, instead of storing it on `conversation.additional_attributes` and `conversation.identifier`. Each call gets its own record, its own `voice_call` bubble matched by `call_sid`, and its own conference name keyed off `Call.id`. Multiple calls on the same conversation (for `lock_to_single_conversation` inboxes) now work correctly, and the conversation card stays in sync with the real latest message. Fixes https://linear.app/chatwoot/issue/PLA-121/lock-to-single-thread --------- Co-authored-by: Muhsin <12408980+muhsin-k@users.noreply.github.com>
135 lines
3.7 KiB
JavaScript
135 lines
3.7 KiB
JavaScript
import { mutations } from '../index';
|
|
import types from '../../../mutation-types';
|
|
|
|
describe('#mutations', () => {
|
|
describe('#UPDATE_MESSAGE_CALL_STATUS', () => {
|
|
it('does nothing if conversation is not found', () => {
|
|
const state = { allConversations: [] };
|
|
mutations[types.UPDATE_MESSAGE_CALL_STATUS](state, {
|
|
conversationId: 1,
|
|
callStatus: 'ringing',
|
|
callSid: 'CA123',
|
|
});
|
|
expect(state.allConversations).toEqual([]);
|
|
});
|
|
|
|
it('does nothing if no matching voice call message exists', () => {
|
|
const state = {
|
|
allConversations: [
|
|
{ id: 1, messages: [{ id: 1, content_type: 'text' }] },
|
|
],
|
|
};
|
|
mutations[types.UPDATE_MESSAGE_CALL_STATUS](state, {
|
|
conversationId: 1,
|
|
callStatus: 'ringing',
|
|
callSid: 'CA123',
|
|
});
|
|
expect(state.allConversations[0].messages[0]).toEqual({
|
|
id: 1,
|
|
content_type: 'text',
|
|
});
|
|
});
|
|
|
|
it('updates only the voice call message matching the given callSid', () => {
|
|
const state = {
|
|
allConversations: [
|
|
{
|
|
id: 1,
|
|
messages: [
|
|
{
|
|
id: 1,
|
|
content_type: 'voice_call',
|
|
call: { provider_call_id: 'CA111', status: 'ringing' },
|
|
},
|
|
{
|
|
id: 2,
|
|
content_type: 'voice_call',
|
|
call: { provider_call_id: 'CA222', status: 'ringing' },
|
|
},
|
|
],
|
|
},
|
|
],
|
|
};
|
|
mutations[types.UPDATE_MESSAGE_CALL_STATUS](state, {
|
|
conversationId: 1,
|
|
callStatus: 'in-progress',
|
|
callSid: 'CA111',
|
|
});
|
|
expect(state.allConversations[0].messages[0].call.status).toBe(
|
|
'in-progress'
|
|
);
|
|
expect(state.allConversations[0].messages[1].call.status).toBe('ringing');
|
|
});
|
|
|
|
it('preserves existing call fields when updating status', () => {
|
|
const state = {
|
|
allConversations: [
|
|
{
|
|
id: 1,
|
|
messages: [
|
|
{
|
|
id: 1,
|
|
content_type: 'voice_call',
|
|
call: {
|
|
provider_call_id: 'CA123',
|
|
status: 'ringing',
|
|
direction: 'incoming',
|
|
duration_seconds: null,
|
|
},
|
|
},
|
|
],
|
|
},
|
|
],
|
|
};
|
|
mutations[types.UPDATE_MESSAGE_CALL_STATUS](state, {
|
|
conversationId: 1,
|
|
callStatus: 'in-progress',
|
|
callSid: 'CA123',
|
|
});
|
|
expect(state.allConversations[0].messages[0].call).toEqual({
|
|
provider_call_id: 'CA123',
|
|
status: 'in-progress',
|
|
direction: 'incoming',
|
|
duration_seconds: null,
|
|
});
|
|
});
|
|
|
|
it('handles empty messages array', () => {
|
|
const state = {
|
|
allConversations: [{ id: 1, messages: [] }],
|
|
};
|
|
mutations[types.UPDATE_MESSAGE_CALL_STATUS](state, {
|
|
conversationId: 1,
|
|
callStatus: 'ringing',
|
|
callSid: 'CA123',
|
|
});
|
|
expect(state.allConversations[0].messages).toEqual([]);
|
|
});
|
|
|
|
it('does nothing if matching message has no call object yet', () => {
|
|
const state = {
|
|
allConversations: [
|
|
{
|
|
id: 1,
|
|
messages: [
|
|
{
|
|
id: 1,
|
|
content_type: 'voice_call',
|
|
call: { provider_call_id: 'CA-OTHER' },
|
|
},
|
|
],
|
|
},
|
|
],
|
|
};
|
|
mutations[types.UPDATE_MESSAGE_CALL_STATUS](state, {
|
|
conversationId: 1,
|
|
callStatus: 'completed',
|
|
callSid: 'CA-MISSING',
|
|
});
|
|
expect(state.allConversations[0].messages[0].call).toEqual({
|
|
provider_call_id: 'CA-OTHER',
|
|
});
|
|
});
|
|
});
|
|
});
|