breaking: remove socket.emit('posts.bookmark/unbookmark')

src/socket.io/posts.js

@@ -20,7 +20,6 @@ const SocketPosts = module.exports;
 require('./posts/edit')(SocketPosts);
 require('./posts/move')(SocketPosts);
 require('./posts/votes')(SocketPosts);
-require('./posts/bookmarks')(SocketPosts);
 require('./posts/tools')(SocketPosts);
 
 SocketPosts.reply = async function (socket, data) {

src/socket.io/posts/bookmarks.js

@@ -1,16 +0,0 @@
-'use strict';
-
-const sockets = require('..');
-const api = require('../../api');
-
-module.exports = function (SocketPosts) {
-	SocketPosts.bookmark = async function (socket, data) {
-		sockets.warnDeprecated(socket, 'PUT /api/v3/posts/:pid/bookmark');
-		return await api.posts.bookmark(socket, data);
-	};
-
-	SocketPosts.unbookmark = async function (socket, data) {
-		sockets.warnDeprecated(socket, 'DELETE /api/v3/posts/:pid/bookmark');
-		return await api.posts.unbookmark(socket, data);
-	};
-};

test/posts.js

@@ -282,28 +282,18 @@ describe('Post\'s', () => {
 	});
 
 	describe('bookmarking', () => {
-		it('should bookmark a post', (done) => {
-			socketPosts.bookmark({ uid: voterUid }, { pid: postData.pid, room_id: `topic_${postData.tid}` }, (err, data) => {
-				assert.ifError(err);
-				assert.equal(data.isBookmarked, true);
-				posts.hasBookmarked(postData.pid, voterUid, (err, hasBookmarked) => {
-					assert.ifError(err);
-					assert.equal(hasBookmarked, true);
-					done();
-				});
-			});
-		});
-
-		it('should unbookmark a post', (done) => {
-			socketPosts.unbookmark({ uid: voterUid }, { pid: postData.pid, room_id: `topic_${postData.tid}` }, (err, data) => {
-				assert.ifError(err);
-				assert.equal(data.isBookmarked, false);
-				posts.hasBookmarked([postData.pid], voterUid, (err, hasBookmarked) => {
-					assert.ifError(err);
-					assert.equal(hasBookmarked[0], false);
-					done();
-				});
-			});
+		it('should bookmark a post', async () => {
+			const data = await apiPosts.bookmark({ uid: voterUid }, { pid: postData.pid, room_id: `topic_${postData.tid}` });
+			assert.equal(data.isBookmarked, true);
+			const hasBookmarked = await posts.hasBookmarked(postData.pid, voterUid);
+			assert.equal(hasBookmarked, true);
+		});
+
+		it('should unbookmark a post', async () => {
+			const data = await apiPosts.unbookmark({ uid: voterUid }, { pid: postData.pid, room_id: `topic_${postData.tid}` });
+			assert.equal(data.isBookmarked, false);
+			const hasBookmarked = await posts.hasBookmarked([postData.pid], voterUid);
+			assert.equal(hasBookmarked[0], false);
 		});
 	});