mirror of
https://github.com/NodeBB/NodeBB.git
synced 2025-10-30 18:46:01 +01:00
* chore: up deps * chore: up composer * fix(deps): bump 2factor to v7 * chore: up harmony * chore: up harmony * fix: missing await * feat: allow middlewares to pass in template values via res.locals * feat: buildAccountData middleware automatically added ot all account routes * fix: properly allow values in res.locals.templateValues to be added to the template data * refactor: user/blocks * refactor(accounts): categories and consent * feat: automatically 404 if exposeUid or exposeGroupName come up empty * refactor: remove calls to getUserDataByUserSlug for most account routes, since it is populated via middleware now * fix: allow exposeUid and exposeGroupName to work with slugs with mixed capitalization * fix: move reputation removal check to accountHelpers method * test: skip i18n tests if ref branch when present is not develop * fix(deps): bump theme versions * fix(deps): bump ntfy and 2factor * chore: up harmony * fix: add missing return * fix: #11191, only focus on search input on md environments and up * feat: allow file uploads on mobile chat closes https://github.com/NodeBB/NodeBB/issues/11217 * chore: up themes * chore: add lang string * fix(deps): bump ntfy to 1.0.15 * refactor: use new if/each syntax * chore: up composer * fix: regression from user helper refactor * chore: up harmony * chore: up composer * chore: up harmony * chore: up harmony * chore: up harmony * chore: fix composer version * feat: add increment helper * chore: up harmony * fix: #11228 no timestamps in future ⌛ * chore: up harmony * check config.theme as well fire action:posts.loaded after processing dom * chore: up harmony * chore: up harmony * chore: up harmony * chore: up themes * chore: up harmony * remove extra class * refactor: move these to core from harmony * chore: up widgets * chore: up widgets * height auto * fix: closes #11238 * dont focus inputs, annoying on mobile * fix: dont focus twice, only focus on chat input on desktop dont wrap widget footer in row * chore: up harmony * chore: up harmony * update chat window * chore: up themes * fix cache buster for skins * chat fixes * chore: up harmony * chore: up composer * refactor: change hook logs to debug * fix: scroll to post right after adding to dom * fix: hash scrolling and highlighting correct post * test: re-enable read API schema tests * fix: add back schema changes for179faa2270andc3920ccb10* fix: schema changes from488f0978a4* fix: schema changes forf4cf482a87* fix: schema update forbe6bbabd0e* fix: schema changes for69c96078ea* fix: schema changes ford1364c3130* fix: schema changes for84ff1152f7* fix: schema changes forb860c2605c* fix: schema changes for23cb67a112* fix: schema changes forb916e42f40* fix: schema change fora9bbb586fc* fix: schema changes for4b738c8cd3* fix: schema changes for58b5781cea* fix: schema changes for794bf01b21* fix: schema changes for80ea12c1c1,e368feef51, and52ead114be* fix: composer-default object in config? * fix: schema changes for9acdc6808cand0930934200* fix: schema changes forc0a52924f1* fix: schema change foraba420a3f3, move loggedInUser to optional props * fix: schema changes for8c67031609* fix: schema changes for27e53b42f3* fix: schema changes for2835966518* fix: breaking test for email confirmation API call * fix: schema changes for refactored search page * fix: schema changes for user object * fix: schema changes for9f531f957e* fix: schema changes forc4042c70deand23175110a2* fix: schema changes for9b3616b103* fix: schema changes for5afd5de07d* fix: schema change for1d7baf1217* fix: schema changes for57bfb37c55andbe6bbabd0e* fix: schema changes for6e86b4afa2and3efad2e13band68f66223e7* fix: allowing optional qs prop in pagination keys (not sure why this didn't break before) * fix: re-login on email change * fix: schema changes forc926358d73* fix: schema changes for388a8270c9* fix: schema change for2658bcc821* fix: no need to call account middlewares for chats routes * fix: schema changes for71743affc3* fix: final schema changes * test: support for anyOf and oneOf * fix: check thumb * dont scroll to top on back press * remove group log * fix: add top margin to merged and deleted alerts * chore: up widgets * fix: improve fix-lists mixin * chore: up harmony/composer * feat: allow hiding quicksearch results during search * dont record searches made by composer * chore: up 54 * chore: up spam be gone * feat: add prev/next page and page count into mobile paginator * chore: up harmony * chore: up harmony * use old style for IS * fix: hide entire toolbar row if no posts or not singlePost * fix: updated messaging for post-queue template, #11206 * fix: btn-sm on post queue back button * fix: bump harmony, closes #11206 * fix: remove unused alert module import * fix: bump harmony * fix: bump harmony * chore: up harmony * refactor: IS scrolltop * fix: update users:search-user-for-chat source string * feat: support for mark-read toggle on chats dropdown and recent chats list * feat: api v3 calls to mark chat read/unread * feat: send event:chats.mark socket event on mark read or unread * refactor: allow frontend to mark chats as unread, use new API v3 routes instead of socket calls, better frontend event handling * docs: openapi schema updates for chat marking * fix: allow unread state toggling in chats dropdown too * fix: issue where repeated openings of the chats dropdown would continually add events for mark-read/unread * fix: debug log * refactor: move userSearch filter to a module * feat(routes): allow remounting /categories (#11230) * feat: send flags count to frontend on flags list page * refactor: filter form client-side js to extract out some logic * fix: applyFilters to not take any arguments, update selectedCids in updateButton instead of onHidden * fix: use userFilter module for assignee, reporterId, targetUid * fix(openapi): schema changes for updated flags page * fix: dont allow adding duplicates to userFilter * use same var * remove log * fix: closes #11282 * feat: lang key for x-topics * chore: up harmony * chore: up emoji * chore: up harmony * fix: update userFilter to allow new option `selectedBlock` * fix: wrong block name passed to userFilter * fix: https://github.com/NodeBB/NodeBB/issues/11283 * fix: chats, allow multiple dropdowns like in harmony * chore: up harmony * refactor: flag note adding/editing, closes #11285 * fix: remove old prepareEdit logic * chore: add caveat about hacky code block in userFilter module * fix: placeholders for userFilter module * refactor: navigator so it works with multiple thumbs/navigators * chore: up harmony * fix: closes #11287, destroy quick reply autocomplete on navigation * fix: filter disabled categories on user categories page count * chore: up harmony * docs: update openapi spec to include info about passing in timestamps for topic creation, removing timestamp as valid request param for topic replying * fix: send back null values on ACP search dashboard for startDate and endDate if not expicitly passed in, fix tests * fix: tweak table order in ACP dash searches * fix: only invoke navigator click drag on left mouse button * feat: add back unread indicator to navigator * clear bookmark on mark unread * fix: navigator crash on ajaxify * better thumb top calculation * fix: reset user bookmark when topic is marked unread * Revert "fix: reset user bookmark when topic is marked unread" This reverts commit9bcd85c2c6. * fix: update unread indicator on scroll, add unread count * chore: bump harmony * fix: crash on navigator unread update when backing out of a topic * fix: closes #11183 * fix: update topics:recent zset when rescheduling a topic * fix: dupe quote button, increase delay, hide immediately on empty selection * fix: navigator not showing up on first load * refactor: remove glance assorted fixes to navigator dont reduce remaning count if user scrolls down and up quickly only call topic.navigatorCallback when index changes * more sanity checks for bookmark dont allow setting bookmark higher than topic postcount * closes #11218, 🚋 * Revert "fix: update topics:recent zset when rescheduling a topic" This reverts commit737973cca9. * fix: #11306, show proper error if queued post doesn't exist was showing no-privileges if someone else accepted the post * https://github.com/NodeBB/NodeBB/issues/11307 dont use li * chore: up harmony * chore: bump version string * fix: copy paste fail * feat: closes #7382, tag filtering add client side support for filtering by tags on /category, /recent and /unread * chore: up harmony * chore: up harmony * Revert "fix: add back req.query fallback for backwards compatibility" [breaking] This reverts commitcf6cc2c454. This commit is no longer required as passing in a CSRF token via query parameter is no longer supported as of NodeBB v3.x This is a breaking change. * fix: pass csrf token in form data, re: NodeBB/NodeBB#11309 * chore: up deps * fix: tests, use x-csrf-token query param removed * test: fix csrf_token * lint: remove unused * feat: add itemprop="image" to avatar helper * fix: get chat upload button in chat modal * breaking: remove deprecated socket.io methods * test: update messaging tests to not use sockets * fix: parent post links * fix: prevent post tooltip if mouse leaves before data/tpl is loaded * chore: up harmony * chore: up harmony * chore: up harmony * chore: up harmony * fix: nested replies indices * fix(deps): bump 2factor * feat: add loggedIn user to all api routes * chore: up themes * refactor: audit admin v3 write api routes as per #11321 * refactor: audit category v3 write api routes as per #11321 [breaking] docs: fix open api spec for #11321 * refactor: audit chat v3 write api routes as per #11321 * refactor: audit files v3 write api routes as per #11321 * refactor: audit flags v3 write api routes as per #11321 * refactor: audit posts v3 write api routes as per #11321 * refactor: audit topics v3 write api routes as per #11321 * refactor: audit users v3 write api routes as per #11321 * fix: lang string * remove min height * fix: empty topic/labels taking up space * fix: tag filtering when changing filter to watched topics or changing popular time limit to month * chore: up harmony * fix: closes #11354, show no post error if queued post already accepted/rejected * test: #11354 * test: #11354 * fix(deps): bump 2factor * fix: #11357 clear cache on thumb remove * fix: thumb remove on windows, closes #11357 * test: openapi for thumbs * test: fix openapi --------- Co-authored-by: Julian Lam <julian@nodebb.org> Co-authored-by: Opliko <opliko.reg@protonmail.com>
792 lines
30 KiB
JavaScript
792 lines
30 KiB
JavaScript
'use strict';
|
|
|
|
const assert = require('assert');
|
|
const async = require('async');
|
|
const request = require('request-promise-native');
|
|
const nconf = require('nconf');
|
|
const util = require('util');
|
|
|
|
const sleep = util.promisify(setTimeout);
|
|
|
|
const db = require('./mocks/databasemock');
|
|
const meta = require('../src/meta');
|
|
const User = require('../src/user');
|
|
const Groups = require('../src/groups');
|
|
const Messaging = require('../src/messaging');
|
|
const api = require('../src/api');
|
|
const helpers = require('./helpers');
|
|
const socketModules = require('../src/socket.io/modules');
|
|
const utils = require('../src/utils');
|
|
const translator = require('../src/translator');
|
|
|
|
describe('Messaging Library', () => {
|
|
const mocks = {
|
|
users: {
|
|
foo: {}, // the admin
|
|
bar: {},
|
|
baz: {}, // the user with chat restriction enabled
|
|
herp: {},
|
|
},
|
|
};
|
|
let roomId;
|
|
|
|
let chatMessageDelay;
|
|
|
|
const callv3API = async (method, path, body, user) => {
|
|
const options = {
|
|
method,
|
|
body,
|
|
json: true,
|
|
jar: mocks.users[user].jar,
|
|
resolveWithFullResponse: true,
|
|
simple: false,
|
|
};
|
|
|
|
if (method !== 'get') {
|
|
options.headers = {
|
|
'x-csrf-token': mocks.users[user].csrf,
|
|
};
|
|
}
|
|
|
|
return request(`${nconf.get('url')}/api/v3${path}`, options);
|
|
};
|
|
|
|
before(async () => {
|
|
// Create 3 users: 1 admin, 2 regular
|
|
({
|
|
foo: mocks.users.foo.uid,
|
|
bar: mocks.users.bar.uid,
|
|
baz: mocks.users.baz.uid,
|
|
herp: mocks.users.herp.uid,
|
|
} = await utils.promiseParallel({
|
|
foo: User.create({ username: 'foo', password: 'barbar' }), // admin
|
|
bar: User.create({ username: 'bar', password: 'bazbaz' }), // admin
|
|
baz: User.create({ username: 'baz', password: 'quuxquux' }), // restricted user
|
|
herp: User.create({ username: 'herp', password: 'derpderp' }), // a regular user
|
|
}));
|
|
|
|
await Groups.join('administrators', mocks.users.foo.uid);
|
|
await User.setSetting(mocks.users.baz.uid, 'restrictChat', '1');
|
|
|
|
({ jar: mocks.users.foo.jar, csrf_token: mocks.users.foo.csrf } = await helpers.loginUser('foo', 'barbar'));
|
|
({ jar: mocks.users.bar.jar, csrf_token: mocks.users.bar.csrf } = await helpers.loginUser('bar', 'bazbaz'));
|
|
({ jar: mocks.users.baz.jar, csrf_token: mocks.users.baz.csrf } = await helpers.loginUser('baz', 'quuxquux'));
|
|
({ jar: mocks.users.herp.jar, csrf_token: mocks.users.herp.csrf } = await helpers.loginUser('herp', 'derpderp'));
|
|
|
|
chatMessageDelay = meta.config.chatMessageDelay;
|
|
meta.config.chatMessageDelay = 0;
|
|
});
|
|
|
|
after(() => {
|
|
meta.configs.chatMessageDelay = chatMessageDelay;
|
|
});
|
|
|
|
describe('.canMessage()', () => {
|
|
it('should allow messages to be sent to an unrestricted user', (done) => {
|
|
Messaging.canMessageUser(mocks.users.baz.uid, mocks.users.herp.uid, (err) => {
|
|
assert.ifError(err);
|
|
done();
|
|
});
|
|
});
|
|
|
|
it('should NOT allow messages to be sent to a restricted user', async () => {
|
|
await User.setSetting(mocks.users.baz.uid, 'restrictChat', '1');
|
|
try {
|
|
await Messaging.canMessageUser(mocks.users.herp.uid, mocks.users.baz.uid);
|
|
} catch (err) {
|
|
assert.strictEqual(err.message, '[[error:chat-restricted]]');
|
|
}
|
|
});
|
|
|
|
it('should always allow admins through', (done) => {
|
|
Messaging.canMessageUser(mocks.users.foo.uid, mocks.users.baz.uid, (err) => {
|
|
assert.ifError(err);
|
|
done();
|
|
});
|
|
});
|
|
|
|
it('should allow messages to be sent to a restricted user if restricted user follows sender', (done) => {
|
|
User.follow(mocks.users.baz.uid, mocks.users.herp.uid, () => {
|
|
Messaging.canMessageUser(mocks.users.herp.uid, mocks.users.baz.uid, (err) => {
|
|
assert.ifError(err);
|
|
done();
|
|
});
|
|
});
|
|
});
|
|
});
|
|
|
|
describe('rooms', () => {
|
|
it('should fail to create a new chat room with invalid data', async () => {
|
|
const { body } = await callv3API('post', '/chats', {}, 'foo');
|
|
assert.equal(body.status.message, await translator.translate('[[error:required-parameters-missing, uids]]'));
|
|
});
|
|
|
|
it('should return rate limit error on second try', async () => {
|
|
const oldValue = meta.config.chatMessageDelay;
|
|
meta.config.chatMessageDelay = 1000;
|
|
|
|
await callv3API('post', '/chats', {
|
|
uids: [mocks.users.baz.uid],
|
|
}, 'foo');
|
|
|
|
const { statusCode, body } = await callv3API('post', `/chats`, {
|
|
uids: [mocks.users.baz.uid],
|
|
}, 'foo');
|
|
|
|
assert.equal(statusCode, 400);
|
|
assert.equal(body.status.code, 'bad-request');
|
|
assert.equal(body.status.message, await translator.translate('[[error:too-many-messages]]'));
|
|
meta.config.chatMessageDelay = oldValue;
|
|
});
|
|
|
|
it('should create a new chat room', async () => {
|
|
await User.setSetting(mocks.users.baz.uid, 'restrictChat', '0');
|
|
const { body } = await callv3API('post', `/chats`, {
|
|
uids: [mocks.users.baz.uid],
|
|
}, 'foo');
|
|
await User.setSetting(mocks.users.baz.uid, 'restrictChat', '1');
|
|
|
|
roomId = body.response.roomId;
|
|
assert(roomId);
|
|
});
|
|
|
|
it('should send a user-join system message when a chat room is created', async () => {
|
|
const { body } = await callv3API('get', `/chats/${roomId}`, {}, 'foo');
|
|
const { messages } = body.response;
|
|
assert.equal(messages.length, 2);
|
|
assert.strictEqual(messages[0].system, true);
|
|
assert.strictEqual(messages[0].content, 'user-join');
|
|
|
|
const { statusCode, body: body2 } = await callv3API('put', `/chats/${roomId}/messages/${messages[0].messageId}`, {
|
|
message: 'test',
|
|
}, 'foo');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.equal(body2.status.message, await translator.translate('[[error:cant-edit-chat-message]]'));
|
|
});
|
|
|
|
it('should fail to add user to room with invalid data', async () => {
|
|
let { statusCode, body } = await callv3API('post', `/chats/${roomId}/users`, {}, 'foo');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:required-parameters-missing, uids]]'));
|
|
|
|
({ statusCode, body } = await callv3API('post', `/chats/${roomId}/users`, { uids: [null] }, 'foo'));
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:no-user]]'));
|
|
});
|
|
|
|
it('should add a user to room', async () => {
|
|
await callv3API('post', `/chats/${roomId}/users`, { uids: [mocks.users.herp.uid] }, 'foo');
|
|
const isInRoom = await Messaging.isUserInRoom(mocks.users.herp.uid, roomId);
|
|
assert(isInRoom);
|
|
});
|
|
|
|
it('should get users in room', async () => {
|
|
const { body } = await callv3API('get', `/chats/${roomId}/users`, {}, 'foo');
|
|
assert(Array.isArray(body.response.users));
|
|
assert.strictEqual(body.response.users.length, 3);
|
|
});
|
|
|
|
it('should throw error if user is not in room', async () => {
|
|
const { statusCode, body } = await callv3API('get', `/chats/${roomId}/users`, {}, 'bar');
|
|
assert.strictEqual(statusCode, 403);
|
|
assert.equal(body.status.message, await translator.translate('[[error:no-privileges]]'));
|
|
});
|
|
|
|
it('should fail to add users to room if max is reached', async () => {
|
|
meta.config.maximumUsersInChatRoom = 2;
|
|
const { statusCode, body } = await callv3API('post', `/chats/${roomId}/users`, { uids: [mocks.users.bar.uid] }, 'foo');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.equal(body.status.message, await translator.translate('[[error:cant-add-more-users-to-chat-room]]'));
|
|
meta.config.maximumUsersInChatRoom = 0;
|
|
});
|
|
|
|
it('should fail to add users to room if user does not exist', async () => {
|
|
const { statusCode, body } = await callv3API('post', `/chats/${roomId}/users`, { uids: [98237498234] }, 'foo');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:no-user]]'));
|
|
});
|
|
|
|
it('should fail to add self to room', async () => {
|
|
const { statusCode, body } = await callv3API('post', `/chats/${roomId}/users`, { uids: [mocks.users.foo.uid] }, 'foo');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:cant-chat-with-yourself]]'));
|
|
});
|
|
|
|
it('should fail to leave room with invalid data', async () => {
|
|
let { statusCode, body } = await callv3API('delete', `/chats/${roomId}/users`, {}, 'foo');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:required-parameters-missing, uids]]'));
|
|
|
|
({ statusCode, body } = await callv3API('delete', `/chats/${roomId}/users`, { uids: [98237423] }, 'foo'));
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:no-user]]'));
|
|
});
|
|
|
|
it('should leave the chat room', async () => {
|
|
await callv3API('delete', `/chats/${roomId}/users/${mocks.users.baz.uid}`, {}, 'baz');
|
|
const isUserInRoom = await Messaging.isUserInRoom(mocks.users.baz.uid, roomId);
|
|
assert.equal(isUserInRoom, false);
|
|
const data = await Messaging.getRoomData(roomId);
|
|
assert.equal(data.owner, mocks.users.foo.uid);
|
|
});
|
|
|
|
it('should send a user-leave system message when a user leaves the chat room', async () => {
|
|
const { body } = await callv3API('get', `/chats/${roomId}`, {}, 'foo');
|
|
const { messages } = body.response;
|
|
const message = messages.pop();
|
|
assert.strictEqual(message.system, true);
|
|
assert.strictEqual(message.content, 'user-leave');
|
|
});
|
|
|
|
it('should not send a user-leave system message when a user tries to leave a room they are not in', async () => {
|
|
await callv3API('delete', `/chats/${roomId}/users/${mocks.users.baz.uid}`, {}, 'baz');
|
|
const { body } = await callv3API('get', `/chats/${roomId}`, {}, 'foo');
|
|
const { messages } = body.response;
|
|
|
|
assert.equal(messages.length, 4);
|
|
let message = messages.pop();
|
|
assert.strictEqual(message.system, true);
|
|
assert.strictEqual(message.content, 'user-leave');
|
|
|
|
// The message before should still be a user-join
|
|
message = messages.pop();
|
|
assert.strictEqual(message.system, true);
|
|
assert.strictEqual(message.content, 'user-join');
|
|
});
|
|
|
|
it('should change owner when owner leaves room', async () => {
|
|
const { body } = await callv3API('post', '/chats', {
|
|
uids: [mocks.users.foo.uid],
|
|
}, 'herp');
|
|
|
|
await callv3API('post', `/chats/${body.response.roomId}/users`, { uids: [mocks.users.baz.uid] }, 'herp');
|
|
|
|
await callv3API('delete', `/chats/${body.response.roomId}/users/${mocks.users.herp.uid}`, {}, 'herp');
|
|
|
|
const data = await Messaging.getRoomData(body.response.roomId);
|
|
assert.equal(data.owner, mocks.users.foo.uid);
|
|
});
|
|
|
|
it('should change owner if owner is deleted', async () => {
|
|
const sender = await User.create({ username: 'deleted_chat_user', password: 'barbar' });
|
|
const { jar: senderJar, csrf_token: senderCsrf } = await helpers.loginUser('deleted_chat_user', 'barbar');
|
|
|
|
const receiver = await User.create({ username: 'receiver' });
|
|
const { response } = await request(`${nconf.get('url')}/api/v3/chats`, {
|
|
method: 'post',
|
|
json: true,
|
|
jar: senderJar,
|
|
body: {
|
|
uids: [receiver],
|
|
},
|
|
headers: {
|
|
'x-csrf-token': senderCsrf,
|
|
},
|
|
});
|
|
await User.deleteAccount(sender);
|
|
const data = await Messaging.getRoomData(response.roomId);
|
|
assert.equal(data.owner, receiver);
|
|
});
|
|
|
|
it('should fail to remove user from room', async () => {
|
|
let { statusCode, body } = await callv3API('delete', `/chats/${roomId}/users`, {}, 'foo');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:required-parameters-missing, uids]]'));
|
|
|
|
({ statusCode, body } = await callv3API('delete', `/chats/${roomId}/users`, { uids: [null] }, 'foo'));
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:no-user]]'));
|
|
});
|
|
|
|
it('should fail to remove user from room if user does not exist', async () => {
|
|
const { statusCode, body } = await callv3API('delete', `/chats/${roomId}/users`, { uids: [99] }, 'foo');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:no-user]]'));
|
|
});
|
|
|
|
it('should remove user from room', async () => {
|
|
const { statusCode, body } = await callv3API('post', `/chats`, {
|
|
uids: [mocks.users.herp.uid],
|
|
}, 'foo');
|
|
const { roomId } = body.response;
|
|
assert.strictEqual(statusCode, 200);
|
|
|
|
let isInRoom = await Messaging.isUserInRoom(mocks.users.herp.uid, roomId);
|
|
assert(isInRoom);
|
|
|
|
await callv3API('delete', `/chats/${roomId}/users`, { uids: [mocks.users.herp.uid] }, 'foo');
|
|
isInRoom = await Messaging.isUserInRoom(mocks.users.herp.uid, roomId);
|
|
assert(!isInRoom);
|
|
});
|
|
|
|
it('should fail to send a message to room with invalid data', async () => {
|
|
let { body } = await callv3API('post', `/chats/abc`, { message: 'test' }, 'foo');
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:invalid-data]]'));
|
|
|
|
({ body } = await callv3API('post', `/chats/1`, {}, 'foo'));
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:required-parameters-missing, message]]'));
|
|
});
|
|
|
|
it('should fail to send chat if content is empty', async () => {
|
|
const { body } = await callv3API('post', `/chats/${roomId}`, {
|
|
message: ' ',
|
|
}, 'foo');
|
|
const { status, response } = body;
|
|
|
|
assert.deepStrictEqual(response, {});
|
|
assert.equal(status.message, await translator.translate('[[error:invalid-chat-message]]'));
|
|
});
|
|
|
|
it('should send a message to a room', async () => {
|
|
const { body } = await callv3API('post', `/chats/${roomId}`, { roomId: roomId, message: 'first chat message' }, 'foo');
|
|
const messageData = body.response;
|
|
assert(messageData);
|
|
assert.equal(messageData.content, 'first chat message');
|
|
assert(messageData.fromUser);
|
|
assert(messageData.roomId, roomId);
|
|
const raw =
|
|
await util.promisify(socketModules.chats.getRaw)({ uid: mocks.users.foo.uid }, { mid: messageData.mid });
|
|
assert.equal(raw, 'first chat message');
|
|
});
|
|
|
|
it('should fail to send second message due to rate limit', async () => {
|
|
const socketMock = { uid: mocks.users.foo.uid };
|
|
const oldValue = meta.config.chatMessageDelay;
|
|
meta.config.chatMessageDelay = 1000;
|
|
|
|
await callv3API('post', `/chats/${roomId}`, { roomId: roomId, message: 'first chat message' }, 'foo');
|
|
const { body } = await callv3API('post', `/chats/${roomId}`, { roomId: roomId, message: 'first chat message' }, 'foo');
|
|
const { status } = body;
|
|
assert.equal(status.message, await translator.translate('[[error:too-many-messages]]'));
|
|
meta.config.chatMessageDelay = oldValue;
|
|
});
|
|
|
|
it('should return invalid-data error', (done) => {
|
|
socketModules.chats.getRaw({ uid: mocks.users.foo.uid }, null, (err) => {
|
|
assert.equal(err.message, '[[error:invalid-data]]');
|
|
socketModules.chats.getRaw({ uid: mocks.users.foo.uid }, {}, (err) => {
|
|
assert.equal(err.message, '[[error:invalid-data]]');
|
|
done();
|
|
});
|
|
});
|
|
});
|
|
|
|
it('should return not allowed error if mid is not in room', async () => {
|
|
const uids = await User.create({ username: 'dummy' });
|
|
let { body } = await callv3API('post', '/chats', { uids: [uids] }, 'baz');
|
|
const myRoomId = body.response.roomId;
|
|
assert(myRoomId);
|
|
|
|
try {
|
|
await util.promisify(socketModules.chats.getRaw)({ uid: mocks.users.baz.uid }, { mid: 200 });
|
|
} catch (err) {
|
|
assert(err);
|
|
assert.equal(err.message, '[[error:not-allowed]]');
|
|
}
|
|
|
|
({ body } = await callv3API('post', `/chats/${myRoomId}`, { roomId: myRoomId, message: 'admin will see this' }, 'baz'));
|
|
const message = body.response;
|
|
const raw = await util.promisify(socketModules.chats.getRaw)({ uid: mocks.users.foo.uid }, { mid: message.mid });
|
|
assert.equal(raw, 'admin will see this');
|
|
});
|
|
|
|
|
|
it('should notify offline users of message', async () => {
|
|
meta.config.notificationSendDelay = 0.1;
|
|
|
|
const { body } = await callv3API('post', '/chats', { uids: [mocks.users.baz.uid] }, 'foo');
|
|
const { roomId } = body.response;
|
|
assert(roomId);
|
|
|
|
await callv3API('post', `/chats/${roomId}/users`, { uids: [mocks.users.herp.uid] }, 'foo');
|
|
await db.sortedSetAdd('users:online', Date.now() - ((meta.config.onlineCutoff * 60000) + 50000), mocks.users.herp.uid);
|
|
|
|
await callv3API('post', `/chats/${roomId}`, { roomId: roomId, message: 'second chat message **bold** text' }, 'foo');
|
|
await sleep(3000);
|
|
const data = await User.notifications.get(mocks.users.herp.uid);
|
|
assert(data.unread[0]);
|
|
const notification = data.unread[0];
|
|
assert.strictEqual(notification.bodyShort, '[[notifications:new_message_from, foo]]');
|
|
assert.strictEqual(notification.nid, `chat_${mocks.users.foo.uid}_${roomId}`);
|
|
assert.strictEqual(notification.path, `${nconf.get('relative_path')}/chats/${roomId}`);
|
|
});
|
|
|
|
it('should get messages from room', async () => {
|
|
const { body } = await callv3API('get', `/chats/${roomId}`, {}, 'foo');
|
|
const { messages } = body.response;
|
|
assert(Array.isArray(messages));
|
|
|
|
// Filter out system messages
|
|
const normalMessages = messages.filter(message => !message.system);
|
|
assert.equal(normalMessages[0].roomId, roomId);
|
|
assert.equal(normalMessages[0].fromuid, mocks.users.foo.uid);
|
|
});
|
|
|
|
it('should fail to mark read with invalid data', async () => {
|
|
let _err;
|
|
try {
|
|
await api.chats.mark({ uid: null }, { state: 0, roomId });
|
|
} catch (err) {
|
|
_err = err;
|
|
}
|
|
assert.strictEqual(_err.message, '[[error:invalid-data]]');
|
|
|
|
try {
|
|
await api.chats.mark({ uid: mocks.users.foo.uid }, null);
|
|
} catch (err) {
|
|
_err = err;
|
|
}
|
|
assert.strictEqual(_err.message, '[[error:invalid-data]]');
|
|
});
|
|
|
|
it('should not error if user is not in room', async () => {
|
|
await api.chats.mark({ uid: mocks.users.herp.uid }, { state: 0, roomId: 10 });
|
|
});
|
|
|
|
it('should mark room read', async () => {
|
|
await api.chats.mark({ uid: mocks.users.foo.uid }, { state: 0, roomId: roomId });
|
|
});
|
|
|
|
it('should mark all rooms read', (done) => {
|
|
socketModules.chats.markAllRead({ uid: mocks.users.foo.uid }, {}, (err) => {
|
|
assert.ifError(err);
|
|
done();
|
|
});
|
|
});
|
|
|
|
it('should fail to rename room with invalid data', async () => {
|
|
let { body } = await callv3API('put', `/chats/${roomId}`, { name: null }, 'foo');
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:invalid-data]]'));
|
|
|
|
({ body } = await callv3API('put', `/chats/${roomId}`, {}, 'foo'));
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:required-parameters-missing, name]]'));
|
|
});
|
|
|
|
it('should rename room', async () => {
|
|
const { statusCode } = await callv3API('put', `/chats/${roomId}`, { name: 'new room name' }, 'foo');
|
|
assert.strictEqual(statusCode, 200);
|
|
});
|
|
|
|
it('should send a room-rename system message when a room is renamed', async () => {
|
|
const { body } = await callv3API('get', `/chats/${roomId}`, {}, 'foo');
|
|
const { messages } = body.response;
|
|
|
|
const message = messages.pop();
|
|
assert.strictEqual(message.system, true);
|
|
assert.strictEqual(message.content, 'room-rename, new room name');
|
|
});
|
|
|
|
it('should fail to load room with invalid-data', async () => {
|
|
const { body } = await callv3API('get', `/chats/abc`, {}, 'foo');
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:invalid-data]]'));
|
|
});
|
|
|
|
it('should fail to load room if user is not in', async () => {
|
|
const { body } = await callv3API('get', `/chats/${roomId}`, {}, 'baz');
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:no-privileges]]'));
|
|
});
|
|
|
|
it('should load chat room', async () => {
|
|
const { body } = await callv3API('get', `/chats/${roomId}`, {}, 'foo');
|
|
assert.strictEqual(body.response.roomName, 'new room name');
|
|
});
|
|
|
|
it('should return true if user is dnd', (done) => {
|
|
db.setObjectField(`user:${mocks.users.herp.uid}`, 'status', 'dnd', (err) => {
|
|
assert.ifError(err);
|
|
socketModules.chats.isDnD({ uid: mocks.users.foo.uid }, mocks.users.herp.uid, (err, isDnD) => {
|
|
assert.ifError(err);
|
|
assert(isDnD);
|
|
done();
|
|
});
|
|
});
|
|
});
|
|
|
|
it('should fail to load recent chats with invalid data', (done) => {
|
|
socketModules.chats.getRecentChats({ uid: mocks.users.foo.uid }, null, (err) => {
|
|
assert.equal(err.message, '[[error:invalid-data]]');
|
|
socketModules.chats.getRecentChats({ uid: mocks.users.foo.uid }, { after: null }, (err) => {
|
|
assert.equal(err.message, '[[error:invalid-data]]');
|
|
socketModules.chats.getRecentChats({ uid: mocks.users.foo.uid }, { after: 0, uid: null }, (err) => {
|
|
assert.equal(err.message, '[[error:invalid-data]]');
|
|
done();
|
|
});
|
|
});
|
|
});
|
|
});
|
|
|
|
it('should load recent chats of user', (done) => {
|
|
socketModules.chats.getRecentChats(
|
|
{ uid: mocks.users.foo.uid },
|
|
{ after: 0, uid: mocks.users.foo.uid },
|
|
(err, data) => {
|
|
assert.ifError(err);
|
|
assert(Array.isArray(data.rooms));
|
|
done();
|
|
}
|
|
);
|
|
});
|
|
|
|
it('should escape teaser', async () => {
|
|
await callv3API('post', `/chats/${roomId}`, { roomId: roomId, message: '<svg/onload=alert(document.location);' }, 'foo');
|
|
const data = await util.promisify(socketModules.chats.getRecentChats)(
|
|
{ uid: mocks.users.foo.uid },
|
|
{ after: 0, uid: mocks.users.foo.uid }
|
|
);
|
|
|
|
assert.equal(data.rooms[0].teaser.content, '<svg/onload=alert(document.location);');
|
|
});
|
|
|
|
it('should fail to check if user has private chat with invalid data', (done) => {
|
|
socketModules.chats.hasPrivateChat({ uid: null }, null, (err) => {
|
|
assert.equal(err.message, '[[error:invalid-data]]');
|
|
socketModules.chats.hasPrivateChat({ uid: mocks.users.foo.uid }, null, (err) => {
|
|
assert.equal(err.message, '[[error:invalid-data]]');
|
|
done();
|
|
});
|
|
});
|
|
});
|
|
|
|
it('should check if user has private chat with another uid', (done) => {
|
|
socketModules.chats.hasPrivateChat({ uid: mocks.users.foo.uid }, mocks.users.herp.uid, (err, roomId) => {
|
|
assert.ifError(err);
|
|
assert(roomId);
|
|
done();
|
|
});
|
|
});
|
|
});
|
|
|
|
describe('edit/delete', () => {
|
|
const socketModules = require('../src/socket.io/modules');
|
|
let mid;
|
|
let mid2;
|
|
before(async () => {
|
|
await callv3API('post', `/chats/${roomId}/users`, { uids: [mocks.users.baz.uid] }, 'foo');
|
|
let { body } = await callv3API('post', `/chats/${roomId}`, { roomId: roomId, message: 'first chat message' }, 'foo');
|
|
mid = body.response.mid;
|
|
({ body } = await callv3API('post', `/chats/${roomId}`, { roomId: roomId, message: 'second chat message' }, 'baz'));
|
|
mid2 = body.response.mid;
|
|
});
|
|
|
|
after(async () => {
|
|
await callv3API('delete', `/chats/${roomId}/users/${mocks.users.baz.uid}`, {}, 'baz');
|
|
});
|
|
|
|
it('should fail to edit message with invalid data', async () => {
|
|
let { statusCode, body } = await callv3API('put', `/chats/1/messages/10000`, { message: 'foo' }, 'foo');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:invalid-mid]]'));
|
|
|
|
({ statusCode, body } = await callv3API('put', `/chats/${roomId}/messages/${mid}`, {}, 'foo'));
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:invalid-chat-message]]'));
|
|
});
|
|
|
|
it('should fail to edit message if new content is empty string', async () => {
|
|
const { statusCode, body } = await callv3API('put', `/chats/${roomId}/messages/${mid}`, { message: ' ' }, 'foo');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:invalid-chat-message]]'));
|
|
});
|
|
|
|
it('should fail to edit message if not own message', async () => {
|
|
const { statusCode, body } = await callv3API('put', `/chats/${roomId}/messages/${mid}`, { message: 'message edited' }, 'herp');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:cant-edit-chat-message]]'));
|
|
});
|
|
|
|
it('should fail to edit message if message not in room', async () => {
|
|
const { statusCode, body } = await callv3API('put', `/chats/${roomId}/messages/1014`, { message: 'message edited' }, 'herp');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, await translator.translate('[[error:invalid-mid]]'));
|
|
});
|
|
|
|
it('should edit message', async () => {
|
|
let { statusCode, body } = await callv3API('put', `/chats/${roomId}/messages/${mid}`, { message: 'message edited' }, 'foo');
|
|
assert.strictEqual(statusCode, 200);
|
|
assert.strictEqual(body.response.content, 'message edited');
|
|
|
|
({ statusCode, body } = await callv3API('get', `/chats/${roomId}/messages/${mid}`, {}, 'foo'));
|
|
assert.strictEqual(statusCode, 200);
|
|
assert.strictEqual(body.response.content, 'message edited');
|
|
});
|
|
|
|
it('should fail to delete message if not owner', async () => {
|
|
const { statusCode, body } = await callv3API('delete', `/chats/${roomId}/messages/${mid}`, {}, 'herp');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, 'You are not allowed to delete this message');
|
|
});
|
|
|
|
it('should mark the message as deleted', async () => {
|
|
await callv3API('delete', `/chats/${roomId}/messages/${mid}`, {}, 'foo');
|
|
const value = await db.getObjectField(`message:${mid}`, 'deleted');
|
|
assert.strictEqual(1, parseInt(value, 10));
|
|
});
|
|
|
|
it('should show deleted message to original users', async () => {
|
|
const { body } = await callv3API('get', `/chats/${roomId}`, {}, 'foo');
|
|
const { messages } = body.response;
|
|
|
|
// Reduce messages to their mids
|
|
const mids = messages.reduce((mids, cur) => {
|
|
mids.push(cur.messageId);
|
|
return mids;
|
|
}, []);
|
|
|
|
assert(mids.includes(mid));
|
|
});
|
|
|
|
it('should not show deleted message to other users', async () => {
|
|
const { body } = await callv3API('get', `/chats/${roomId}`, {}, 'herp');
|
|
const { messages } = body.response;
|
|
messages.forEach((msg) => {
|
|
assert(!msg.deleted || msg.content === '[[modules:chat.message-deleted]]', msg.content);
|
|
assert(!msg.deleted || msg.cleanedContent, '[[modules:chat.message-deleted]]', msg.content);
|
|
});
|
|
});
|
|
|
|
it('should error out if a message is deleted again', async () => {
|
|
const { statusCode, body } = await callv3API('delete', `/chats/${roomId}/messages/${mid}`, {}, 'foo');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, 'This chat message has already been deleted.');
|
|
});
|
|
|
|
it('should restore the message', async () => {
|
|
await callv3API('post', `/chats/${roomId}/messages/${mid}`, {}, 'foo');
|
|
const value = await db.getObjectField(`message:${mid}`, 'deleted');
|
|
assert.strictEqual(0, parseInt(value, 10));
|
|
});
|
|
|
|
it('should error out if a message is restored again', async () => {
|
|
const { statusCode, body } = await callv3API('post', `/chats/${roomId}/messages/${mid}`, {}, 'foo');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, 'This chat message has already been restored.');
|
|
});
|
|
|
|
describe('disabled via ACP', () => {
|
|
before(async () => {
|
|
meta.config.disableChatMessageEditing = true;
|
|
});
|
|
|
|
after(async () => {
|
|
meta.config.disableChatMessageEditing = false;
|
|
});
|
|
|
|
it('should error out for regular users', async () => {
|
|
const { statusCode, body } = await callv3API('delete', `/chats/${roomId}/messages/${mid2}`, {}, 'baz');
|
|
assert.strictEqual(statusCode, 400);
|
|
assert.strictEqual(body.status.message, 'chat-message-editing-disabled');
|
|
});
|
|
|
|
it('should succeed for administrators', async () => {
|
|
await callv3API('delete', `/chats/${roomId}/messages/${mid2}`, {}, 'foo');
|
|
await callv3API('post', `/chats/${roomId}/messages/${mid2}`, {}, 'foo');
|
|
});
|
|
|
|
it('should succeed for global moderators', async () => {
|
|
await Groups.join(['Global Moderators'], mocks.users.baz.uid);
|
|
|
|
await callv3API('delete', `/chats/${roomId}/messages/${mid2}`, {}, 'baz');
|
|
await callv3API('post', `/chats/${roomId}/messages/${mid2}`, {}, 'baz');
|
|
|
|
await Groups.leave(['Global Moderators'], mocks.users.baz.uid);
|
|
});
|
|
});
|
|
});
|
|
|
|
describe('controller', () => {
|
|
it('should 404 if chat is disabled', async () => {
|
|
meta.config.disableChat = 1;
|
|
const response = await request(`${nconf.get('url')}/user/baz/chats`, {
|
|
resolveWithFullResponse: true,
|
|
simple: false,
|
|
});
|
|
|
|
assert.equal(response.statusCode, 404);
|
|
});
|
|
|
|
it('should 500 for guest with no privilege error', async () => {
|
|
meta.config.disableChat = 0;
|
|
const response = await request(`${nconf.get('url')}/api/user/baz/chats`, {
|
|
resolveWithFullResponse: true,
|
|
simple: false,
|
|
json: true,
|
|
});
|
|
const { body } = response;
|
|
|
|
assert.equal(response.statusCode, 500);
|
|
assert.equal(body.error, '[[error:no-privileges]]');
|
|
});
|
|
|
|
it('should 404 for non-existent user', async () => {
|
|
const response = await request(`${nconf.get('url')}/user/doesntexist/chats`, {
|
|
resolveWithFullResponse: true,
|
|
simple: false,
|
|
});
|
|
|
|
assert.equal(response.statusCode, 404);
|
|
});
|
|
});
|
|
|
|
describe('logged in chat controller', () => {
|
|
let jar;
|
|
before(async () => {
|
|
({ jar } = await helpers.loginUser('herp', 'derpderp'));
|
|
});
|
|
|
|
it('should return chats page data', async () => {
|
|
const response = await request(`${nconf.get('url')}/api/user/herp/chats`, {
|
|
resolveWithFullResponse: true,
|
|
simple: false,
|
|
json: true,
|
|
jar,
|
|
});
|
|
const { body } = response;
|
|
|
|
assert.equal(response.statusCode, 200);
|
|
assert(Array.isArray(body.rooms));
|
|
assert.equal(body.rooms.length, 2);
|
|
assert.equal(body.title, '[[pages:chats]]');
|
|
});
|
|
|
|
it('should return room data', async () => {
|
|
const response = await request(`${nconf.get('url')}/api/user/herp/chats/${roomId}`, {
|
|
resolveWithFullResponse: true,
|
|
simple: false,
|
|
json: true,
|
|
jar,
|
|
});
|
|
const { body } = response;
|
|
|
|
assert.equal(response.statusCode, 200);
|
|
assert.equal(body.roomId, roomId);
|
|
assert.equal(body.isOwner, false);
|
|
});
|
|
|
|
it('should redirect to chats page', async () => {
|
|
const res = await request(`${nconf.get('url')}/api/chats`, {
|
|
resolveWithFullResponse: true,
|
|
simple: false,
|
|
jar,
|
|
json: true,
|
|
});
|
|
const { body } = res;
|
|
|
|
assert.equal(res.statusCode, 200);
|
|
assert.equal(res.headers['x-redirect'], '/user/herp/chats');
|
|
assert.equal(body, '/user/herp/chats');
|
|
});
|
|
|
|
it('should return 404 if user is not in room', async () => {
|
|
const data = await helpers.loginUser('baz', 'quuxquux');
|
|
const response = await request(`${nconf.get('url')}/api/user/baz/chats/${roomId}`, {
|
|
resolveWithFullResponse: true,
|
|
simple: false,
|
|
json: true,
|
|
jar: data.jar,
|
|
});
|
|
|
|
assert.equal(response.statusCode, 404);
|
|
});
|
|
});
|
|
});
|