derp

src/user/blocks.js

@@ -91,7 +91,7 @@ module.exports = function (User) {
 			set = property;
 			property = 'uid';
 		}
-
+console.log('derp')
 		if (!Array.isArray(set) || !set.length || !set.every((item) => {
 			if (!item) {
 				return false;
@@ -104,15 +104,15 @@ module.exports = function (User) {
 		}
 
 		const isPlain = typeof set[0] !== 'object';
-		User.blocks.list(uid, function (err, blocked_uids) {
+		const values = set.map(function (item) {
+			return parseInt(isPlain ? item : item[property], 10);
+		});
+
+		db.isSortedSetMembers('uid:' + uid + ':blocked_uids', values, function (err, isMembers) {
 			if (err) {
 				return callback(err);
 			}
-
-			set = set.filter(function (item) {
-				return !blocked_uids.includes(parseInt(isPlain ? item : item[property], 10));
-			});
-
+			set = set.filter((item, index) => !isMembers[index]);
 			callback(null, set);
 		});
 	};

test/controllers.js

@@ -1229,6 +1229,7 @@ describe('Controllers', function () {
 				function (res, body, next) {
 					assert.equal(res.statusCode, 200);
 					assert(body);
+					console.log(body);
 					var notif = body.notifications[0];
 					assert.equal(notif.bodyShort, notifData.bodyShort);
 					assert.equal(notif.bodyLong, notifData.bodyLong);

test/user.js

@@ -1823,12 +1823,15 @@ describe('User', function () {
 
 		describe('.add()', function () {
 			it('should block a uid', function (done) {
-				User.blocks.add(blockeeUid, 1, function (err, blocked_uids) {
+				User.blocks.add(blockeeUid, 1, function (err) {
 					assert.ifError(err);
-					assert.strictEqual(Array.isArray(blocked_uids), true);
-					assert.strictEqual(blocked_uids.length, 1);
-					assert.strictEqual(blocked_uids.includes(blockeeUid), true);
-					done();
+					User.blocks.list(1, function (err, blocked_uids) {
+						assert.ifError(err);
+						assert.strictEqual(Array.isArray(blocked_uids), true);
+						assert.strictEqual(blocked_uids.length, 1);
+						assert.strictEqual(blocked_uids.includes(blockeeUid), true);
+						done();
+					});
 				});
 			});
 
@@ -1850,11 +1853,14 @@ describe('User', function () {
 
 		describe('.remove()', function () {
 			it('should unblock a uid', function (done) {
-				User.blocks.remove(blockeeUid, 1, function (err, blocked_uids) {
+				User.blocks.remove(blockeeUid, 1, function (err) {
 					assert.ifError(err);
-					assert.strictEqual(Array.isArray(blocked_uids), true);
-					assert.strictEqual(blocked_uids.length, 0);
-					done();
+					User.blocks.list(1, function (err, blocked_uids) {
+						assert.ifError(err);
+						assert.strictEqual(Array.isArray(blocked_uids), true);
+						assert.strictEqual(blocked_uids.length, 0);
+						done();
+					});
 				});
 			});