Some checks failed
Create issues from TODOs / todos (push) Failing after 3s
Close inactive issues / close-issues (push) Has been cancelled
Deploy GitHub Pages / deploy-preview (push) Has been cancelled
CI / ⚙️ Variables Setup (push) Has been cancelled
CI / ⚙️ Test Guard (push) Has been cancelled
CI / 🚀 Notify external services - draft (push) Has been cancelled
CI / 📦 Build Packages (push) Has been cancelled
CI / 📦 Meteor Build (coverage) (push) Has been cancelled
CI / 📦 Meteor Build (production) (push) Has been cancelled
CI / 🚢 Build Docker (amd64, [account-service presence-service omnichannel-transcript-service], ${{ (github.event_name != 'release' && github.ref != 'refs/heads/develop') && 'coverage' || 'production' }}) (push) Has been cancelled
CI / 🚢 Build Docker (amd64, [account-service presence-service omnichannel-transcript-service], fips) (push) Has been cancelled
CI / 🚢 Build Docker (amd64, [authorization-service queue-worker-service ddp-streamer-service], ${{ (github.event_name != 'release' && github.ref != 'refs/heads/develop') && 'coverage' || 'production' }}) (push) Has been cancelled
CI / 🚢 Build Docker (amd64, [authorization-service queue-worker-service ddp-streamer-service], fips) (push) Has been cancelled
CI / 🚢 Build Docker (amd64, [rocketchat], ${{ (github.event_name != 'release' && github.ref != 'refs/heads/develop') && 'coverage' || 'production' }}) (push) Has been cancelled
CI / 🚢 Build Docker (amd64, [rocketchat], coverage) (push) Has been cancelled
CI / 🚢 Build Docker (amd64, [rocketchat], fips) (push) Has been cancelled
CI / 🚢 Build Docker (arm64, [account-service presence-service omnichannel-transcript-service], ${{ (github.event_name != 'release' && github.ref != 'refs/heads/develop') && 'coverage' || 'production' }}) (push) Has been cancelled
CI / 🚢 Build Docker (arm64, [authorization-service queue-worker-service ddp-streamer-service], ${{ (github.event_name != 'release' && github.ref != 'refs/heads/develop') && 'coverage' || 'production' }}) (push) Has been cancelled
CI / 🚢 Build Docker (arm64, [rocketchat], ${{ (github.event_name != 'release' && github.ref != 'refs/heads/develop') && 'coverage' || 'production' }}) (push) Has been cancelled
CI / 🚢 Build Docker (arm64, [rocketchat], coverage) (push) Has been cancelled
CI / 🚢 Publish Docker Images (ghcr.io) (push) Has been cancelled
CI / 📦 Track Image Sizes (push) Has been cancelled
CI / 🔎 Code Check - Action Lint (push) Has been cancelled
CI / 🔎 Code Check (push) Has been cancelled
CI / 🔨 Test Storybook (push) Has been cancelled
CI / 🔨 Test Unit (push) Has been cancelled
CI / 🔨 Test API (CE) (push) Has been cancelled
CI / 🔨 Test API Livechat (CE) (push) Has been cancelled
CI / 🔨 Test UI (CE) (push) Has been cancelled
CI / 🔨 Test API (EE) (push) Has been cancelled
CI / 🔨 Test API Livechat (EE) (push) Has been cancelled
CI / 🔨 Test API Apps (deno-runtime - EE) (push) Has been cancelled
CI / 🔨 Test UI (EE) (push) Has been cancelled
CI / 🔨 Test API (FIPS) (push) Has been cancelled
CI / 🔨 Test API Livechat (FIPS) (push) Has been cancelled
CI / 🔨 Test UI (FIPS) (push) Has been cancelled
CI / 🔨 Test Federation Matrix (push) Has been cancelled
CI / 📊 Report Coverage (push) Has been cancelled
CI / ✅ Tests Done (push) Has been cancelled
CI / 🚀 Publish build assets (push) Has been cancelled
CI / 🚀 Publish Docker Images (DockerHub) (push) Has been cancelled
CI / 🚀 Notify external services (push) Has been cancelled
CI / Update Version Durability (push) Has been cancelled
Release / Release (push) Has been cancelled
Auto-close duplicate issues / auto-close-duplicates (push) Failing after 1s
Code scanning - action / CodeQL-Build (push) Failing after 1s
124 lines
5.4 KiB
TypeScript
124 lines
5.4 KiB
TypeScript
import { expect } from 'chai';
|
|
import { describe, it } from 'mocha';
|
|
import proxyquire from 'proxyquire';
|
|
import sinon from 'sinon';
|
|
|
|
import { createFakeRoom, createFakeSubscription, createFakeUser } from '../../../../mocks/data';
|
|
|
|
const subscriptionsStub = {
|
|
findOneByRoomIdAndUserId: sinon.stub(),
|
|
removeByRoomId: sinon.stub(),
|
|
countByRoomId: sinon.stub(),
|
|
};
|
|
|
|
const livechatRoomsStub = {
|
|
findOneById: sinon.stub(),
|
|
};
|
|
|
|
const livechatStub = {
|
|
closeRoom: sinon.stub(),
|
|
};
|
|
|
|
const hasPermissionStub = sinon.stub();
|
|
|
|
const { closeLivechatRoom } = proxyquire.noCallThru().load('../../../../../server/lib/omnichannel/closeLivechatRoom.ts', {
|
|
'./closeRoom': livechatStub,
|
|
'../authorization/hasPermission': {
|
|
hasPermissionAsync: hasPermissionStub,
|
|
},
|
|
'@rocket.chat/models': {
|
|
Subscriptions: subscriptionsStub,
|
|
LivechatRooms: livechatRoomsStub,
|
|
},
|
|
});
|
|
|
|
describe('closeLivechatRoom', () => {
|
|
const user = createFakeUser();
|
|
const room = createFakeRoom({ t: 'l', open: true });
|
|
const subscription = createFakeSubscription({ rid: room._id, u: { username: user.username, _id: user._id } });
|
|
|
|
beforeEach(() => {
|
|
subscriptionsStub.findOneByRoomIdAndUserId.reset();
|
|
subscriptionsStub.removeByRoomId.reset();
|
|
subscriptionsStub.countByRoomId.reset();
|
|
livechatRoomsStub.findOneById.reset();
|
|
livechatStub.closeRoom.reset();
|
|
hasPermissionStub.reset();
|
|
});
|
|
|
|
it('should not perform any operation when an invalid room id is provided', async () => {
|
|
livechatRoomsStub.findOneById.resolves(null);
|
|
hasPermissionStub.resolves(true);
|
|
|
|
await expect(closeLivechatRoom(user, room._id, {})).to.be.rejectedWith('error-invalid-room');
|
|
expect(livechatStub.closeRoom.notCalled).to.be.true;
|
|
expect(livechatRoomsStub.findOneById.calledOnceWith(room._id)).to.be.true;
|
|
expect(subscriptionsStub.findOneByRoomIdAndUserId.notCalled).to.be.true;
|
|
expect(subscriptionsStub.removeByRoomId.notCalled).to.be.true;
|
|
});
|
|
|
|
it('should not perform any operation when the caller is not subscribed to an open room and does not have the permission to close others rooms', async () => {
|
|
livechatRoomsStub.findOneById.resolves(room);
|
|
subscriptionsStub.findOneByRoomIdAndUserId.resolves(null);
|
|
hasPermissionStub.resolves(false);
|
|
|
|
await expect(closeLivechatRoom(user, room._id, {})).to.be.rejectedWith('error-not-authorized');
|
|
expect(livechatStub.closeRoom.notCalled).to.be.true;
|
|
expect(livechatRoomsStub.findOneById.calledOnceWith(room._id)).to.be.true;
|
|
expect(subscriptionsStub.findOneByRoomIdAndUserId.calledOnceWith(room._id, user._id)).to.be.true;
|
|
expect(subscriptionsStub.removeByRoomId.notCalled).to.be.true;
|
|
});
|
|
|
|
it('should close the room when the caller is not subscribed to it but has the permission to close others rooms', async () => {
|
|
livechatRoomsStub.findOneById.resolves(room);
|
|
subscriptionsStub.findOneByRoomIdAndUserId.resolves(null);
|
|
hasPermissionStub.resolves(true);
|
|
|
|
await closeLivechatRoom(user, room._id, {});
|
|
expect(livechatStub.closeRoom.calledOnceWith(sinon.match({ room, user }))).to.be.true;
|
|
expect(livechatRoomsStub.findOneById.calledOnceWith(room._id)).to.be.true;
|
|
expect(subscriptionsStub.findOneByRoomIdAndUserId.calledOnceWith(room._id, user._id)).to.be.true;
|
|
expect(subscriptionsStub.removeByRoomId.notCalled).to.be.true;
|
|
});
|
|
|
|
it('should close the room when the caller is subscribed to it and does not have the permission to close others rooms', async () => {
|
|
livechatRoomsStub.findOneById.resolves(room);
|
|
subscriptionsStub.findOneByRoomIdAndUserId.resolves(subscription);
|
|
hasPermissionStub.resolves(false);
|
|
|
|
await closeLivechatRoom(user, room._id, {});
|
|
expect(livechatStub.closeRoom.calledOnceWith(sinon.match({ room, user }))).to.be.true;
|
|
expect(livechatRoomsStub.findOneById.calledOnceWith(room._id)).to.be.true;
|
|
expect(subscriptionsStub.findOneByRoomIdAndUserId.calledOnceWith(room._id, user._id)).to.be.true;
|
|
expect(subscriptionsStub.removeByRoomId.notCalled).to.be.true;
|
|
});
|
|
|
|
it('should call Livechat.closeRoom directly when forceClose is true even if room is in invalid state', async () => {
|
|
// Here we're using `open: false` as a way to simulate an invalid state. This should not happen in production. A room like this is effectively a "bad egg"
|
|
livechatRoomsStub.findOneById.resolves({ ...room, open: false });
|
|
subscriptionsStub.findOneByRoomIdAndUserId.resolves(subscription);
|
|
hasPermissionStub.resolves(true);
|
|
|
|
await closeLivechatRoom(user, room._id, { forceClose: true });
|
|
|
|
expect(
|
|
livechatStub.closeRoom.calledOnceWith(
|
|
sinon.match({ room: { ...room, open: false }, user, options: sinon.match({ clientAction: true }), forceClose: true }),
|
|
),
|
|
).to.be.true;
|
|
expect(livechatRoomsStub.findOneById.calledOnceWith(room._id)).to.be.true;
|
|
expect(subscriptionsStub.findOneByRoomIdAndUserId.calledOnceWith(room._id, user._id)).to.be.true;
|
|
});
|
|
|
|
it('should throw an error if forceClose is false and room is already closed', async () => {
|
|
livechatRoomsStub.findOneById.resolves({ ...room, open: false });
|
|
subscriptionsStub.findOneByRoomIdAndUserId.resolves(subscription);
|
|
hasPermissionStub.resolves(true);
|
|
|
|
await expect(closeLivechatRoom(user, room._id, { forceClose: false })).to.be.rejectedWith('error-room-already-closed');
|
|
expect(livechatStub.closeRoom.notCalled).to.be.true;
|
|
expect(livechatRoomsStub.findOneById.calledOnceWith(room._id)).to.be.true;
|
|
expect(subscriptionsStub.findOneByRoomIdAndUserId.calledOnceWith(room._id, user._id)).to.be.true;
|
|
});
|
|
});
|