add tests for top topics

fix popular page displaying 18 topics per page

src/categories/topics.js

@@ -57,7 +57,7 @@ module.exports = function (Categories) {
 			function (results, next) {
 				var totalPinnedCount = results.pinnedTids.length;
 
-				pinnedTids = results.pinnedTids.slice(data.start, data.stop === -1 ? undefined : data.stop + 1);
+				pinnedTids = results.pinnedTids.slice(data.start, data.stop !== -1 ? data.stop + 1 : undefined);
 
 				var pinnedCount = pinnedTids.length;
 

src/topics/popular.js

@@ -53,7 +53,7 @@ module.exports = function (Topics) {
 			function (topics, next) {
 				tids = topics.filter(function (topic) {
 					return topic && parseInt(topic.deleted, 10) !== 1;
-				}).sort(sortPopular).slice(start, stop !== -1 ? stop - 1 : undefined).map(function (topic) {
+				}).sort(sortPopular).slice(start, stop !== -1 ? stop + 1 : undefined).map(function (topic) {
 					return topic.tid;
 				});
 				privileges.topics.filterTids('read', tids, uid, next);

src/topics/recent.js

@@ -41,7 +41,7 @@ module.exports = function (Topics) {
 			},
 			function (tids, next) {
 				recentTopics.topicCount = tids.length;
-				tids = tids.slice(start, stop + 1);
+				tids = tids.slice(start, stop !== -1 ? stop + 1 : undefined);
 				Topics.getTopicsByTids(tids, uid, next);
 			},
 			function (topicData, next) {

src/topics/suggested.js

@@ -35,13 +35,7 @@ module.exports = function (Topics) {
 				}
 			},
 			function (categoryTids, next) {
-				tids = _.uniq(tids.concat(categoryTids));
-				if (stop === -1) {
-					tids = tids.slice(start);
-				} else {
-					tids = tids.slice(start, stop + 1);
-				}
-
+				tids = _.uniq(tids.concat(categoryTids)).slice(start, stop !== -1 ? stop + 1 : undefined);
 				Topics.getTopicsByTids(tids, uid, next);
 			},
 		], callback);

src/topics/top.js

@@ -33,7 +33,7 @@ module.exports = function (Topics) {
 			},
 			function (tids, next) {
 				topTopics.topicCount = tids.length;
-				tids = tids.slice(start, stop + 1);
+				tids = tids.slice(start, stop !== -1 ? stop + 1 : undefined);
 				Topics.getTopicsByTids(tids, uid, next);
 			},
 			function (topicData, next) {

src/topics/unread.js

@@ -42,11 +42,7 @@ module.exports = function (Topics) {
 					return next(null, []);
 				}
 
-				if (params.stop === -1) {
-					tids = tids.slice(params.start);
-				} else {
-					tids = tids.slice(params.start, params.stop + 1);
-				}
+				tids = tids.slice(params.start, params.stop !== -1 ? params.stop + 1 : undefined);
 
 				Topics.getTopicsByTids(tids, params.uid, next);
 			},

test/topics.js

@@ -1823,4 +1823,20 @@ describe('Topic\'s', function () {
 			], done);
 		});
 	});
+
+	describe('top topics', function () {
+		it('should get top topics in category', function (done) {
+			var filters = ['', 'watched', 'unreplied', 'new'];
+			async.map(filters, function (filter, next) {
+				topics.getTopTopics(topic.categoryId, topic.userId, 0, -1, filter, next);
+			}, function (err, data) {
+				assert.ifError(err);
+				assert(data);
+				data.forEach(function (filterTopics) {
+					assert(Array.isArray(filterTopics.topics));
+				});
+				done();
+			});
+		});
+	});
 });