Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-26 16:46:12 +01:00
Code Issues Packages Projects Releases Wiki Activity

refactor: get rid of async.waterfall/series

Browse Source
This commit is contained in:
Barış Soner Uşaklı
2023-09-13 16:23:49 -04:00
parent 12823ba77c
commit 8bc2ef227e
1 changed files with 32 additions and 96 deletions
Download Patch File Download Diff File Expand all files Collapse all files

116
test/groups.js
View File

@@ -1,7 +1,6 @@
'use strict';
const assert = require('assert');
const async = require('async');
const fs = require('fs');
const path = require('path');
const nconf = require('nconf');
@@ -161,33 +160,15 @@ describe('Groups', () => {
});
});
it('should return all users if no query', (done) => {
function createAndJoinGroup(username, email, callback) {
async.waterfall([
function (next) {
User.create({ username: username, email: email }, next);
},
function (uid, next) {
Groups.join('Test', uid, next);
},
], callback);
it('should return all users if no query', async () => {
async function createAndJoinGroup(username, email) {
const uid = await User.create({ username: username, email: email });
await Groups.join('Test', uid);
}
async.series([
function (next) {
createAndJoinGroup('newuser', 'newuser@b.com', next);
},
function (next) {
createAndJoinGroup('bob', 'bob@b.com', next);
},
], (err) => {
assert.ifError(err);
socketGroups.searchMembers({ uid: adminUid }, { groupName: 'Test', query: '' }, (err, data) => {
assert.ifError(err);
await createAndJoinGroup('newuser', 'newuser@b.com');
await createAndJoinGroup('bob', 'bob@b.com');
const data = await socketGroups.searchMembers({ uid: adminUid }, { groupName: 'Test', query: '' });
assert.equal(data.users.length, 3);
done();
});
});
});
it('should search group members', (done) => {
@@ -584,37 +565,20 @@ describe('Groups', () => {
});
});
it('should remove group from privilege groups', (done) => {
it('should remove group from privilege groups', async () => {
const privileges = require('../src/privileges');
const cid = 1;
const groupName = '1';
const uid = 1;
async.waterfall([
function (next) {
Groups.create({ name: groupName }, next);
},
function (groupData, next) {
privileges.categories.give(['groups:topics:create'], cid, groupName, next);
},
function (next) {
Groups.isMember(groupName, 'cid:1:privileges:groups:topics:create', next);
},
function (isMember, next) {
await Groups.create({ name: groupName });
await privileges.categories.give(['groups:topics:create'], cid, groupName);
let isMember = await Groups.isMember(groupName, 'cid:1:privileges:groups:topics:create');
assert(isMember);
Groups.destroy(groupName, next);
},
function (next) {
Groups.isMember(groupName, 'cid:1:privileges:groups:topics:create', next);
},
function (isMember, next) {
await Groups.destroy(groupName);
isMember = await Groups.isMember(groupName, 'cid:1:privileges:groups:topics:create');
assert(!isMember);
Groups.isMember(uid, 'registered-users', next);
},
function (isMember, next) {
isMember = await Groups.isMember(uid, 'registered-users');
assert(isMember);
next();
},
], done);
});
});
@@ -779,36 +743,19 @@ describe('Groups', () => {
});
describe('.leaveAllGroups()', () => {
it('should remove a user from all groups', (done) => {
Groups.leaveAllGroups(testUid, (err) => {
assert.ifError(err);
it('should remove a user from all groups', async () => {
await Groups.leaveAllGroups(testUid);
const groups = ['Test', 'Hidden'];
async.every(groups, (group, next) => {
Groups.isMember(testUid, group, (err, isMember) => {
next(err, !isMember);
});
}, (err, result) => {
assert.ifError(err);
assert(result);
done();
});
});
const isMembers = await Groups.isMemberOfGroups(testUid, groups);
assert(!isMembers.includes(true));
});
});
describe('.show()', () => {
it('should make a group visible', (done) => {
Groups.show('Test', function (err) {
assert.ifError(err);
assert.equal(arguments.length, 1);
db.isSortedSetMember('groups:visible:createtime', 'Test', (err, isMember) => {
assert.ifError(err);
it('should make a group visible', async () => {
await Groups.show('Test');
const isMember = await db.isSortedSetMember('groups:visible:createtime', 'Test');
assert.strictEqual(isMember, true);
done();
});
});
});
});
@@ -1262,22 +1209,11 @@ describe('Groups', () => {
let regularUid;
const logoPath = path.join(__dirname, '../test/files/test.png');
const imagePath = path.join(__dirname, '../test/files/groupcover.png');
before((done) => {
User.create({ username: 'regularuser', password: '123456' }, (err, uid) => {
assert.ifError(err);
regularUid = uid;
async.series([
function (next) {
Groups.join('Test', adminUid, next);
},
function (next) {
Groups.join('Test', regularUid, next);
},
function (next) {
helpers.copyFile(logoPath, imagePath, next);
},
], done);
});
before(async () => {
regularUid = await User.create({ username: 'regularuser', password: '123456' });
await Groups.join('Test', adminUid);
await Groups.join('Test', regularUid);
await helpers.copyFile(logoPath, imagePath);
});
it('should fail if user is not logged in or not owner', (done) => {