Rename method getRecommendations to getSimilar

The method on the Badge model `getRecommendations` was actually getting
similar badges based on the category of a badge. I'm about to implement
the *actual* `getRecommendations` function, which is based on a user's
earned badges history, as well as a handful of other rules.

models/badge.js

@@ -518,7 +518,7 @@ Badge.prototype.getRubricItems = function() {
          : [];
 };
 
-Badge.prototype.getRecommendations = function (email, callback) {
+Badge.prototype.getSimilar = function (email, callback) {
   const thisShortname = this.shortname;
   if (typeof email == 'function')
     callback = email, email = null;

routes/api.js

@@ -110,12 +110,12 @@ function parseLimit(userLimit) {
   return userLimit;
 }
 
-exports.recommendations = function recommendations(req, res, next) {
+exports.similarBadges = function similarBadges(req, res, next) {
   const badge = req.badge;
   const email = req.query.email;
   const limit = parseLimit(req.query.limit, 10);
 
-  badge.getRecommendations(email, function (err, badges) {
+  badge.getSimilar(email, function (err, badges) {
     if (err)
       return res.json(500, { status: 'error', error: err });
 

routes/index.js

@@ -174,9 +174,13 @@ exports.define = function defineRoutes(app) {
     findBadgeByParamShortname
   ], api.badge);
 
+  // 'similar' aliased to 'recommendations'
+  app.get('/v2/badge/:shortname/similar', [
+    findBadgeByParamShortname
+  ], api.similarBadges);
   app.get('/v2/badge/:shortname/recommendations', [
     findBadgeByParamShortname
-  ], api.recommendations);
+  ], api.similarBadges);
 
   app.get('/v2/unclaimed', [
     api.auth()

tests/api.test.js

@@ -300,7 +300,7 @@ test.applyFixtures(badgeFixtures, function(fx) {
 
   test('api can give badge recommendations', function(t) {
     conmock({
-      handler: api.recommendations,
+      handler: api.similarBadges,
       request: {
         badge: fx['science1']
       }
@@ -316,7 +316,7 @@ test.applyFixtures(badgeFixtures, function(fx) {
 
   test('api can give badge recommendations with a limit', function(t) {
     conmock({
-      handler: api.recommendations,
+      handler: api.similarBadges,
       request: {
         badge: fx['science1'],
         query: { limit: '2' },

tests/badge-recommendations.test.js

@@ -45,7 +45,7 @@ test.applyFixtures({
 
   test('Recommendation, no user', function (t) {
     const pureScience = fx['pure-science'];
-    pureScience.getRecommendations(function (err, badges) {
+    pureScience.getSimilar(function (err, badges) {
       const shortnames = badges.map(util.prop('shortname'));
       const hasMath = shortnames.some(has('pure-math'));
       const hasSelf = shortnames.some(has('pure-science'));
@@ -59,7 +59,7 @@ test.applyFixtures({
 
   test('Recommendation, with user', function (t) {
     const pureScience = fx['pure-science'];
-    pureScience.getRecommendations(TESTUSER, function (err, badges) {
+    pureScience.getSimilar(TESTUSER, function (err, badges) {
       const names = badges.map(util.prop('shortname'));
       const hasMath = badges.some(has('pure-math'));
       const hasEarned = badges.some(has('earned'));