mirror of
https://github.com/NodeBB/NodeBB.git
synced 2025-10-31 11:05:54 +01:00
breaking: remove deprecated groups socket calls
socket.emit('groups.join') removed use `PUT /api/v3/groups/:slug/membership/:uid`
socket.emit('groups.leave') removed use `DELETE /api/v3/groups/:slug/membership/:uid`
socket.emit('groups.grant') removed, `PUT /api/v3/groups/:slug/ownership/:uid`
socket.emit('groups.rescind') removed, `DELETE /api/v3/groups/:slug/ownership/:uid`
socket.emit('groups.update') removed, `PUT /api/v3/groups/:slug`
socket.emit('groups.create') removed, `POST /api/v3/groups`
socket.emit('groups.delete') removed, `DELETE /api/v3/groups`
This commit is contained in:
Barış Soner Uşaklı
@@ -5,8 +5,6 @@ const user = require('../user');
|
||||
const utils = require('../utils');
|
||||
const events = require('../events');
|
||||
const privileges = require('../privileges');
|
||||
const api = require('../api');
|
||||
const sockets = require('.');
|
||||
|
||||
const SocketGroups = module.exports;
|
||||
|
||||
@@ -16,18 +14,6 @@ SocketGroups.before = async (socket, method, data) => {
|
||||
}
|
||||
};
|
||||
|
||||
SocketGroups.join = async (socket, data) => {
|
||||
sockets.warnDeprecated(socket, 'PUT /api/v3/groups/:slug/membership/:uid');
|
||||
const slug = await groups.getGroupField(data.groupName, 'slug');
|
||||
await api.groups.join(socket, { slug: slug, uid: data.uid || socket.uid });
|
||||
};
|
||||
|
||||
SocketGroups.leave = async (socket, data) => {
|
||||
sockets.warnDeprecated(socket, 'DELETE /api/v3/groups/:slug/membership/:uid');
|
||||
const slug = await groups.getGroupField(data.groupName, 'slug');
|
||||
await api.groups.leave(socket, { slug: slug, uid: data.uid || socket.uid });
|
||||
};
|
||||
|
||||
SocketGroups.addMember = async (socket, data) => {
|
||||
await isOwner(socket, data);
|
||||
if (data.groupName === 'administrators' || groups.isPrivilegeGroup(data.groupName)) {
|
||||
@@ -78,28 +64,6 @@ async function isInvited(socket, data) {
|
||||
}
|
||||
}
|
||||
|
||||
SocketGroups.grant = async (socket, data) => {
|
||||
sockets.warnDeprecated(socket, 'PUT /api/v3/groups/:slug/ownership/:uid');
|
||||
|
||||
await isOwner(socket, data);
|
||||
await groups.ownership.grant(data.toUid, data.groupName);
|
||||
logGroupEvent(socket, 'group-owner-grant', {
|
||||
groupName: data.groupName,
|
||||
targetUid: data.toUid,
|
||||
});
|
||||
};
|
||||
|
||||
SocketGroups.rescind = async (socket, data) => {
|
||||
sockets.warnDeprecated(socket, 'DELETE /api/v3/groups/:slug/ownership/:uid');
|
||||
|
||||
await isOwner(socket, data);
|
||||
await groups.ownership.rescind(data.toUid, data.groupName);
|
||||
logGroupEvent(socket, 'group-owner-rescind', {
|
||||
groupName: data.groupName,
|
||||
targetUid: data.toUid,
|
||||
});
|
||||
};
|
||||
|
||||
SocketGroups.accept = async (socket, data) => {
|
||||
await isOwner(socket, data);
|
||||
await groups.acceptMembership(data.groupName, data.toUid);
|
||||
@@ -189,15 +153,6 @@ SocketGroups.rejectInvite = async (socket, data) => {
|
||||
});
|
||||
};
|
||||
|
||||
SocketGroups.update = async (socket, data) => {
|
||||
sockets.warnDeprecated(socket, 'PUT /api/v3/groups/:slug');
|
||||
await isOwner(socket, data);
|
||||
|
||||
const slug = await groups.getGroupField(data.groupName, 'slug');
|
||||
await api.groups.update(socket, { slug, ...data.values });
|
||||
};
|
||||
|
||||
|
||||
SocketGroups.kick = async (socket, data) => {
|
||||
await isOwner(socket, data);
|
||||
if (socket.uid === parseInt(data.uid, 10)) {
|
||||
@@ -212,18 +167,6 @@ SocketGroups.kick = async (socket, data) => {
|
||||
});
|
||||
};
|
||||
|
||||
SocketGroups.create = async (socket, data) => {
|
||||
sockets.warnDeprecated(socket, 'POST /api/v3/groups');
|
||||
const groupData = await api.groups.create(socket, data);
|
||||
return groupData;
|
||||
};
|
||||
|
||||
SocketGroups.delete = async (socket, data) => {
|
||||
sockets.warnDeprecated(socket, 'DEL /api/v3/groups');
|
||||
const slug = await groups.getGroupField(data.groupName, 'slug');
|
||||
await api.groups.delete(socket, { slug: slug });
|
||||
};
|
||||
|
||||
SocketGroups.search = async (socket, data) => {
|
||||
data.options = data.options || {};
|
||||
|
||||
|
||||
543
test/groups.js
543
test/groups.js
@@ -11,6 +11,7 @@ const helpers = require('./helpers');
|
||||
const Groups = require('../src/groups');
|
||||
const User = require('../src/user');
|
||||
const socketGroups = require('../src/socket.io/groups');
|
||||
const apiGroups = require('../src/api/groups');
|
||||
const meta = require('../src/meta');
|
||||
const navigation = require('../src/navigation/admin');
|
||||
|
||||
@@ -366,22 +367,19 @@ describe('Groups', () => {
|
||||
});
|
||||
});
|
||||
|
||||
it('should fail if group name is invalid', (done) => {
|
||||
socketGroups.create({ uid: adminUid }, { name: ['test', 'administrators'] }, (err) => {
|
||||
assert.equal(err.message, '[[error:invalid-group-name]]');
|
||||
done();
|
||||
});
|
||||
it('should fail if group name is invalid', async () => {
|
||||
try {
|
||||
await apiGroups.create({ uid: adminUid }, { name: ['test', 'administrators'] });
|
||||
} catch (err) {
|
||||
return assert.equal(err.message, '[[error:invalid-group-name]]');
|
||||
}
|
||||
assert(false);
|
||||
});
|
||||
|
||||
it('should not create a system group', (done) => {
|
||||
socketGroups.create({ uid: adminUid }, { name: 'mysystemgroup', system: true }, (err) => {
|
||||
assert.ifError(err);
|
||||
Groups.getGroupData('mysystemgroup', (err, data) => {
|
||||
assert.ifError(err);
|
||||
assert.strictEqual(data.system, 0);
|
||||
done();
|
||||
});
|
||||
});
|
||||
it('should not create a system group', async () => {
|
||||
await apiGroups.create({ uid: adminUid }, { name: 'mysystemgroup', system: true });
|
||||
const data = await Groups.getGroupData('mysystemgroup');
|
||||
assert.strictEqual(data.system, 0);
|
||||
});
|
||||
|
||||
it('should fail if group name is invalid', (done) => {
|
||||
@@ -466,60 +464,67 @@ describe('Groups', () => {
|
||||
});
|
||||
});
|
||||
|
||||
it('should fail to rename if group name is invalid', (done) => {
|
||||
socketGroups.update({ uid: adminUid }, { groupName: ['updateTestGroup?'], values: {} }, (err) => {
|
||||
assert.strictEqual(err.message, '[[error:invalid-group-name]]');
|
||||
done();
|
||||
});
|
||||
it('should fail to rename if group name is invalid', async () => {
|
||||
try {
|
||||
await apiGroups.update({ uid: adminUid }, { slug: ['updateTestGroup?'], values: {} });
|
||||
} catch (err) {
|
||||
return assert.strictEqual(err.message, '[[error:invalid-group-name]]');
|
||||
}
|
||||
assert(false);
|
||||
});
|
||||
|
||||
it('should fail to rename if group name is too short', (done) => {
|
||||
socketGroups.update({ uid: adminUid }, { groupName: 'updateTestGroup?', values: { name: '' } }, (err) => {
|
||||
assert.strictEqual(err.message, '[[error:group-name-too-short]]');
|
||||
done();
|
||||
});
|
||||
it('should fail to rename if group name is too short', async () => {
|
||||
try {
|
||||
const slug = await Groups.getGroupField('updateTestGroup?', 'slug');
|
||||
await apiGroups.update({ uid: adminUid }, { slug: slug, name: '' });
|
||||
} catch (err) {
|
||||
return assert.strictEqual(err.message, '[[error:group-name-too-short]]');
|
||||
}
|
||||
assert(false);
|
||||
});
|
||||
|
||||
it('should fail to rename if group name is invalid', (done) => {
|
||||
socketGroups.update({ uid: adminUid }, { groupName: 'updateTestGroup?', values: { name: ['invalid'] } }, (err) => {
|
||||
assert.strictEqual(err.message, '[[error:invalid-group-name]]');
|
||||
done();
|
||||
});
|
||||
it('should fail to rename if group name is invalid', async () => {
|
||||
try {
|
||||
const slug = await Groups.getGroupField('updateTestGroup?', 'slug');
|
||||
await apiGroups.update({ uid: adminUid }, { slug: slug, name: ['invalid'] });
|
||||
} catch (err) {
|
||||
return assert.strictEqual(err.message, '[[error:invalid-group-name]]');
|
||||
}
|
||||
assert(false);
|
||||
});
|
||||
|
||||
it('should fail to rename if group name is invalid', (done) => {
|
||||
socketGroups.update({ uid: adminUid }, { groupName: 'updateTestGroup?', values: { name: 'cid:0:privileges:ban' } }, (err) => {
|
||||
assert.strictEqual(err.message, '[[error:invalid-group-name]]');
|
||||
done();
|
||||
});
|
||||
it('should fail to rename if group name is invalid', async () => {
|
||||
try {
|
||||
const slug = await Groups.getGroupField('updateTestGroup?', 'slug');
|
||||
await apiGroups.update({ uid: adminUid }, { slug: slug, name: 'cid:0:privileges:ban' });
|
||||
} catch (err) {
|
||||
return assert.strictEqual(err.message, '[[error:invalid-group-name]]');
|
||||
}
|
||||
assert(false);
|
||||
});
|
||||
|
||||
it('should fail to rename if group name is too long', (done) => {
|
||||
socketGroups.update({ uid: adminUid }, { groupName: 'updateTestGroup?', values: { name: 'verylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstring' } }, (err) => {
|
||||
assert.strictEqual(err.message, '[[error:group-name-too-long]]');
|
||||
done();
|
||||
});
|
||||
it('should fail to rename if group name is too long', async () => {
|
||||
try {
|
||||
const slug = await Groups.getGroupField('updateTestGroup?', 'slug');
|
||||
await apiGroups.update({ uid: adminUid }, { slug: slug, name: 'verylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstringverylongstring' });
|
||||
} catch (err) {
|
||||
return assert.strictEqual(err.message, '[[error:group-name-too-long]]');
|
||||
}
|
||||
assert(false);
|
||||
});
|
||||
|
||||
it('should fail to rename if group name is invalid', (done) => {
|
||||
socketGroups.update({ uid: adminUid }, { groupName: 'updateTestGroup?', values: { name: 'test:test' } }, (err) => {
|
||||
assert.strictEqual(err.message, '[[error:invalid-group-name]]');
|
||||
done();
|
||||
});
|
||||
});
|
||||
|
||||
it('should fail to rename if group name is invalid', (done) => {
|
||||
socketGroups.update({ uid: adminUid }, { groupName: 'updateTestGroup?', values: { name: 'another/test' } }, (err) => {
|
||||
assert.strictEqual(err.message, '[[error:invalid-group-name]]');
|
||||
done();
|
||||
});
|
||||
});
|
||||
|
||||
it('should fail to rename if group name is invalid', (done) => {
|
||||
socketGroups.update({ uid: adminUid }, { groupName: 'updateTestGroup?', values: { name: '---' } }, (err) => {
|
||||
assert.strictEqual(err.message, '[[error:invalid-group-name]]');
|
||||
done();
|
||||
});
|
||||
it('should fail to rename if group name is invalid', async () => {
|
||||
const slug = await Groups.getGroupField('updateTestGroup?', 'slug');
|
||||
const invalidNames = ['test:test', 'another/test', '---'];
|
||||
for (const name of invalidNames) {
|
||||
try {
|
||||
// eslint-disable-next-line no-await-in-loop
|
||||
await apiGroups.update({ uid: adminUid }, { slug: slug, name: name });
|
||||
assert(false);
|
||||
} catch (err) {
|
||||
assert.strictEqual(err.message, '[[error:invalid-group-name]]');
|
||||
}
|
||||
}
|
||||
});
|
||||
|
||||
it('should fail to rename group to an existing group', (done) => {
|
||||
@@ -621,7 +626,7 @@ describe('Groups', () => {
|
||||
try {
|
||||
meta.config.allowPrivateGroups = false;
|
||||
const newUid = await User.create({ username: 'newadmin' });
|
||||
await socketGroups.join({ uid: newUid }, { groupName: ['test', 'administrators'], uid: newUid }, 1);
|
||||
await apiGroups.join({ uid: newUid }, { slug: ['test', 'administrators'], uid: newUid }, 1);
|
||||
const isMember = await Groups.isMember(newUid, 'administrators');
|
||||
assert(!isMember);
|
||||
} catch (err) {
|
||||
@@ -658,7 +663,8 @@ describe('Groups', () => {
|
||||
|
||||
it('should add user to Global Moderators group', async () => {
|
||||
const uid = await User.create({ username: 'glomod' });
|
||||
await socketGroups.join({ uid: adminUid }, { groupName: 'Global Moderators', uid: uid });
|
||||
const slug = await Groups.getGroupField('Global Moderators', 'slug');
|
||||
await apiGroups.join({ uid: adminUid }, { slug: slug, uid: uid });
|
||||
const isGlobalMod = await User.isGlobalModerator(uid);
|
||||
assert.strictEqual(isGlobalMod, true);
|
||||
});
|
||||
@@ -709,7 +715,8 @@ describe('Groups', () => {
|
||||
async function test(groupName) {
|
||||
let err;
|
||||
try {
|
||||
await socketGroups.join({ uid: uid }, { groupName: groupName });
|
||||
const slug = await Groups.getGroupField(groupName, 'slug');
|
||||
await apiGroups.join({ uid: uid }, { slug: slug, uid: uid });
|
||||
const isMember = await Groups.isMember(uid, groupName);
|
||||
assert.strictEqual(isMember, false);
|
||||
} catch (_err) {
|
||||
@@ -726,8 +733,7 @@ describe('Groups', () => {
|
||||
});
|
||||
|
||||
it('should allow admins to join private groups', async () => {
|
||||
const groupsAPI = require('../src/api/groups');
|
||||
await groupsAPI.join({ uid: adminUid }, { uid: adminUid, slug: 'global-moderators' });
|
||||
await apiGroups.join({ uid: adminUid }, { uid: adminUid, slug: 'global-moderators' });
|
||||
assert(await Groups.isMember(adminUid, 'Global Moderators'));
|
||||
});
|
||||
});
|
||||
@@ -810,102 +816,96 @@ describe('Groups', () => {
|
||||
});
|
||||
});
|
||||
|
||||
it('should return error if not logged in', (done) => {
|
||||
socketGroups.join({ uid: 0 }, {}, (err) => {
|
||||
it('should return error if not logged in', async () => {
|
||||
try {
|
||||
await apiGroups.join({ uid: 0 }, {});
|
||||
assert(false);
|
||||
} catch (err) {
|
||||
assert.equal(err.message, '[[error:invalid-uid]]');
|
||||
done();
|
||||
});
|
||||
}
|
||||
});
|
||||
|
||||
it('should return error if group name is special', (done) => {
|
||||
socketGroups.join({ uid: testUid }, { groupName: 'administrators' }, (err) => {
|
||||
it('should return error if group name is special', async () => {
|
||||
try {
|
||||
await apiGroups.join({ uid: testUid }, { slug: 'administrators', uid: testUid });
|
||||
assert(false);
|
||||
} catch (err) {
|
||||
assert.equal(err.message, '[[error:not-allowed]]');
|
||||
done();
|
||||
});
|
||||
}
|
||||
});
|
||||
|
||||
it('should error if group does not exist', (done) => {
|
||||
socketGroups.join({ uid: adminUid }, { groupName: 'doesnotexist' }, (err) => {
|
||||
it('should error if group does not exist', async () => {
|
||||
try {
|
||||
await apiGroups.join({ uid: adminUid }, { slug: 'doesnotexist', uid: adminUid });
|
||||
assert(false);
|
||||
} catch (err) {
|
||||
assert.equal(err.message, '[[error:no-group]]');
|
||||
done();
|
||||
});
|
||||
}
|
||||
});
|
||||
|
||||
it('should join test group', (done) => {
|
||||
it('should join test group', async () => {
|
||||
meta.config.allowPrivateGroups = 0;
|
||||
socketGroups.join({ uid: adminUid }, { groupName: 'Test' }, (err) => {
|
||||
assert.ifError(err);
|
||||
Groups.isMember(adminUid, 'Test', (err, isMember) => {
|
||||
assert.ifError(err);
|
||||
assert(isMember);
|
||||
done();
|
||||
});
|
||||
});
|
||||
await apiGroups.join({ uid: adminUid }, { slug: 'test', uid: adminUid });
|
||||
const isMember = await Groups.isMember(adminUid, 'Test');
|
||||
assert(isMember);
|
||||
});
|
||||
|
||||
it('should error if not logged in', (done) => {
|
||||
socketGroups.leave({ uid: 0 }, {}, (err) => {
|
||||
it('should error if not logged in', async () => {
|
||||
try {
|
||||
await apiGroups.leave({ uid: 0 }, {});
|
||||
assert(false);
|
||||
} catch (err) {
|
||||
assert.equal(err.message, '[[error:invalid-uid]]');
|
||||
done();
|
||||
});
|
||||
}
|
||||
});
|
||||
|
||||
it('should return error if group name is special', (done) => {
|
||||
socketGroups.leave({ uid: adminUid }, { groupName: 'administrators' }, (err) => {
|
||||
it('should return error if group name is special', async () => {
|
||||
try {
|
||||
await apiGroups.leave({ uid: adminUid }, { slug: 'administrators', uid: adminUid });
|
||||
assert(false);
|
||||
} catch (err) {
|
||||
assert.equal(err.message, '[[error:cant-remove-self-as-admin]]');
|
||||
done();
|
||||
});
|
||||
}
|
||||
});
|
||||
|
||||
it('should leave test group', (done) => {
|
||||
socketGroups.leave({ uid: adminUid }, { groupName: 'Test' }, (err) => {
|
||||
assert.ifError(err);
|
||||
Groups.isMember('Test', adminUid, (err, isMember) => {
|
||||
assert.ifError(err);
|
||||
assert(!isMember);
|
||||
done();
|
||||
});
|
||||
});
|
||||
it('should leave test group', async () => {
|
||||
await apiGroups.leave({ uid: adminUid }, { slug: 'test', uid: adminUid });
|
||||
const isMember = await Groups.isMember(adminUid, 'Test');
|
||||
assert(!isMember);
|
||||
});
|
||||
|
||||
it('should fail to join if group is private and join requests are disabled', (done) => {
|
||||
it('should fail to join if group is private and join requests are disabled', async () => {
|
||||
meta.config.allowPrivateGroups = 1;
|
||||
socketGroups.join({ uid: testUid }, { groupName: 'PrivateNoJoin' }, (err) => {
|
||||
try {
|
||||
await apiGroups.join({ uid: testUid }, { slug: 'privatenojoin', uid: testUid });
|
||||
assert(false);
|
||||
} catch (err) {
|
||||
assert.equal(err.message, '[[error:group-join-disabled]]');
|
||||
done();
|
||||
});
|
||||
}
|
||||
});
|
||||
|
||||
it('should fail to leave if group is private and leave is disabled', async () => {
|
||||
await socketGroups.join({ uid: testUid }, { groupName: 'PrivateNoLeave' });
|
||||
|
||||
await Groups.join('PrivateNoLeave', testUid);
|
||||
const isMember = await Groups.isMember(testUid, 'PrivateNoLeave');
|
||||
assert(isMember);
|
||||
try {
|
||||
await socketGroups.leave({ uid: testUid }, { groupName: 'PrivateNoLeave' });
|
||||
await apiGroups.leave({ uid: testUid }, { slug: 'privatenoleave', uid: testUid });
|
||||
assert(false);
|
||||
} catch (err) {
|
||||
assert.equal(err.message, '[[error:group-leave-disabled]]');
|
||||
}
|
||||
});
|
||||
|
||||
it('should join if user is admin', (done) => {
|
||||
socketGroups.join({ uid: adminUid }, { groupName: 'PrivateCanJoin' }, (err) => {
|
||||
assert.ifError(err);
|
||||
Groups.isMember(adminUid, 'PrivateCanJoin', (err, isMember) => {
|
||||
assert.ifError(err);
|
||||
assert(isMember);
|
||||
done();
|
||||
});
|
||||
});
|
||||
it('should join if user is admin', async () => {
|
||||
await apiGroups.join({ uid: adminUid }, { slug: 'privatecanjoin', uid: adminUid });
|
||||
const isMember = await Groups.isMember(adminUid, 'PrivateCanJoin');
|
||||
assert(isMember);
|
||||
});
|
||||
|
||||
it('should request membership for regular user', (done) => {
|
||||
socketGroups.join({ uid: testUid }, { groupName: 'PrivateCanJoin' }, (err) => {
|
||||
assert.ifError(err);
|
||||
Groups.isPending(testUid, 'PrivateCanJoin', (err, isPending) => {
|
||||
assert.ifError(err);
|
||||
assert(isPending);
|
||||
done();
|
||||
});
|
||||
});
|
||||
it('should request membership for regular user', async () => {
|
||||
await apiGroups.join({ uid: testUid }, { slug: 'privatecanjoin', uid: testUid });
|
||||
const isPending = await Groups.isPending(testUid, 'PrivateCanJoin');
|
||||
assert(isPending);
|
||||
});
|
||||
|
||||
it('should reject membership of user', (done) => {
|
||||
@@ -926,73 +926,30 @@ describe('Groups', () => {
|
||||
});
|
||||
});
|
||||
|
||||
it('should accept membership of user', (done) => {
|
||||
socketGroups.join({ uid: testUid }, { groupName: 'PrivateCanJoin' }, (err) => {
|
||||
assert.ifError(err);
|
||||
socketGroups.accept({ uid: adminUid }, { groupName: 'PrivateCanJoin', toUid: testUid }, (err) => {
|
||||
assert.ifError(err);
|
||||
Groups.isMember(testUid, 'PrivateCanJoin', (err, isMember) => {
|
||||
assert.ifError(err);
|
||||
assert(isMember);
|
||||
done();
|
||||
});
|
||||
});
|
||||
});
|
||||
it('should accept membership of user', async () => {
|
||||
await apiGroups.join({ uid: testUid }, { slug: 'privatecanjoin', uid: testUid });
|
||||
await socketGroups.accept({ uid: adminUid }, { groupName: 'PrivateCanJoin', toUid: testUid });
|
||||
const isMember = await Groups.isMember(testUid, 'PrivateCanJoin');
|
||||
assert(isMember);
|
||||
});
|
||||
|
||||
it('should reject/accept all memberships requests', (done) => {
|
||||
function requestMembership(uids, callback) {
|
||||
async.series([
|
||||
function (next) {
|
||||
socketGroups.join({ uid: uids.uid1 }, { groupName: 'PrivateCanJoin' }, next);
|
||||
},
|
||||
function (next) {
|
||||
socketGroups.join({ uid: uids.uid2 }, { groupName: 'PrivateCanJoin' }, next);
|
||||
},
|
||||
], (err) => {
|
||||
callback(err);
|
||||
});
|
||||
it('should reject/accept all memberships requests', async () => {
|
||||
async function requestMembership(uid1, uid2) {
|
||||
await apiGroups.join({ uid: uid1 }, { slug: 'privatecanjoin', uid: uid1 });
|
||||
await apiGroups.join({ uid: uid2 }, { slug: 'privatecanjoin', uid: uid2 });
|
||||
}
|
||||
let uids;
|
||||
async.waterfall([
|
||||
function (next) {
|
||||
async.parallel({
|
||||
uid1: function (next) {
|
||||
User.create({ username: 'groupuser1' }, next);
|
||||
},
|
||||
uid2: function (next) {
|
||||
User.create({ username: 'groupuser2' }, next);
|
||||
},
|
||||
}, next);
|
||||
},
|
||||
function (results, next) {
|
||||
uids = results;
|
||||
requestMembership(results, next);
|
||||
},
|
||||
function (next) {
|
||||
socketGroups.rejectAll({ uid: adminUid }, { groupName: 'PrivateCanJoin' }, next);
|
||||
},
|
||||
function (next) {
|
||||
Groups.getPending('PrivateCanJoin', next);
|
||||
},
|
||||
function (pending, next) {
|
||||
assert.equal(pending.length, 0);
|
||||
requestMembership(uids, next);
|
||||
},
|
||||
function (next) {
|
||||
socketGroups.acceptAll({ uid: adminUid }, { groupName: 'PrivateCanJoin' }, next);
|
||||
},
|
||||
function (next) {
|
||||
Groups.isMembers([uids.uid1, uids.uid2], 'PrivateCanJoin', next);
|
||||
},
|
||||
function (isMembers, next) {
|
||||
assert(isMembers[0]);
|
||||
assert(isMembers[1]);
|
||||
next();
|
||||
},
|
||||
], (err) => {
|
||||
done(err);
|
||||
});
|
||||
const [uid1, uid2] = await Promise.all([
|
||||
User.create({ username: 'groupuser1' }),
|
||||
User.create({ username: 'groupuser2' }),
|
||||
]);
|
||||
await requestMembership(uid1, uid2);
|
||||
await socketGroups.rejectAll({ uid: adminUid }, { groupName: 'PrivateCanJoin' });
|
||||
const pending = await Groups.getPending('PrivateCanJoin');
|
||||
assert.equal(pending.length, 0);
|
||||
await requestMembership(uid1, uid2);
|
||||
await socketGroups.acceptAll({ uid: adminUid }, { groupName: 'PrivateCanJoin' });
|
||||
const isMembers = await Groups.isMembers([uid1, uid2], 'PrivateCanJoin');
|
||||
assert.deepStrictEqual(isMembers, [true, true]);
|
||||
});
|
||||
|
||||
it('should issue invite to user', (done) => {
|
||||
@@ -1088,26 +1045,16 @@ describe('Groups', () => {
|
||||
});
|
||||
});
|
||||
|
||||
it('should grant ownership to user', (done) => {
|
||||
socketGroups.grant({ uid: adminUid }, { groupName: 'PrivateCanJoin', toUid: testUid }, (err) => {
|
||||
assert.ifError(err);
|
||||
Groups.ownership.isOwner(testUid, 'PrivateCanJoin', (err, isOwner) => {
|
||||
assert.ifError(err);
|
||||
assert(isOwner);
|
||||
done();
|
||||
});
|
||||
});
|
||||
it('should grant ownership to user', async () => {
|
||||
await apiGroups.grant({ uid: adminUid }, { slug: 'privatecanjoin', uid: testUid });
|
||||
const isOwner = await Groups.ownership.isOwner(testUid, 'PrivateCanJoin');
|
||||
assert(isOwner);
|
||||
});
|
||||
|
||||
it('should rescind ownership from user', (done) => {
|
||||
socketGroups.rescind({ uid: adminUid }, { groupName: 'PrivateCanJoin', toUid: testUid }, (err) => {
|
||||
assert.ifError(err);
|
||||
Groups.ownership.isOwner(testUid, 'PrivateCanJoin', (err, isOwner) => {
|
||||
assert.ifError(err);
|
||||
assert(!isOwner);
|
||||
done();
|
||||
});
|
||||
});
|
||||
it('should rescind ownership from user', async () => {
|
||||
await apiGroups.rescind({ uid: adminUid }, { slug: 'privatecanjoin', uid: testUid });
|
||||
const isOwner = await Groups.ownership.isOwner(testUid, 'PrivateCanJoin');
|
||||
assert(!isOwner);
|
||||
});
|
||||
|
||||
it('should fail to kick user with invalid data', (done) => {
|
||||
@@ -1128,116 +1075,108 @@ describe('Groups', () => {
|
||||
});
|
||||
});
|
||||
|
||||
it('should fail to create group with invalid data', (done) => {
|
||||
socketGroups.create({ uid: 0 }, {}, (err) => {
|
||||
it('should fail to create group with invalid data', async () => {
|
||||
try {
|
||||
await apiGroups.create({ uid: 0 }, {});
|
||||
assert(false);
|
||||
} catch (err) {
|
||||
assert.equal(err.message, '[[error:no-privileges]]');
|
||||
done();
|
||||
});
|
||||
}
|
||||
});
|
||||
|
||||
it('should fail to create group if group creation is disabled', (done) => {
|
||||
socketGroups.create({ uid: testUid }, { name: 'avalidname' }, (err) => {
|
||||
it('should fail to create group if group creation is disabled', async () => {
|
||||
try {
|
||||
await apiGroups.create({ uid: testUid }, { name: 'avalidname' });
|
||||
assert(false);
|
||||
} catch (err) {
|
||||
assert.equal(err.message, '[[error:no-privileges]]');
|
||||
done();
|
||||
});
|
||||
}
|
||||
});
|
||||
|
||||
it('should fail to create group if name is privilege group', (done) => {
|
||||
socketGroups.create({ uid: 1 }, { name: 'cid:1:privileges:groups:find' }, (err) => {
|
||||
it('should fail to create group if name is privilege group', async () => {
|
||||
try {
|
||||
await apiGroups.create({ uid: 1 }, { name: 'cid:1:privileges:groups:find' });
|
||||
assert(false);
|
||||
} catch (err) {
|
||||
assert.equal(err.message, '[[error:invalid-group-name]]');
|
||||
done();
|
||||
});
|
||||
}
|
||||
});
|
||||
|
||||
it('should create/update group', (done) => {
|
||||
socketGroups.create({ uid: adminUid }, { name: 'createupdategroup' }, (err, groupData) => {
|
||||
assert.ifError(err);
|
||||
assert(groupData);
|
||||
const data = {
|
||||
groupName: 'createupdategroup',
|
||||
values: {
|
||||
name: 'renamedupdategroup',
|
||||
description: 'cat group',
|
||||
userTitle: 'cats',
|
||||
userTitleEnabled: 1,
|
||||
disableJoinRequests: 1,
|
||||
hidden: 1,
|
||||
private: 0,
|
||||
},
|
||||
};
|
||||
socketGroups.update({ uid: adminUid }, data, (err) => {
|
||||
assert.ifError(err);
|
||||
Groups.get('renamedupdategroup', {}, (err, groupData) => {
|
||||
assert.ifError(err);
|
||||
assert.equal(groupData.name, 'renamedupdategroup');
|
||||
assert.equal(groupData.userTitle, 'cats');
|
||||
assert.equal(groupData.description, 'cat group');
|
||||
assert.equal(groupData.hidden, true);
|
||||
assert.equal(groupData.disableJoinRequests, true);
|
||||
assert.equal(groupData.private, false);
|
||||
done();
|
||||
});
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
it('should fail to create a group with name guests', (done) => {
|
||||
socketGroups.create({ uid: adminUid }, { name: 'guests' }, (err) => {
|
||||
assert.equal(err.message, '[[error:invalid-group-name]]');
|
||||
done();
|
||||
});
|
||||
});
|
||||
|
||||
it('should fail to rename guests group', (done) => {
|
||||
it('should create/update group', async () => {
|
||||
const groupData = await apiGroups.create({ uid: adminUid }, { name: 'createupdategroup' });
|
||||
assert(groupData);
|
||||
const data = {
|
||||
groupName: 'guests',
|
||||
values: {
|
||||
name: 'guests2',
|
||||
},
|
||||
slug: 'createupdategroup',
|
||||
name: 'renamedupdategroup',
|
||||
description: 'cat group',
|
||||
userTitle: 'cats',
|
||||
userTitleEnabled: 1,
|
||||
disableJoinRequests: 1,
|
||||
hidden: 1,
|
||||
private: 0,
|
||||
};
|
||||
socketGroups.update({ uid: adminUid }, data, (err) => {
|
||||
await apiGroups.update({ uid: adminUid }, data);
|
||||
const updatedData = await Groups.get('renamedupdategroup', {});
|
||||
assert.equal(updatedData.name, 'renamedupdategroup');
|
||||
assert.equal(updatedData.userTitle, 'cats');
|
||||
assert.equal(updatedData.description, 'cat group');
|
||||
assert.equal(updatedData.hidden, true);
|
||||
assert.equal(updatedData.disableJoinRequests, true);
|
||||
assert.equal(updatedData.private, false);
|
||||
});
|
||||
|
||||
it('should fail to create a group with name guests', async () => {
|
||||
try {
|
||||
await apiGroups.create({ uid: adminUid }, { name: 'guests' });
|
||||
assert(false);
|
||||
} catch (err) {
|
||||
assert.equal(err.message, '[[error:invalid-group-name]]');
|
||||
done();
|
||||
});
|
||||
}
|
||||
});
|
||||
|
||||
it('should delete group', (done) => {
|
||||
socketGroups.delete({ uid: adminUid }, { groupName: 'renamedupdategroup' }, (err) => {
|
||||
assert.ifError(err);
|
||||
Groups.exists('renamedupdategroup', (err, exists) => {
|
||||
assert.ifError(err);
|
||||
assert(!exists);
|
||||
done();
|
||||
});
|
||||
});
|
||||
});
|
||||
it('should fail to rename guests group', async () => {
|
||||
const data = {
|
||||
slug: 'guests',
|
||||
name: 'guests2',
|
||||
};
|
||||
|
||||
it('should fail to delete group if name is special', (done) => {
|
||||
socketGroups.delete({ uid: adminUid }, { groupName: 'administrators' }, (err) => {
|
||||
assert.equal(err.message, '[[error:not-allowed]]');
|
||||
done();
|
||||
});
|
||||
});
|
||||
|
||||
it('should fail to delete group if name is special', (done) => {
|
||||
socketGroups.delete({ uid: adminUid }, { groupName: 'registered-users' }, (err) => {
|
||||
assert.equal(err.message, '[[error:not-allowed]]');
|
||||
done();
|
||||
});
|
||||
});
|
||||
|
||||
it('should fail to delete group if name is special', (done) => {
|
||||
socketGroups.delete({ uid: adminUid }, { groupName: 'Global Moderators' }, (err) => {
|
||||
assert.equal(err.message, '[[error:not-allowed]]');
|
||||
done();
|
||||
});
|
||||
});
|
||||
|
||||
it('should fail to delete group if name is special', (done) => {
|
||||
socketGroups.delete({ uid: adminUid }, { groupName: 'guests' }, (err) => {
|
||||
try {
|
||||
await apiGroups.update({ uid: adminUid }, data);
|
||||
assert(false);
|
||||
} catch (err) {
|
||||
assert.equal(err.message, '[[error:invalid-group-name]]');
|
||||
done();
|
||||
});
|
||||
}
|
||||
});
|
||||
|
||||
it('should delete group', async () => {
|
||||
await apiGroups.delete({ uid: adminUid }, { slug: 'renamedupdategroup' });
|
||||
const exists = await Groups.exists('renamedupdategroup');
|
||||
assert(!exists);
|
||||
});
|
||||
|
||||
it('should fail to delete group if name is special', async () => {
|
||||
const specialGroups = [
|
||||
'administrators', 'registered-users', 'verified-users',
|
||||
'unverified-users', 'global-moderators',
|
||||
];
|
||||
for (const slug of specialGroups) {
|
||||
try {
|
||||
// eslint-disable-next-line no-await-in-loop
|
||||
await apiGroups.delete({ uid: adminUid }, { slug: slug });
|
||||
assert(false);
|
||||
} catch (err) {
|
||||
assert.equal(err.message, '[[error:not-allowed]]');
|
||||
}
|
||||
}
|
||||
});
|
||||
|
||||
it('should fail to delete group if name is special', async () => {
|
||||
try {
|
||||
await apiGroups.delete({ uid: adminUid }, { slug: 'guests' });
|
||||
assert(false);
|
||||
} catch (err) {
|
||||
assert.equal(err.message, '[[error:invalid-group-name]]');
|
||||
}
|
||||
});
|
||||
|
||||
it('should fail to load more groups with invalid data', (done) => {
|
||||
|
||||
Reference in New Issue
Block a user