3.78.0

.eslintignore

@@ -8,13 +8,16 @@ dist/
 dist-client/
 
 # Not linted
+migrations/*
 website/client-old/
+scripts/*
+test/server_side/**/*
 test/client-old/spec/**/*
 
 # Temporarilly disabled. These should be removed when the linting errors are fixed TODO
-migrations/*
-scripts/*
+website/common/script/content/index.js
 website/common/browserify.js
+test/content/**/*
 Gruntfile.js
 gulpfile.js
 gulp

config.json.example

@@ -73,7 +73,7 @@
     "LOGGLY_ACCOUNT": "account",
     "PUSH_CONFIGS": {
         "GCM_SERVER_API_KEY": "",
-        "APN_ENABLED": "false",
+        "APN_ENABLED": "true",
         "FCM_SERVER_API_KEY": ""
     },
     "PUSHER": {

gulp/gulp-build.js

@@ -25,7 +25,7 @@ gulp.task('build:common', () => {
 
 gulp.task('build:server', ['build:src', 'build:common']);
 
-gulp.task('build:dev', ['browserify', 'prepare:staticNewStuff'], (done) => {
+gulp.task('build:dev', ['browserify', 'prepare:staticNewStuff', 'semantic-ui'], (done) => {
   gulp.start('grunt-build:dev', done);
 });
 
@@ -33,7 +33,7 @@ gulp.task('build:dev:watch', ['build:dev'], () => {
   gulp.watch(['website/client-old/**/*.styl', 'website/common/script/*']);
 });
 
-gulp.task('build:prod', ['browserify', 'build:server', 'prepare:staticNewStuff'], (done) => {
+gulp.task('build:prod', ['browserify', 'build:server', 'prepare:staticNewStuff', 'semantic-ui'], (done) => {
   runSequence(
     'grunt-build:prod',
     'apidoc',

gulp/gulp-semanticui.js

@@ -0,0 +1,43 @@
+import gulp from 'gulp';
+import fs from 'fs';
+
+// Make semantic-ui-less work with a theme in a different folder
+// Code taken from https://www.artembutusov.com/webpack-semantic-ui/
+
+// Relative to node_modules/semantic-ui-less
+const SEMANTIC_THEME_PATH = '../../website/client/assets/less/semantic-ui/theme.config';
+
+// fix well known bug with default distribution
+function fixFontPath (filename) {
+  return new Promise((resolve, reject) => {
+    fs.readFile(filename, 'utf8', (err, content) => {
+      if (err) return reject(err);
+
+      let newContent = content.replace(
+        '@fontPath  : \'../../themes/',
+        '@fontPath  : \'../../../themes/'
+      );
+
+      fs.writeFile(filename, newContent, 'utf8', (err1) => {
+        if (err) return reject(err1);
+        resolve();
+      });
+    });
+  });
+}
+
+gulp.task('semantic-ui', (done) => {
+  // relocate default config
+  fs.writeFile(
+    'node_modules/semantic-ui-less/theme.config',
+    `@import '${SEMANTIC_THEME_PATH}';\n`,
+    'utf8',
+    (err) => {
+      if (err) return done(err);
+
+      fixFontPath('node_modules/semantic-ui-less/themes/default/globals/site.variables')
+        .then(() => done())
+        .catch(done);
+    }
+  );
+});

gulp/gulp-transifex-test.js

@@ -67,7 +67,7 @@ gulp.task('transifex:malformedStrings', () => {
   let stringsWithIncorrectNumberOfInterpolations = [];
 
   let count = 0;
-  _.each(ALL_LANGUAGES, function (lang) {
+  _(ALL_LANGUAGES).each(function (lang) {
 
     _.each(stringsToLookFor, function (strings, file) {
       let translationFile = fs.readFileSync(LOCALES + lang + '/' + file);
@@ -89,7 +89,7 @@ gulp.task('transifex:malformedStrings', () => {
         }
       });
     });
-  });
+  }).value();
 
   if (!_.isEmpty(stringsWithMalformedInterpolations)) {
     let message = 'The following strings have malformed or missing interpolations';
@@ -114,7 +114,7 @@ function getArrayOfLanguages () {
 function eachTranslationFile (languages, cb) {
   let jsonFiles = stripOutNonJsonFiles(fs.readdirSync(ENGLISH_LOCALE));
 
-  _.each(languages, (lang) => {
+  _(languages).each((lang) => {
     _.each(jsonFiles, (filename) => {
       try {
         var translationFile = fs.readFileSync(LOCALES + lang + '/' + filename);
@@ -128,7 +128,7 @@ function eachTranslationFile (languages, cb) {
 
       cb(null, lang, filename, parsedEnglishFile, parsedTranslationFile);
     });
-  });
+  }).value();
 }
 
 function eachTranslationString (languages, cb) {
@@ -153,7 +153,7 @@ function formatMessageForPosting (msg, items) {
 function getStringsWith (json, interpolationRegex) {
   var strings = {};
 
-  _.each(json, function (file_name) {
+  _(json).each(function (file_name) {
     var raw_file = fs.readFileSync(ENGLISH_LOCALE + file_name);
     var parsed_json = JSON.parse(raw_file);
 
@@ -162,7 +162,7 @@ function getStringsWith (json, interpolationRegex) {
       var match = value.match(interpolationRegex);
       if (match) strings[file_name][key] = match;
     });
-  });
+  }).value();
 
   return strings;
 }

gulpfile.js

@@ -9,6 +9,7 @@
 require('babel-register');
 
 if (process.env.NODE_ENV === 'production') {
+  require('./gulp/gulp-semanticui');
   require('./gulp/gulp-apidoc');
   require('./gulp/gulp-newstuff');
   require('./gulp/gulp-build');

migrations/20130204_count_habits.js

@@ -1,12 +1,5 @@
 // %mongo server:27017/dbname underscore.js my_commands.js
 // %mongo server:27017/dbname underscore.js --shell
-
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var habits = 0,
     dailies = 0,
     todos = 0,

migrations/20130326_migrate_pets.js

@@ -3,12 +3,6 @@
  */
 // mongo habitrpg ./node_modules/underscore/underscore.js ./migrations/20130326_migrate_pets.js
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var mapping = {
     bearcub: {name:'BearCub', modifier: 'Base'},
     cactus: {name:'Cactus', modifier:'Base'},

migrations/20130327_apply_tokens.js

@@ -4,12 +4,6 @@
 
 // mongo habitrpg ./node_modules/underscore/underscore.js migrations/20130327_apply_tokens.js
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var mapping = [
     {
         tier: 1,

migrations/20130508_fix_duff_party_subscriptions.js

@@ -6,11 +6,6 @@
  * mongo habitrpg ./node_modules/underscore/underscore.js ./migrations/20130508_fix_duff_party_subscriptions.js
  */
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
 
 // since our primary subscription will first hit parties now, we *definitely* need an index there
 db.parties.ensureIndex( { 'members': 1}, {background: true} );

migrations/20130602_survey_rewards.js

@@ -1,11 +1,5 @@
 //mongo habitrpg ./node_modules/lodash/lodash.js migrations/20130602_survey_rewards.js
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var members = []
 members = _.uniq(members);
 

migrations/20130908_cleanup_derby_corruption.js

@@ -3,12 +3,6 @@
 // Racer was notorious for adding duplicates, randomly deleting documents, etc. Once we pull the plug on old.habit,
 // run this migration to cleanup all the corruption
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 db.users.find().forEach(function(user){
 
   // remove corrupt tasks, which will either be null-value or no id

migrations/20131028_task_subdocs_tags_invites.js

@@ -5,12 +5,6 @@
 // @see http://stackoverflow.com/questions/14867697/mongoose-full-collection-scan
 //Also, what do we think of a Mongoose Migration module? something like https://github.com/madhums/mongoose-migrate
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 db.users.find().forEach(function(user){
 
   // Add invites to groups

migrations/20131104_restore_lost_task_data.js

@@ -8,12 +8,6 @@
 var mongo = require('mongoskin');
 var _ = require('lodash');
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var backupUsers = mongo.db('localhost:27017/habitrpg_old?auto_reconnect').collection('users');
 var liveUsers = mongo.db('localhost:27017/habitrpg_new?auto_reconnect').collection('users');
 

migrations/20131127_restore_dayStart.js

@@ -1,11 +1,5 @@
 // node .migrations/20131127_restore_dayStart.js
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var mongo = require('mongoskin');
 var _ = require('lodash');
 

migrations/20131214_classes.coffee

@@ -8,12 +8,6 @@ mongo = require('mongoskin')
 _ = require('lodash')
 async = require('async')
 
-# IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-# We've now upgraded to lodash v4 but the code used in this migration has not been
-# adapted to work with it. Before this migration is used again any lodash method should
-# be checked for compatibility against the v4 changelog and changed if necessary.
-# https://github.com/lodash/lodash/wiki/Changelog#v400
-
 db = mongo.db('localhost:27017/habitrpg?auto_reconnect')
 
 ###

migrations/20131221_restore_NaN_history.js

@@ -1,11 +1,5 @@
 // node .migrations/20131221_restore_NaN_history.js
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 /**
  * After the classes migration, users lost some history entries
  */

migrations/20131225_restore_streaks.js

@@ -1,11 +1,5 @@
 // node .migrations/20131225_restore_streaks.js
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 /**
  * After the classes migration, users lost some history entries
  */

migrations/20140823_remove_undefined_and_false_notifications.js

@@ -5,12 +5,6 @@ var migrationName = '20140823_remove_undefined_and_false_notifications';
 var authorName = 'Alys'; // in case script author needs to know when their ...
 var authorUuid = 'd904bd62-da08-416b-a816-ba797c9ee265'; //... own data is done
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 /**
  * https://github.com/HabitRPG/habitrpg/pull/3907
  */

migrations/20140829_change_headAccessory_to_eyewear.js

@@ -4,12 +4,6 @@ var migrationName = '20140829_change_headAccessory_to_eyewear';
 var authorName = 'Alys'; // in case script author needs to know when their ...
 var authorUuid = 'd904bd62-da08-416b-a816-ba797c9ee265'; //... own data is done
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 /**
  * https://github.com/HabitRPG/habitrpg/issues/3645
  */

migrations/20140831_increase_gems_for_previous_contributions.js

@@ -4,11 +4,6 @@
 //
 // node 20140831_increase_gems_for_previous_contributions.js  > 20140831_increase_gems_for_previous_contributions_output.txt
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
 
 var migrationName = '20140831_increase_gems_for_previous_contributions';
 

migrations/20140914_upgrade_admin_contrib_tiers.js

@@ -8,12 +8,6 @@ var authorUuid = 'd904bd62-da08-416b-a816-ba797c9ee265'; //... own data is done
  * Convert Tier 7 contributors with admin flag to Tier 8 (moderators).
  */
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var mongo = require('mongoskin');
 var _ = require('lodash');
 

migrations/20141117_consecutive_months.js

@@ -1,9 +1,3 @@
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 // require moment, lodash
 db.users.find(
   {'purchased.plan.customerId':{$ne:null}},

migrations/20150124_mountmaster_fix.js

@@ -9,12 +9,6 @@ var authorUuid = 'd904bd62-da08-416b-a816-ba797c9ee265'; //... own data is done
 
 var dbserver = 'localhost:27017' // CHANGE THIS FOR PRODUCTION DATABASE
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var mongo = require('mongoskin');
 var _ = require('lodash');
 

migrations/20150131_birthday_goodies_fix_remove_robe.js

@@ -8,12 +8,6 @@ var authorUuid = 'd904bd62-da08-416b-a816-ba797c9ee265'; //... own data is done
 
 var dbserver = 'localhost:27017' // CHANGE THIS FOR PRODUCTION DATABASE
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var mongo = require('mongoskin');
 var _ = require('lodash');
 

migrations/20150201_convert_creation_date_from_string_to_object.js

@@ -19,12 +19,6 @@ var authorUuid = 'd904bd62-da08-416b-a816-ba797c9ee265'; //... own data is done
 
 var dbserver = 'localhost:27017' // CHANGE THIS FOR PRODUCTION DATABASE
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var mongo = require('mongoskin');
 var _ = require('lodash');
 var moment = require('moment');

migrations/20150224_force_resting_in_inn.js

@@ -6,12 +6,6 @@ var authorUuid = 'd904bd62-da08-416b-a816-ba797c9ee265'; //... own data is done
  * force all active players to rest in the inn due to massive server fail
  */
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var dbserver = 'localhost:27017' // CHANGE THIS FOR PRODUCTION DATABASE
 
 var mongo = require('mongoskin');

migrations/20150604_ultimateGearSets.js

@@ -19,12 +19,6 @@ var authorUuid = 'd904bd62-da08-416b-a816-ba797c9ee265'; //... own data is done
  * means minimal new testing.
  */
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var dbserver = 'localhost:27017' // FOR TEST DATABASE
 // var dbserver = 'username:password@ds031379-a0.mongolab.com:31379' // FOR PRODUCTION DATABASE
 var dbname = 'habitrpg';

migrations/20160111_challenges_condense_same_day_history_entries.js

@@ -7,12 +7,6 @@ var migrationName = '20160111_challenges_condense_same_day_history_entries.js';
 var dbserver = '';
 var dbname = '';
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var mongo = require('mongoskin');
 var _ = require('lodash');
 var moment = require('moment');

migrations/20160129_habit_birthday.js

@@ -11,12 +11,6 @@ var dbserver = 'localhost:27017'; // FOR TEST DATABASE
 // var dbserver = 'username:password@ds031379-a0.mongolab.com:31379'; // FOR PRODUCTION DATABASE
 var dbname = 'habitrpg';
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var mongo = require('mongoskin');
 var _ = require('lodash');
 

migrations/20160521_veteran_ladder.js

@@ -14,12 +14,6 @@ var dbname = 'habitrpg';
 var mongo = require('mongoskin');
 var _ = require('lodash');
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var dbUsers = mongo.db(dbserver + '/' + dbname + '?auto_reconnect').collection('users');
 
 // specify a query to limit the affected users (empty for all users):

migrations/20160527_fix_empty_checklist_id.js

@@ -2,12 +2,6 @@ var uuid = require('uuid').v4;
 var mongo = require('mongodb').MongoClient;
 var _ = require('lodash');
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 var taskIds = require('checklists-no-id.json').map(function (obj) {
   return obj._id;
 });

migrations/api_v3/challenges.js

@@ -6,12 +6,6 @@
 
 // Due to some big user profiles it needs more RAM than is allowed by default by v8 (arounf 1.7GB).
 // Run the script with --max-old-space-size=4096 to allow up to 4GB of RAM
-
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
 console.log('Starting migrations/api_v3/challenges.js.');
 
 require('babel-register');

migrations/api_v3/challengesMembers.js

@@ -9,12 +9,6 @@
 // Run the script with --max-old-space-size=4096 to allow up to 4GB of RAM
 console.log('Starting migrations/api_v3/challengesMembers.js.');
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 require('babel-register');
 require('babel-polyfill');
 

migrations/api_v3/coupons.js

@@ -8,12 +8,6 @@
 // Run the script with --max-old-space-size=4096 to allow up to 4GB of RAM
 console.log('Starting migrations/api_v3/coupons.js.');
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 require('babel-register');
 require('babel-polyfill');
 

migrations/api_v3/emailUnsubscriptions.js

@@ -8,12 +8,6 @@
 // Run the script with --max-old-space-size=4096 to allow up to 4GB of RAM
 console.log('Starting migrations/api_v3/unsubscriptions.js.');
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 require('babel-register');
 require('babel-polyfill');
 

migrations/api_v3/groups.js

@@ -16,12 +16,6 @@
 // Run the script with --max-old-space-size=4096 to allow up to 4GB of RAM
 console.log('Starting migrations/api_v3/groups.js.');
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 require('babel-register');
 require('babel-polyfill');
 

migrations/api_v3/users.js

@@ -9,12 +9,6 @@
 // Run the script with --max-old-space-size=4096 to allow up to 4GB of RAM
 console.log('Starting migrations/api_v3/users.js.');
 
-// IMPORTANT NOTE: this migration was written when we were using version 3 of lodash.
-// We've now upgraded to lodash v4 but the code used in this migration has not been
-// adapted to work with it. Before this migration is used again any lodash method should
-// be checked for compatibility against the v4 changelog and changed if necessary.
-// https://github.com/lodash/lodash/wiki/Changelog#v400
-
 require('babel-register');
 require('babel-polyfill');
 

migrations/find_unique_user.js

@@ -7,6 +7,6 @@
 
 db.users.find().forEach(function(user){
   user.tasks = user.habits.concat(user.dailys).concat(user.todos).concat(user.rewards);
-  var found = _.some(user.tasks, {text: ""})
+  var found = _.any(user.tasks, {text: ""})
   if (found) printjson({id:user._id, auth:user.auth});
 })

migrations/groups/add-unlimited-subscription.js

@@ -1,33 +0,0 @@
-var migrationName = 'AddUnlimitedSubscription';
-var authorName = 'TheHollidayInn'; // in case script author needs to know when their ...
-var authorUuid = ''; //... own data is done
-
-/*
- * This migrations will add a free subscription to a specified group
- */
-
-import { model as Group } from '../../website/server/models/group';
-
-// @TODO: this should probably be a GroupManager library method
-async function addUnlimitedSubscription (groupId) {
-    let group = await Group.findById(groupId);
-
-    group.purchased.plan.customerId = "group-unlimited";
-    group.purchased.plan.dateCreated = new Date();
-    group.purchased.plan.dateUpdated = new Date();
-    group.purchased.plan.paymentMethod = "Group Unlimited";
-    group.purchased.plan.planId = "group_monthly";
-    group.purchased.plan.dateTerminated = null;
-    // group.purchased.plan.owner = ObjectId();
-    group.purchased.plan.subscriptionId = "";
-
-    return group.save();
-};
-
-module.exports = async function addUnlimitedSubscriptionCreator () {
-    let groupId = process.argv[2];
-
-    if (!groupId) throw Error('Group ID is required');
-
-    let result = await addUnlimitedSubscription(groupId)
-};

migrations/groups/create-group.js

@@ -1,32 +0,0 @@
-import Bluebird from 'bluebird';
-
-import { model as Group } from '../../website/server/models/group';
-import { model as User } from '../../website/server/models/user';
-
-// @TODO: this should probably be a GroupManager library method
-async function createGroup (name, privacy, type, leaderId) {
-    let user = await User.findById(leaderId);
-    
-    let group = new Group({
-        name,
-        privacy,
-        type,
-    });
-
-    group.leader = user._id;
-    user.guilds.push(group._id);
-
-    return Bluebird.all([group.save(), user.save()]);
-};
-
-module.exports = async function groupCreator () {
-    let name = process.argv[2];
-    let privacy = process.argv[3];
-    let type = process.argv[4];
-    let leaderId  = process.argv[5];
-
-    let result = await createGroup(name, privacy, type, leaderId)
-};
-
-
-

migrations/groups/habitrpg-jackalopes.js

@@ -1,46 +0,0 @@
-var migrationName = 'Jackalopes for Unlimited Subscribers';
-
-/*
- * This migration will find users with unlimited subscriptions who are also eligible for Jackalope mounts, and award them
- */
-import Bluebird from 'bluebird';
-
-import { model as Group } from '../../website/server/models/group';
-import { model as User } from '../../website/server/models/user';
-import * as payments from '../../website/server/libs/payments';
-
-async function handOutJackalopes () {
-  let promises = [];
-  let cursor = User.find({
-    'purchased.plan.customerId':'habitrpg',
-  }).cursor();
-
-  cursor.on('data', async function(user) {
-    console.log('User: ' + user._id);
-
-    let groupList = [];
-    if (user.party._id) groupList.push(user.party._id);
-    groupList = groupList.concat(user.guilds);
-
-    let subscribedGroup =
-      await Group.findOne({
-        '_id': {$in: groupList},
-        'purchased.plan.planId': 'group_monthly',
-        'purchased.plan.dateTerminated': null,
-      },
-      {'_id':1}
-    );
-
-    if (subscribedGroup) {
-      User.update({'_id':user._id},{$set:{'items.mounts.Jackalope-RoyalPurple':true}}).exec();
-      promises.push(user.save());
-    }
-  });
-
-  cursor.on('close', async function() {
-    console.log('done');
-    return await Bluebird.all(promises);
-  });
-};
-
-module.exports = handOutJackalopes;

migrations/groups/update-groups-with-group-plans.js

@@ -1,33 +0,0 @@
-var migrationName = 'ResyncGroupPlanMembers';
-var authorName = 'TheHollidayInn'; // in case script author needs to know when their ...
-var authorUuid = ''; //... own data is done
-
-/*
- * This migrations will iterate through all groups with a group plan a subscription and resync the free
- * subscription to all members
- */
-
-import Bluebird from 'bluebird';
-
-import { model as Group } from '../../website/server/models/group';
-import * as payments from '../../website/server/libs/payments';
-
-async function updateGroupsWithGroupPlans () {
-  let cursor = Group.find({
-    'purchased.plan.planId': 'group_monthly',
-    'purchased.plan.dateTerminated': null,
-  }).cursor();
-
-  let promises = [];
-
-  cursor.on('data', function(group) {
-    promises.push(payments.addSubscriptionToGroupUsers(group));
-    promises.push(group.save())
-  });
-
-  cursor.on('close', async function() {
-    return await Bluebird.all(promises);
-  });
-};
-
-module.exports = updateGroupsWithGroupPlans;

migrations/manual_password_reset.js

@@ -1,9 +1,6 @@
 // EMAIL="x@y.com" node ./migrations/manual_password_reset.js
 // Be sure to have PRODUCTION_DB in your config.json
 
-// IMPORTANT: this script isn't updated to use the new password encryption that uses bcrypt
-// using it will break accounts and should not be used until upgraded
-
 var nconf = require('nconf'),
   path = require('path');
 nconf.argv().env().file('user', path.join(path.resolve(__dirname, '../config.json')));

migrations/migration-runner.js

@@ -1,24 +1,21 @@
-require("babel-register");
-require("babel-polyfill");
-
-// This file must use ES5, everything required can be in ES6
-
-function setUpServer () {
-  var nconf = require('nconf');
-  var mongoose = require('mongoose');
-  var Bluebird = require('bluebird');
-  var setupNconf = require('../website/server/libs/setupNconf');
-  setupNconf();
-  // We require src/server and npt src/index because
-  // 1. nconf is already setup
-  // 2. we don't need clustering
-  require('../website/server/server'); // eslint-disable-line global-require
-}
-setUpServer();
-
-// Replace this with your migration
-var processUsers = require('./groups/update-groups-with-group-plans');
-processUsers()
-  .catch(function (err) {
-      console.log(err)
-  })
+require("babel-register");
+require("babel-polyfill");
+
+// This file must use ES5, everything required can be in ES6
+
+function setUpServer () {
+  var nconf = require('nconf');
+  var mongoose = require('mongoose');
+  var Bluebird = require('bluebird');
+  var setupNconf = require('../website/server/libs/setupNconf');
+  setupNconf();
+  // We require src/server and npt src/index because
+  // 1. nconf is already setup
+  // 2. we don't need clustering
+  require('../website/server/server'); // eslint-disable-line global-require
+}
+setUpServer();
+
+// Replace this with your migration
+var processUsers = require('./new_stuff');
+processUsers();

package.json

@@ -1,7 +1,7 @@
 {
   "name": "habitica",
   "description": "A habit tracker app which treats your goals like a Role Playing Game.",
-  "version": "3.79.2",
+  "version": "3.78.0",
   "main": "./website/server/index.js",
   "dependencies": {
     "@slack/client": "^3.8.1",
@@ -28,7 +28,6 @@
     "bcrypt": "^1.0.2",
     "bluebird": "^3.3.5",
     "body-parser": "^1.15.0",
-    "bootstrap": "^4.0.0-alpha.6",
     "bower": "~1.3.12",
     "browserify": "~12.0.1",
     "compression": "^1.6.1",
@@ -71,18 +70,20 @@
     "jade": "~1.11.0",
     "jquery": "^3.1.1",
     "js2xmlparser": "~1.0.0",
-    "lodash": "^4.17.4",
+    "less": "^2.7.1",
+    "less-loader": "^2.2.3",
+    "lodash": "^3.10.1",
+    "lodash.pickby": "^4.2.0",
+    "lodash.setwith": "^4.2.0",
     "merge-stream": "^1.0.0",
     "method-override": "^2.3.5",
     "moment": "^2.13.0",
-    "moment-recur": "habitrpg/moment-recur#v1.0.6",
-    "mongoose": "^4.8.6",
+    "mongoose": "^4.7.1",
     "mongoose-id-autoinc": "~2013.7.14-4",
     "morgan": "^1.7.0",
     "nconf": "~0.8.2",
     "nib": "^1.1.0",
     "node-gcm": "^0.14.4",
-    "node-sass": "^4.5.0",
     "nodemailer": "^2.3.2",
     "object-path": "^0.9.2",
     "ora": "^1.1.0",
@@ -102,7 +103,7 @@
     "rimraf": "^2.4.3",
     "run-sequence": "^1.1.4",
     "s3-upload-stream": "^1.0.6",
-    "sass-loader": "^6.0.2",
+    "semantic-ui-less": "~2.2.4",
     "serve-favicon": "^2.3.0",
     "shelljs": "^0.7.6",
     "stripe": "^4.2.0",
@@ -170,7 +171,7 @@
     "csv": "~0.3.6",
     "deep-diff": "~0.1.4",
     "eslint": "^3.0.0",
-    "eslint-config-habitrpg": "^3.0.0",
+    "eslint-config-habitrpg": "^2.0.0",
     "eslint-friendly-formatter": "^2.0.5",
     "eslint-loader": "^1.3.0",
     "eslint-plugin-html": "^2.0.0",
@@ -190,7 +191,6 @@
     "karma-mocha-reporter": "^1.1.1",
     "karma-phantomjs-launcher": "^1.0.0",
     "karma-sinon-chai": "^1.2.0",
-    "karma-sinon-stub-promise": "^1.0.0",
     "karma-sourcemap-loader": "^0.3.7",
     "karma-spec-reporter": "0.0.24",
     "karma-webpack": "^2.0.2",

test/api/v3/integration/challenges/POST-challenges_challengeId_leave.test.js

@@ -117,6 +117,7 @@ describe('POST /challenges/:challengeId/leave', () => {
       });
 
       expect(testTask).to.not.be.undefined;
+      expect(testTask.challenge).to.eql({});
     });
   });
 });

test/api/v3/integration/groups/GET-groups_groupId_members.test.js

@@ -84,7 +84,7 @@ describe('GET /groups/:groupId/members', () => {
     expect(Object.keys(memberRes.auth)).to.eql(['timestamps']);
     expect(Object.keys(memberRes.preferences).sort()).to.eql([
       'size', 'hair', 'skin', 'shirt',
-      'chair', 'costume', 'sleep', 'background', 'tasks',
+      'chair', 'costume', 'sleep', 'background',
     ].sort());
 
     expect(memberRes.stats.maxMP).to.exist;

test/api/v3/integration/groups/POST-groups_groupId_leave.js

@@ -78,7 +78,7 @@ describe('POST /groups/:groupId/leave', () => {
         expect(leader.newMessages[groupToLeave._id]).to.be.empty;
       });
 
-      context('with challenges', () => {
+      context('With challenges', () => {
         let challenge;
 
         beforeEach(async () => {
@@ -106,25 +106,10 @@ describe('POST /groups/:groupId/leave', () => {
 
           let userWithChallengeTasks = await leader.get('/user');
 
+          expect(userWithChallengeTasks.challenges).to.not.include(challenge._id);
           // @TODO find elegant way to assert against the task existing
           expect(userWithChallengeTasks.tasksOrder.habits).to.not.be.empty;
         });
-
-        it('keeps the user in the challenge when the keepChallenges parameter is set to remain-in-challenges', async () => {
-          await leader.post(`/groups/${groupToLeave._id}/leave`, {keepChallenges: 'remain-in-challenges'});
-
-          let userWithChallengeTasks = await leader.get('/user');
-
-          expect(userWithChallengeTasks.challenges).to.include(challenge._id);
-        });
-
-        it('drops the user in the challenge when the keepChallenges parameter isn\'t set', async () => {
-          await leader.post(`/groups/${groupToLeave._id}/leave`);
-
-          let userWithChallengeTasks = await leader.get('/user');
-
-          expect(userWithChallengeTasks.challenges).to.not.include(challenge._id);
-        });
       });
 
       it('prevents quest leader from leaving a groupToLeave');

test/api/v3/integration/groups/POST-groups_invite.test.js

@@ -1,6 +1,5 @@
 import {
   generateUser,
-  generateGroup,
   translate as t,
 } from '../../../../helpers/api-v3-integration.helper';
 import { v4 as generateUUID } from 'uuid';
@@ -13,7 +12,7 @@ describe('Post /groups/:groupId/invite', () => {
   let groupName = 'Test Public Guild';
 
   beforeEach(async () => {
-    inviter = await generateUser({balance: 4});
+    inviter = await generateUser({balance: 1});
     group = await inviter.post('/groups', {
       name: groupName,
       type: 'guild',
@@ -266,25 +265,6 @@ describe('Post /groups/:groupId/invite', () => {
       expect(invitedUser.invitations.guilds[0].id).to.equal(group._id);
       expect(invite).to.exist;
     });
-
-    it('invites marks invite with cancelled plan', async () => {
-      let cancelledPlanGroup = await generateGroup(inviter, {
-        type: 'guild',
-        name: generateUUID(),
-      });
-      await cancelledPlanGroup.createCancelledSubscription();
-
-      let newUser = await generateUser();
-      let invite = await inviter.post(`/groups/${cancelledPlanGroup._id}/invite`, {
-        uuids: [newUser._id],
-        emails: [{name: 'test', email: 'test@habitica.com'}],
-      });
-      let invitedUser = await newUser.get('/user');
-
-      expect(invitedUser.invitations.guilds[0].id).to.equal(cancelledPlanGroup._id);
-      expect(invitedUser.invitations.guilds[0].cancelledPlan).to.be.true;
-      expect(invite).to.exist;
-    });
   });
 
   describe('guild invites', () => {

test/api/v3/integration/groups/PUT-groups.test.js

@@ -43,15 +43,4 @@ describe('PUT /group', () => {
     expect(updatedGroup.leader.profile.name).to.eql(leader.profile.name);
     expect(updatedGroup.name).to.equal(groupUpdatedName);
   });
-
-  it('allows a leader to change leaders', async () => {
-    let updatedGroup = await leader.put(`/groups/${groupToUpdate._id}`, {
-      name: groupUpdatedName,
-      leader: nonLeader._id,
-    });
-
-    expect(updatedGroup.leader._id).to.eql(nonLeader._id);
-    expect(updatedGroup.leader.profile.name).to.eql(nonLeader.profile.name);
-    expect(updatedGroup.name).to.equal(groupUpdatedName);
-  });
 });

test/api/v3/integration/members/GET-members_id.test.js

@@ -37,7 +37,7 @@ describe('GET /members/:memberId', () => {
     expect(Object.keys(memberRes.auth)).to.eql(['timestamps']);
     expect(Object.keys(memberRes.preferences).sort()).to.eql([
       'size', 'hair', 'skin', 'shirt',
-      'chair', 'costume', 'sleep', 'background', 'tasks',
+      'chair', 'costume', 'sleep', 'background',
     ].sort());
 
     expect(memberRes.stats.maxMP).to.exist;

test/api/v3/integration/tasks/POST-tasks_id_score_direction.test.js

@@ -315,30 +315,6 @@ describe('POST /tasks/:id/score/:direction', () => {
 
       expect(updatedUser.stats.gp).to.be.greaterThan(user.stats.gp);
     });
-
-    it('adds score notes to task', async () => {
-      let scoreNotesString = 'test-notes';
-
-      await user.post(`/tasks/${habit._id}/score/up`, {
-        scoreNotes: scoreNotesString,
-      });
-      let updatedTask = await user.get(`/tasks/${habit._id}`);
-
-      expect(updatedTask.history[0].scoreNotes).to.eql(scoreNotesString);
-    });
-
-    it('errors when score notes are too large', async () => {
-      let scoreNotesString = new Array(258).join('a');
-
-      await expect(user.post(`/tasks/${habit._id}/score/up`, {
-        scoreNotes: scoreNotesString,
-      }))
-      .to.eventually.be.rejected.and.eql({
-        code: 401,
-        error: 'NotAuthorized',
-        message: t('taskScoreNotesTooLong'),
-      });
-    });
   });
 
   context('reward', () => {

test/api/v3/integration/tasks/POST-tasks_user.test.js

@@ -496,8 +496,6 @@ describe('POST /tasks/user', () => {
         frequency: 'daily',
         everyX: 5,
         startDate: now,
-        daysOfMonth: [15],
-        weeksOfMonth: [3],
       });
 
       expect(task.userId).to.equal(user._id);
@@ -506,8 +504,6 @@ describe('POST /tasks/user', () => {
       expect(task.type).to.eql('daily');
       expect(task.frequency).to.eql('daily');
       expect(task.everyX).to.eql(5);
-      expect(task.daysOfMonth).to.eql([15]);
-      expect(task.weeksOfMonth).to.eql([3]);
       expect(new Date(task.startDate)).to.eql(now);
     });
 

test/api/v3/integration/user/DELETE-user.test.js

@@ -3,7 +3,6 @@ import {
   createAndPopulateGroup,
   generateGroup,
   generateUser,
-  generateChallenge,
   translate as t,
 } from '../../../../helpers/api-integration/v3';
 import {
@@ -65,30 +64,6 @@ describe('DELETE /user', () => {
     }));
   });
 
-  it('reduces memberCount in challenges user is linked to', async () => {
-    let populatedGroup = await createAndPopulateGroup({
-      members: 2,
-    });
-
-    let group = populatedGroup.group;
-    let authorizedUser = populatedGroup.members[1];
-
-    let challenge = await generateChallenge(populatedGroup.groupLeader, group);
-    await authorizedUser.post(`/challenges/${challenge._id}/join`);
-
-    await challenge.sync();
-
-    expect(challenge.memberCount).to.eql(2);
-
-    await authorizedUser.del('/user', {
-      password,
-    });
-
-    await challenge.sync();
-
-    expect(challenge.memberCount).to.eql(1);
-  });
-
   it('deletes the user', async () => {
     await user.del('/user', {
       password,

test/api/v3/integration/user/auth/POST-register_local.test.js

@@ -5,7 +5,6 @@ import {
   createAndPopulateGroup,
   getProperty,
 } from '../../../../../helpers/api-integration/v3';
-import { ApiUser } from '../../../../../helpers/api-integration/api-classes';
 import { v4 as generateRandomUserName } from 'uuid';
 import { each } from 'lodash';
 import { encrypt } from '../../../../../../website/server/libs/encryption';
@@ -417,37 +416,5 @@ describe('POST /user/auth/local/register', () => {
 
       expect(user.tags).to.not.be.empty;
     });
-
-    it('adds the correct tags to the correct tasks', async () => {
-      let user = await api.post('/user/auth/local/register', {
-        username,
-        email,
-        password,
-        confirmPassword: password,
-      });
-
-      let requests = new ApiUser(user);
-
-      let habits = await requests.get('/tasks/user?type=habits');
-      let todos = await requests.get('/tasks/user?type=todos');
-
-      function findTag (tagName) {
-        let tag = user.tags.find((userTag) => {
-          return userTag.name === t(tagName);
-        });
-        return tag.id;
-      }
-
-      expect(habits[0].tags).to.have.a.lengthOf(3);
-      expect(habits[0].tags).to.include.members(['defaultTag1', 'defaultTag4', 'defaultTag6'].map(findTag));
-
-      expect(habits[1].tags).to.have.a.lengthOf(1);
-      expect(habits[1].tags).to.include.members(['defaultTag3'].map(findTag));
-
-      expect(habits[2].tags).to.have.a.lengthOf(2);
-      expect(habits[2].tags).to.include.members(['defaultTag2', 'defaultTag3'].map(findTag));
-
-      expect(todos[0].tags).to.have.a.lengthOf(0);
-    });
   });
 });

test/api/v3/unit/libs/amazonPayments.test.js

@@ -1,12 +1,10 @@
 import moment from 'moment';
 import cc from 'coupon-code';
-import uuid from 'uuid';
 
 import {
   generateGroup,
 } from '../../../../helpers/api-unit.helper.js';
 import { model as User } from '../../../../../website/server/models/user';
-import { model as Group } from '../../../../../website/server/models/group';
 import { model as Coupon } from '../../../../../website/server/models/coupon';
 import amzLib from '../../../../../website/server/libs/amazonPayments';
 import payments from '../../../../../website/server/libs/payments';
@@ -107,7 +105,7 @@ describe('Amazon Payments', () => {
       expect(paymentBuyGemsStub).to.be.calledOnce;
       expect(paymentBuyGemsStub).to.be.calledWith({
         user,
-        paymentMethod: amzLib.constants.PAYMENT_METHOD,
+        paymentMethod: amzLib.constants.PAYMENT_METHOD_AMAZON,
         headers,
       });
       expectAmazonStubs();
@@ -130,7 +128,7 @@ describe('Amazon Payments', () => {
       expect(paymentBuyGemsStub).to.be.calledOnce;
       expect(paymentBuyGemsStub).to.be.calledWith({
         user,
-        paymentMethod: amzLib.constants.PAYMENT_METHOD_GIFT,
+        paymentMethod: amzLib.constants.PAYMENT_METHOD_AMAZON_GIFT,
         headers,
         gift,
       });
@@ -155,7 +153,7 @@ describe('Amazon Payments', () => {
       expect(paymentCreateSubscritionStub).to.be.calledOnce;
       expect(paymentCreateSubscritionStub).to.be.calledWith({
         user,
-        paymentMethod: amzLib.constants.PAYMENT_METHOD_GIFT,
+        paymentMethod: amzLib.constants.PAYMENT_METHOD_AMAZON_GIFT,
         headers,
         gift,
       });
@@ -318,7 +316,7 @@ describe('Amazon Payments', () => {
       expect(createSubSpy).to.be.calledWith({
         user,
         customerId: billingAgreementId,
-        paymentMethod: amzLib.constants.PAYMENT_METHOD,
+        paymentMethod: amzLib.constants.PAYMENT_METHOD_AMAZON,
         sub,
         headers,
         groupId,
@@ -377,7 +375,7 @@ describe('Amazon Payments', () => {
       expect(createSubSpy).to.be.calledWith({
         user,
         customerId: billingAgreementId,
-        paymentMethod: amzLib.constants.PAYMENT_METHOD,
+        paymentMethod: amzLib.constants.PAYMENT_METHOD_AMAZON,
         sub,
         headers,
         groupId,
@@ -457,7 +455,7 @@ describe('Amazon Payments', () => {
         user,
         groupId: undefined,
         nextBill: moment(user.purchased.plan.lastBillingDate).add({ days: subscriptionLength }),
-        paymentMethod: amzLib.constants.PAYMENT_METHOD,
+        paymentMethod: amzLib.constants.PAYMENT_METHOD_AMAZON,
         headers,
       });
       expectAmazonStubs();
@@ -487,7 +485,7 @@ describe('Amazon Payments', () => {
         user,
         groupId: undefined,
         nextBill: moment(user.purchased.plan.lastBillingDate).add({ days: subscriptionLength }),
-        paymentMethod: amzLib.constants.PAYMENT_METHOD,
+        paymentMethod: amzLib.constants.PAYMENT_METHOD_AMAZON,
         headers,
       });
       amzLib.closeBillingAgreement.restore();
@@ -525,7 +523,7 @@ describe('Amazon Payments', () => {
         user,
         groupId: group._id,
         nextBill: moment(group.purchased.plan.lastBillingDate).add({ days: subscriptionLength }),
-        paymentMethod: amzLib.constants.PAYMENT_METHOD,
+        paymentMethod: amzLib.constants.PAYMENT_METHOD_AMAZON,
         headers,
       });
       expectAmazonStubs();
@@ -555,84 +553,10 @@ describe('Amazon Payments', () => {
         user,
         groupId: group._id,
         nextBill: moment(group.purchased.plan.lastBillingDate).add({ days: subscriptionLength }),
-        paymentMethod: amzLib.constants.PAYMENT_METHOD,
+        paymentMethod: amzLib.constants.PAYMENT_METHOD_AMAZON,
         headers,
       });
       amzLib.closeBillingAgreement.restore();
     });
   });
-
-  describe('#upgradeGroupPlan', () => {
-    let spy, data, user, group, uuidString;
-
-    beforeEach(async function () {
-      user = new User();
-      user.profile.name = 'sender';
-
-      data = {
-        user,
-        sub: {
-          key: 'basic_3mo', // @TODO: Validate that this is group
-        },
-        customerId: 'customer-id',
-        paymentMethod: 'Payment Method',
-        headers: {
-          'x-client': 'habitica-web',
-          'user-agent': '',
-        },
-      };
-
-      group = generateGroup({
-        name: 'test group',
-        type: 'guild',
-        privacy: 'public',
-        leader: user._id,
-      });
-      await group.save();
-
-      spy = sinon.stub(amzLib, 'authorizeOnBillingAgreement');
-      spy.returnsPromise().resolves([]);
-
-      uuidString = 'uuid-v4';
-      sinon.stub(uuid, 'v4').returns(uuidString);
-
-      data.groupId = group._id;
-      data.sub.quantity = 3;
-    });
-
-    afterEach(function () {
-      sinon.restore(amzLib.authorizeOnBillingAgreement);
-      uuid.v4.restore();
-    });
-
-    it('charges for a new member', async () => {
-      data.paymentMethod = amzLib.constants.PAYMENT_METHOD;
-      await payments.createSubscription(data);
-
-      let updatedGroup = await Group.findById(group._id).exec();
-
-      updatedGroup.memberCount += 1;
-      await updatedGroup.save();
-
-      await amzLib.chargeForAdditionalGroupMember(updatedGroup);
-
-      expect(spy.calledOnce).to.be.true;
-      expect(spy).to.be.calledWith({
-        AmazonBillingAgreementId: updatedGroup.purchased.plan.customerId,
-        AuthorizationReferenceId: uuidString.substring(0, 32),
-        AuthorizationAmount: {
-          CurrencyCode: amzLib.constants.CURRENCY_CODE,
-          Amount: 3,
-        },
-        SellerAuthorizationNote: amzLib.constants.SELLER_NOTE_GROUP_NEW_MEMBER,
-        TransactionTimeout: 0,
-        CaptureNow: true,
-        SellerNote: amzLib.constants.SELLER_NOTE_GROUP_NEW_MEMBER,
-        SellerOrderAttributes: {
-          SellerOrderId: uuidString,
-          StoreName: amzLib.constants.STORE_NAME,
-        },
-      });
-    });
-  });
 });

test/api/v3/unit/libs/cron.test.js

@@ -6,6 +6,7 @@ import requireAgain from 'require-again';
 import { recoverCron, cron } from '../../../../../website/server/libs/cron';
 import { model as User } from '../../../../../website/server/models/user';
 import * as Tasks from '../../../../../website/server/models/task';
+import { clone } from 'lodash';
 import common from '../../../../../website/common';
 import analytics from '../../../../../website/server/libs/analyticsService';
 
@@ -134,10 +135,7 @@ describe('cron', () => {
       user.purchased.plan.dateUpdated = moment().subtract(6, 'months').toDate();
       user.purchased.plan.dateTerminated = moment().subtract(3, 'months').toDate();
       user.purchased.plan.consecutive.count = 5;
-      user.purchased.plan.consecutive.trinkets = 1;
-
       cron({user, tasksByType, daysMissed, analytics});
-
       expect(user.purchased.plan.consecutive.trinkets).to.equal(1);
     });
 
@@ -483,67 +481,6 @@ describe('cron', () => {
 
       expect(tasksByType.habits[0].value).to.equal(1);
     });
-
-    describe('counters', () => {
-      let notStartOfWeekOrMonth = new Date(2016, 9, 28).getTime(); // a Friday
-      let clock;
-
-      beforeEach(() => {
-        // Replace system clocks so we can get predictable results
-        clock = sinon.useFakeTimers(notStartOfWeekOrMonth);
-      });
-      afterEach(() => {
-        return clock.restore();
-      });
-
-      it('should reset a daily habit counter each day', () => {
-        tasksByType.habits[0].counterUp = 1;
-        tasksByType.habits[0].counterDown = 1;
-
-        cron({user, tasksByType, daysMissed, analytics});
-
-        expect(tasksByType.habits[0].counterUp).to.equal(0);
-        expect(tasksByType.habits[0].counterDown).to.equal(0);
-      });
-
-      it('should reset a weekly habit counter each Monday', () => {
-        tasksByType.habits[0].frequency = 'weekly';
-        tasksByType.habits[0].counterUp = 1;
-        tasksByType.habits[0].counterDown = 1;
-
-        // should not reset
-        cron({user, tasksByType, daysMissed, analytics});
-
-        expect(tasksByType.habits[0].counterUp).to.equal(1);
-        expect(tasksByType.habits[0].counterDown).to.equal(1);
-
-        // should reset
-        daysMissed = 8;
-        cron({user, tasksByType, daysMissed, analytics});
-
-        expect(tasksByType.habits[0].counterUp).to.equal(0);
-        expect(tasksByType.habits[0].counterDown).to.equal(0);
-      });
-
-      it('should reset a monthly habit counter the first day of each month', () => {
-        tasksByType.habits[0].frequency = 'monthly';
-        tasksByType.habits[0].counterUp = 1;
-        tasksByType.habits[0].counterDown = 1;
-
-        // should not reset
-        cron({user, tasksByType, daysMissed, analytics});
-
-        expect(tasksByType.habits[0].counterUp).to.equal(1);
-        expect(tasksByType.habits[0].counterDown).to.equal(1);
-
-        // should reset
-        daysMissed = 32;
-        cron({user, tasksByType, daysMissed, analytics});
-
-        expect(tasksByType.habits[0].counterUp).to.equal(0);
-        expect(tasksByType.habits[0].counterDown).to.equal(0);
-      });
-    });
   });
 
   describe('perfect day', () => {
@@ -596,7 +533,7 @@ describe('cron', () => {
       tasksByType.dailys[0].completed = true;
       tasksByType.dailys[0].startDate = moment(new Date()).subtract({days: 1});
 
-      let previousBuffs = user.stats.buffs.toObject();
+      let previousBuffs = clone(user.stats.buffs);
 
       cron({user, tasksByType, daysMissed, analytics});
 
@@ -661,7 +598,7 @@ describe('cron', () => {
       tasksByType.dailys[0].completed = false;
       tasksByType.dailys[0].startDate = moment(new Date()).subtract({days: 1});
 
-      let previousBuffs = user.stats.buffs.toObject();
+      let previousBuffs = clone(user.stats.buffs);
 
       cronOverride({user, tasksByType, daysMissed, analytics});
 

test/api/v3/unit/libs/payments.test.js

@@ -1,18 +1,22 @@
-import moment from 'moment';
-
 import * as sender from '../../../../../website/server/libs/email';
 import * as api from '../../../../../website/server/libs/payments';
 import analytics from '../../../../../website/server/libs/analyticsService';
 import notifications from '../../../../../website/server/libs/pushNotifications';
 import { model as User } from '../../../../../website/server/models/user';
+import { model as Group } from '../../../../../website/server/models/group';
+import stripeModule from 'stripe';
+import moment from 'moment';
 import { translate as t } from '../../../../helpers/api-v3-integration.helper';
 import {
   generateGroup,
 } from '../../../../helpers/api-unit.helper.js';
+import i18n from '../../../../../website/common/script/i18n';
 
 describe('payments/index', () => {
   let user, group, data, plan;
 
+  let stripe = stripeModule('test');
+
   beforeEach(async () => {
     user = new User();
     user.profile.name = 'sender';
@@ -315,6 +319,53 @@ describe('payments/index', () => {
       });
     });
 
+    context('Purchasing a subscription for group', () => {
+      it('creates a subscription', async () => {
+        expect(group.purchased.plan.planId).to.not.exist;
+        data.groupId = group._id;
+
+        await api.createSubscription(data);
+
+        let updatedGroup = await Group.findById(group._id).exec();
+
+        expect(updatedGroup.purchased.plan.planId).to.eql('basic_3mo');
+        expect(updatedGroup.purchased.plan.customerId).to.eql('customer-id');
+        expect(updatedGroup.purchased.plan.dateUpdated).to.exist;
+        expect(updatedGroup.purchased.plan.gemsBought).to.eql(0);
+        expect(updatedGroup.purchased.plan.paymentMethod).to.eql('Payment Method');
+        expect(updatedGroup.purchased.plan.extraMonths).to.eql(0);
+        expect(updatedGroup.purchased.plan.dateTerminated).to.eql(null);
+        expect(updatedGroup.purchased.plan.lastBillingDate).to.not.exist;
+        expect(updatedGroup.purchased.plan.dateCreated).to.exist;
+      });
+
+      it('sets extraMonths if plan has dateTerminated date', async () => {
+        group.purchased.plan = plan;
+        group.purchased.plan.dateTerminated = moment(new Date()).add(2, 'months');
+        await group.save();
+        expect(group.purchased.plan.extraMonths).to.eql(0);
+        data.groupId = group._id;
+
+        await api.createSubscription(data);
+
+        let updatedGroup = await Group.findById(group._id).exec();
+        expect(updatedGroup.purchased.plan.extraMonths).to.within(1.9, 2);
+      });
+
+      it('does not set negative extraMonths if plan has past dateTerminated date', async () => {
+        group.purchased.plan = plan;
+        group.purchased.plan.dateTerminated = moment(new Date()).subtract(2, 'months');
+        await group.save();
+        expect(group.purchased.plan.extraMonths).to.eql(0);
+        data.groupId = group._id;
+
+        await api.createSubscription(data);
+
+        let updatedGroup = await Group.findById(group._id).exec();
+        expect(updatedGroup.purchased.plan.extraMonths).to.eql(0);
+      });
+    });
+
     context('Block subscription perks', () => {
       it('adds block months to plan.consecutive.offset', async () => {
         await api.createSubscription(data);
@@ -434,6 +485,7 @@ describe('payments/index', () => {
         sandbox.spy(user.purchased.plan.mysteryItems, 'push');
 
         data = { paymentMethod: 'PaymentMethod', user, sub: { key: 'basic_3mo' } };
+
         await api.createSubscription(data);
 
         expect(user.purchased.plan.mysteryItems.push).to.be.calledOnce;
@@ -507,6 +559,112 @@ describe('payments/index', () => {
         expect(sender.sendTxn).to.be.calledWith(user, 'cancel-subscription');
       });
     });
+
+    context('Canceling a subscription for group', () => {
+      it('adds a month termination date by default', async () => {
+        data.groupId = group._id;
+        await api.cancelSubscription(data);
+
+        let now = new Date();
+        let updatedGroup = await Group.findById(group._id).exec();
+        let daysTillTermination = moment(updatedGroup.purchased.plan.dateTerminated).diff(now, 'days');
+
+        expect(daysTillTermination).to.be.within(29, 30); // 1 month +/- 1 days
+      });
+
+      it('adds extraMonths to dateTerminated value', async () => {
+        group.purchased.plan.extraMonths = 2;
+        await group.save();
+        data.groupId = group._id;
+
+        await api.cancelSubscription(data);
+
+        let now = new Date();
+        let updatedGroup = await Group.findById(group._id).exec();
+        let daysTillTermination = moment(updatedGroup.purchased.plan.dateTerminated).diff(now, 'days');
+
+        expect(daysTillTermination).to.be.within(89, 90); // 3 months +/- 1 days
+      });
+
+      it('handles extra month fractions', async () => {
+        group.purchased.plan.extraMonths = 0.3;
+        await group.save();
+        data.groupId = group._id;
+
+        await api.cancelSubscription(data);
+
+        let now = new Date();
+        let updatedGroup = await Group.findById(group._id).exec();
+        let daysTillTermination = moment(updatedGroup.purchased.plan.dateTerminated).diff(now, 'days');
+
+        expect(daysTillTermination).to.be.within(38, 39); // should be about 1 month + 1/3 month
+      });
+
+      it('terminates at next billing date if it exists', async () => {
+        data.nextBill = moment().add({ days: 15 });
+        data.groupId = group._id;
+
+        await api.cancelSubscription(data);
+
+        let now = new Date();
+        let updatedGroup = await Group.findById(group._id).exec();
+        let daysTillTermination = moment(updatedGroup.purchased.plan.dateTerminated).diff(now, 'days');
+
+        expect(daysTillTermination).to.be.within(13, 15);
+      });
+
+      it('resets plan.extraMonths', async () => {
+        group.purchased.plan.extraMonths = 5;
+        await group.save();
+        data.groupId = group._id;
+
+        await api.cancelSubscription(data);
+
+        let updatedGroup = await Group.findById(group._id).exec();
+        expect(updatedGroup.purchased.plan.extraMonths).to.eql(0);
+      });
+
+      it('sends an email', async () => {
+        data.groupId = group._id;
+        await api.cancelSubscription(data);
+
+        expect(sender.sendTxn).to.be.calledOnce;
+        expect(sender.sendTxn).to.be.calledWith(user, 'group-cancel-subscription');
+      });
+
+      it('prevents non group leader from manging subscription', async () => {
+        let groupMember = new User();
+        data.user = groupMember;
+        data.groupId = group._id;
+
+        await expect(api.cancelSubscription(data))
+          .eventually.be.rejected.and.to.eql({
+            httpCode: 401,
+            message: i18n.t('onlyGroupLeaderCanManageSubscription'),
+            name: 'NotAuthorized',
+          });
+      });
+
+      it('allows old group leader to cancel if they created the subscription', async () => {
+        data.groupId = group._id;
+        data.sub = {
+          key: 'group_monthly',
+        };
+        data.paymentMethod = 'Payment Method';
+        await api.createSubscription(data);
+
+        let updatedGroup = await Group.findById(group._id).exec();
+        let newLeader = new User();
+        updatedGroup.leader = newLeader._id;
+        await updatedGroup.save();
+
+        await api.cancelSubscription(data);
+
+        updatedGroup = await Group.findById(group._id).exec();
+
+        expect(updatedGroup.purchased.plan.dateTerminated).to.exist;
+      });
+    });
   });
 
   describe('#buyGems', () => {
@@ -614,24 +772,49 @@ describe('payments/index', () => {
     });
   });
 
-  describe('addSubToGroupUser', () => {
-    it('adds a group subscription to a new user', async () => {
-      expect(group.purchased.plan.planId).to.not.exist;
+  describe('#upgradeGroupPlan', () => {
+    let spy;
+
+    beforeEach(function () {
+      spy = sinon.stub(stripe.subscriptions, 'update');
+      spy.returnsPromise().resolves([]);
       data.groupId = group._id;
+      data.sub.quantity = 3;
+    });
 
-      await api.addSubToGroupUser(user, group);
+    afterEach(function () {
+      sinon.restore(stripe.subscriptions.update);
+    });
 
-      let updatedUser = await User.findById(user._id).exec();
+    it('updates a group plan quantity', async () => {
+      data.paymentMethod = 'Stripe';
+      await api.createSubscription(data);
 
-      expect(updatedUser.purchased.plan.planId).to.eql('group_plan_auto');
-      expect(updatedUser.purchased.plan.customerId).to.eql('group-plan');
-      expect(updatedUser.purchased.plan.dateUpdated).to.exist;
-      expect(updatedUser.purchased.plan.gemsBought).to.eql(0);
-      expect(updatedUser.purchased.plan.paymentMethod).to.eql('Group Plan');
-      expect(updatedUser.purchased.plan.extraMonths).to.eql(0);
-      expect(updatedUser.purchased.plan.dateTerminated).to.eql(null);
-      expect(updatedUser.purchased.plan.lastBillingDate).to.not.exist;
-      expect(updatedUser.purchased.plan.dateCreated).to.exist;
+      let updatedGroup = await Group.findById(group._id).exec();
+      expect(updatedGroup.purchased.plan.quantity).to.eql(3);
+
+      updatedGroup.memberCount += 1;
+      await updatedGroup.save();
+
+      await api.updateStripeGroupPlan(updatedGroup, stripe);
+
+      expect(spy.calledOnce).to.be.true;
+      expect(updatedGroup.purchased.plan.quantity).to.eql(4);
+    });
+
+    it('does not update a group plan quantity that has a payment method other than stripe', async () => {
+      await api.createSubscription(data);
+
+      let updatedGroup = await Group.findById(group._id).exec();
+      expect(updatedGroup.purchased.plan.quantity).to.eql(3);
+
+      updatedGroup.memberCount += 1;
+      await updatedGroup.save();
+
+      await api.updateStripeGroupPlan(updatedGroup, stripe);
+
+      expect(spy.calledOnce).to.be.false;
+      expect(updatedGroup.purchased.plan.quantity).to.eql(3);
     });
   });
 });

test/api/v3/unit/libs/payments/group-plans/group-payments-cancel.test.js

@@ -1,326 +0,0 @@
-import moment from 'moment';
-
-import * as sender from '../../../../../../../website/server/libs/email';
-import * as api from '../../../../../../../website/server/libs/payments';
-import { model as User } from '../../../../../../../website/server/models/user';
-import { model as Group } from '../../../../../../../website/server/models/group';
-import {
-  generateGroup,
-} from '../../../../../../helpers/api-unit.helper.js';
-import i18n from '../../../../../../../website/common/script/i18n';
-
-describe('Canceling a subscription for group', () => {
-  let plan, group, user, data;
-
-  beforeEach(async () => {
-    user = new User();
-    user.profile.name = 'sender';
-    await user.save();
-
-    group = generateGroup({
-      name: 'test group',
-      type: 'guild',
-      privacy: 'public',
-      leader: user._id,
-    });
-    await group.save();
-
-    data = {
-      user,
-      sub: {
-        key: 'basic_3mo',
-      },
-      customerId: 'customer-id',
-      paymentMethod: 'Payment Method',
-      headers: {
-        'x-client': 'habitica-web',
-        'user-agent': '',
-      },
-    };
-
-    plan = {
-      planId: 'basic_3mo',
-      customerId: 'customer-id',
-      dateUpdated: new Date(),
-      gemsBought: 0,
-      paymentMethod: 'paymentMethod',
-      extraMonths: 0,
-      dateTerminated: null,
-      lastBillingDate: new Date(),
-      dateCreated: new Date(),
-      mysteryItems: [],
-      consecutive: {
-        trinkets: 0,
-        offset: 0,
-        gemCapExtra: 0,
-      },
-    };
-
-    sandbox.stub(sender, 'sendTxn');
-  });
-
-  afterEach(() => {
-    sender.sendTxn.restore();
-  });
-
-  it('adds a month termination date by default', async () => {
-    data.groupId = group._id;
-    await api.cancelSubscription(data);
-
-    let now = new Date();
-    let updatedGroup = await Group.findById(group._id).exec();
-    let daysTillTermination = moment(updatedGroup.purchased.plan.dateTerminated).diff(now, 'days');
-
-    expect(daysTillTermination).to.be.within(29, 30); // 1 month +/- 1 days
-  });
-
-  it('adds extraMonths to dateTerminated value', async () => {
-    group.purchased.plan.extraMonths = 2;
-    await group.save();
-    data.groupId = group._id;
-
-    await api.cancelSubscription(data);
-
-    let now = new Date();
-    let updatedGroup = await Group.findById(group._id).exec();
-    let daysTillTermination = moment(updatedGroup.purchased.plan.dateTerminated).diff(now, 'days');
-
-    expect(daysTillTermination).to.be.within(89, 90); // 3 months +/- 1 days
-  });
-
-  it('handles extra month fractions', async () => {
-    group.purchased.plan.extraMonths = 0.3;
-    await group.save();
-    data.groupId = group._id;
-
-    await api.cancelSubscription(data);
-
-    let now = new Date();
-    let updatedGroup = await Group.findById(group._id).exec();
-    let daysTillTermination = moment(updatedGroup.purchased.plan.dateTerminated).diff(now, 'days');
-
-    expect(daysTillTermination).to.be.within(38, 39); // should be about 1 month + 1/3 month
-  });
-
-  it('terminates at next billing date if it exists', async () => {
-    data.nextBill = moment().add({ days: 15 });
-    data.groupId = group._id;
-
-    await api.cancelSubscription(data);
-
-    let now = new Date();
-    let updatedGroup = await Group.findById(group._id).exec();
-    let daysTillTermination = moment(updatedGroup.purchased.plan.dateTerminated).diff(now, 'days');
-
-    expect(daysTillTermination).to.be.within(13, 15);
-  });
-
-  it('resets plan.extraMonths', async () => {
-    group.purchased.plan.extraMonths = 5;
-    await group.save();
-    data.groupId = group._id;
-
-    await api.cancelSubscription(data);
-
-    let updatedGroup = await Group.findById(group._id).exec();
-    expect(updatedGroup.purchased.plan.extraMonths).to.eql(0);
-  });
-
-  it('sends an email', async () => {
-    data.groupId = group._id;
-    await api.cancelSubscription(data);
-
-    expect(sender.sendTxn).to.be.calledOnce;
-    expect(sender.sendTxn.firstCall.args[0]._id).to.equal(user._id);
-    expect(sender.sendTxn.firstCall.args[1]).to.equal('group-cancel-subscription');
-    expect(sender.sendTxn.firstCall.args[2]).to.eql([
-      {name: 'GROUP_NAME', content: group.name},
-    ]);
-  });
-
-  it('prevents non group leader from manging subscription', async () => {
-    let groupMember = new User();
-    data.user = groupMember;
-    data.groupId = group._id;
-
-    await expect(api.cancelSubscription(data))
-      .eventually.be.rejected.and.to.eql({
-        httpCode: 401,
-        message: i18n.t('onlyGroupLeaderCanManageSubscription'),
-        name: 'NotAuthorized',
-      });
-  });
-
-  it('allows old group leader to cancel if they created the subscription', async () => {
-    data.groupId = group._id;
-    data.sub = {
-      key: 'group_monthly',
-    };
-    data.paymentMethod = 'Payment Method';
-    await api.createSubscription(data);
-
-    let updatedGroup = await Group.findById(group._id).exec();
-    let newLeader = new User();
-    updatedGroup.leader = newLeader._id;
-    await updatedGroup.save();
-
-    await api.cancelSubscription(data);
-
-    updatedGroup = await Group.findById(group._id).exec();
-
-    expect(updatedGroup.purchased.plan.dateTerminated).to.exist;
-  });
-
-  it('cancels member subscriptions', async () => {
-    data = {
-      user,
-      sub: {
-        key: 'basic_3mo',
-      },
-      customerId: 'customer-id',
-      paymentMethod: 'Payment Method',
-      headers: {
-        'x-client': 'habitica-web',
-        'user-agent': '',
-      },
-    };
-    user.guilds.push(group._id);
-    await user.save();
-    expect(group.purchased.plan.planId).to.not.exist;
-    data.groupId = group._id;
-    await api.createSubscription(data);
-
-    await api.cancelSubscription(data);
-
-    let now = new Date();
-    now.setHours(0, 0, 0, 0);
-    let updatedLeader = await User.findById(user._id).exec();
-    let daysTillTermination = moment(updatedLeader.purchased.plan.dateTerminated).diff(now, 'days');
-    expect(daysTillTermination).to.be.within(2, 3); // only a few days
-  });
-
-  it('sends an email to members of group', async () => {
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    recipient.guilds.push(group._id);
-    await recipient.save();
-
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-    await api.cancelSubscription(data);
-
-    expect(sender.sendTxn).to.be.have.callCount(4);
-    expect(sender.sendTxn.thirdCall.args[0]._id).to.equal(recipient._id);
-    expect(sender.sendTxn.thirdCall.args[1]).to.equal('group-member-cancel');
-    expect(sender.sendTxn.thirdCall.args[2]).to.eql([
-      {name: 'LEADER', content: user.profile.name},
-      {name: 'GROUP_NAME', content: group.name},
-    ]);
-  });
-
-  it('does not cancel member subscriptions when member does not have a group plan sub (i.e. UNLIMITED_CUSTOMER_ID)', async () => {
-    plan.key = 'basic_earned';
-    plan.customerId = api.constants.UNLIMITED_CUSTOMER_ID;
-
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-    await recipient.save();
-
-    data.groupId = group._id;
-
-    await api.cancelSubscription(data);
-
-    let updatedLeader = await User.findById(user._id).exec();
-    expect(updatedLeader.purchased.plan.dateTerminated).to.not.exist;
-  });
-
-  it('does not cancel a user subscription if they are still in another active group plan', async () => {
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    plan.key = 'basic_earned';
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-    await recipient.save();
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-    let firstDateCreated = updatedUser.purchased.plan.dateCreated;
-    let extraMonthsBeforeSecond = updatedUser.purchased.plan.extraMonths;
-
-    let group2 = generateGroup({
-      name: 'test group2',
-      type: 'guild',
-      privacy: 'public',
-      leader: user._id,
-    });
-    data.groupId = group2._id;
-    await group2.save();
-    recipient.guilds.push(group2._id);
-    await recipient.save();
-
-    await api.createSubscription(data);
-
-    await api.cancelSubscription(data);
-
-    updatedUser = await User.findById(recipient._id).exec();
-
-    expect(updatedUser.purchased.plan.planId).to.eql('group_plan_auto');
-    expect(updatedUser.purchased.plan.customerId).to.eql('group-plan');
-    expect(updatedUser.purchased.plan.dateUpdated).to.exist;
-    expect(updatedUser.purchased.plan.gemsBought).to.eql(0);
-    expect(updatedUser.purchased.plan.paymentMethod).to.eql('Group Plan');
-    expect(updatedUser.purchased.plan.extraMonths).to.eql(extraMonthsBeforeSecond);
-    expect(updatedUser.purchased.plan.dateTerminated).to.eql(null);
-    expect(updatedUser.purchased.plan.lastBillingDate).to.not.exist;
-    expect(updatedUser.purchased.plan.dateCreated).to.eql(firstDateCreated);
-  });
-
-  it('does cancel a leader subscription with two cancelled group plans', async () => {
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(user._id).exec();
-    let firstDateCreated = updatedUser.purchased.plan.dateCreated;
-    let extraMonthsBeforeSecond = updatedUser.purchased.plan.extraMonths;
-
-    let group2 = generateGroup({
-      name: 'test group2',
-      type: 'guild',
-      privacy: 'public',
-      leader: user._id,
-    });
-    user.guilds.push(group2._id);
-    await user.save();
-    data.groupId = group2._id;
-    await group2.save();
-
-    await api.createSubscription(data);
-    await api.cancelSubscription(data);
-
-    data.groupId = group._id;
-    await api.cancelSubscription(data);
-
-    updatedUser = await User.findById(user._id).exec();
-
-    expect(updatedUser.purchased.plan.planId).to.eql('group_plan_auto');
-    expect(updatedUser.purchased.plan.customerId).to.eql('group-plan');
-    expect(updatedUser.purchased.plan.dateUpdated).to.exist;
-    expect(updatedUser.purchased.plan.gemsBought).to.eql(0);
-    expect(updatedUser.purchased.plan.paymentMethod).to.eql('Group Plan');
-    expect(updatedUser.purchased.plan.extraMonths).to.eql(extraMonthsBeforeSecond);
-    expect(updatedUser.purchased.plan.dateTerminated).to.exist;
-    expect(updatedUser.purchased.plan.lastBillingDate).to.not.exist;
-    expect(updatedUser.purchased.plan.dateCreated).to.eql(firstDateCreated);
-  });
-});

test/api/v3/unit/libs/payments/group-plans/group-payments-create.test.js

@@ -1,635 +0,0 @@
-import moment from 'moment';
-import stripeModule from 'stripe';
-
-import * as sender from '../../../../../../../website/server/libs/email';
-import * as api from '../../../../../../../website/server/libs/payments';
-import amzLib from '../../../../../../../website/server/libs/amazonPayments';
-import stripePayments from '../../../../../../../website/server/libs/stripePayments';
-import paypalPayments from '../../../../../../../website/server/libs/paypalPayments';
-import { model as User } from '../../../../../../../website/server/models/user';
-import { model as Group } from '../../../../../../../website/server/models/group';
-import {
-  generateGroup,
-} from '../../../../../../helpers/api-unit.helper.js';
-
-describe('Purchasing a subscription for group', () => {
-  let plan, group, user, data;
-  let stripe = stripeModule('test');
-
-  beforeEach(async () => {
-    user = new User();
-    user.profile.name = 'sender';
-    await user.save();
-
-    group = generateGroup({
-      name: 'test group',
-      type: 'guild',
-      privacy: 'public',
-      leader: user._id,
-    });
-    await group.save();
-
-    data = {
-      user,
-      sub: {
-        key: 'basic_3mo',
-      },
-      customerId: 'customer-id',
-      paymentMethod: 'Payment Method',
-      headers: {
-        'x-client': 'habitica-web',
-        'user-agent': '',
-      },
-    };
-
-    plan = {
-      planId: 'basic_3mo',
-      customerId: 'customer-id',
-      dateUpdated: new Date(),
-      gemsBought: 0,
-      paymentMethod: 'paymentMethod',
-      extraMonths: 0,
-      dateTerminated: null,
-      lastBillingDate: new Date(),
-      dateCreated: new Date(),
-      mysteryItems: [],
-      consecutive: {
-        trinkets: 0,
-        offset: 0,
-        gemCapExtra: 0,
-      },
-    };
-
-    let subscriptionId = 'subId';
-    sinon.stub(stripe.customers, 'del').returnsPromise().resolves({});
-
-    let currentPeriodEndTimeStamp = moment().add(3, 'months').unix();
-    sinon.stub(stripe.customers, 'retrieve')
-      .returnsPromise().resolves({
-        subscriptions: {
-          data: [{id: subscriptionId, current_period_end: currentPeriodEndTimeStamp}], // eslint-disable-line camelcase
-        },
-      });
-
-    stripePayments.setStripeApi(stripe);
-    sandbox.stub(sender, 'sendTxn');
-  });
-
-  afterEach(() => {
-    stripe.customers.del.restore();
-    stripe.customers.retrieve.restore();
-    sender.sendTxn.restore();
-  });
-
-  it('creates a subscription', async () => {
-    expect(group.purchased.plan.planId).to.not.exist;
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedGroup = await Group.findById(group._id).exec();
-
-    expect(updatedGroup.purchased.plan.planId).to.eql('basic_3mo');
-    expect(updatedGroup.purchased.plan.customerId).to.eql('customer-id');
-    expect(updatedGroup.purchased.plan.dateUpdated).to.exist;
-    expect(updatedGroup.purchased.plan.gemsBought).to.eql(0);
-    expect(updatedGroup.purchased.plan.paymentMethod).to.eql('Payment Method');
-    expect(updatedGroup.purchased.plan.extraMonths).to.eql(0);
-    expect(updatedGroup.purchased.plan.dateTerminated).to.eql(null);
-    expect(updatedGroup.purchased.plan.lastBillingDate).to.not.exist;
-    expect(updatedGroup.purchased.plan.dateCreated).to.exist;
-  });
-
-  it('sends an email', async () => {
-    expect(group.purchased.plan.planId).to.not.exist;
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    expect(sender.sendTxn).to.be.calledWith(user, 'group-subscription-begins');
-  });
-
-  it('sets extraMonths if plan has dateTerminated date', async () => {
-    group.purchased.plan = plan;
-    group.purchased.plan.dateTerminated = moment(new Date()).add(2, 'months');
-    await group.save();
-    expect(group.purchased.plan.extraMonths).to.eql(0);
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedGroup = await Group.findById(group._id).exec();
-    expect(updatedGroup.purchased.plan.extraMonths).to.within(1.9, 2);
-  });
-
-  it('does not set negative extraMonths if plan has past dateTerminated date', async () => {
-    group.purchased.plan = plan;
-    group.purchased.plan.dateTerminated = moment(new Date()).subtract(2, 'months');
-    await group.save();
-    expect(group.purchased.plan.extraMonths).to.eql(0);
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedGroup = await Group.findById(group._id).exec();
-    expect(updatedGroup.purchased.plan.extraMonths).to.eql(0);
-  });
-
-  it('grants all members of a group a subscription', async () => {
-    user.guilds.push(group._id);
-    await user.save();
-    expect(group.purchased.plan.planId).to.not.exist;
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-    let updatedLeader = await User.findById(user._id).exec();
-
-    expect(updatedLeader.purchased.plan.planId).to.eql('group_plan_auto');
-    expect(updatedLeader.purchased.plan.customerId).to.eql('group-plan');
-    expect(updatedLeader.purchased.plan.dateUpdated).to.exist;
-    expect(updatedLeader.purchased.plan.gemsBought).to.eql(0);
-    expect(updatedLeader.purchased.plan.paymentMethod).to.eql('Group Plan');
-    expect(updatedLeader.purchased.plan.extraMonths).to.eql(0);
-    expect(updatedLeader.purchased.plan.dateTerminated).to.eql(null);
-    expect(updatedLeader.purchased.plan.lastBillingDate).to.not.exist;
-    expect(updatedLeader.purchased.plan.dateCreated).to.exist;
-
-    expect(updatedLeader.items.mounts['Jackalope-RoyalPurple']).to.be.true;
-  });
-
-  it('sends an email to members of group', async () => {
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    recipient.guilds.push(group._id);
-    await recipient.save();
-
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    expect(sender.sendTxn).to.be.calledTwice;
-    expect(sender.sendTxn.firstCall.args[0]._id).to.equal(recipient._id);
-    expect(sender.sendTxn.firstCall.args[1]).to.equal('group-member-joining');
-    expect(sender.sendTxn.firstCall.args[2]).to.eql([
-      {name: 'LEADER', content: user.profile.name},
-      {name: 'GROUP_NAME', content: group.name},
-    ]);
-  });
-
-  it('adds months to members with existing gift subscription', async () => {
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-    plan.planId = 'basic_earned';
-    plan.paymentMethod = 'paymentMethod';
-    data.gift = {
-      member: recipient,
-      subscription: {
-        key: 'basic_earned',
-        months: 1,
-      },
-    };
-    await api.createSubscription(data);
-    await recipient.save();
-
-    data.gift = undefined;
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-
-    expect(updatedUser.purchased.plan.planId).to.eql('group_plan_auto');
-    expect(updatedUser.purchased.plan.customerId).to.eql('group-plan');
-    expect(updatedUser.purchased.plan.dateUpdated).to.exist;
-    expect(updatedUser.purchased.plan.gemsBought).to.eql(0);
-    expect(updatedUser.purchased.plan.paymentMethod).to.eql('Group Plan');
-    expect(updatedUser.purchased.plan.extraMonths).to.within(1, 3);
-    expect(updatedUser.purchased.plan.dateTerminated).to.eql(null);
-    expect(updatedUser.purchased.plan.lastBillingDate).to.not.exist;
-    expect(updatedUser.purchased.plan.dateCreated).to.exist;
-  });
-
-  it('adds months to members with existing multi-month gift subscription', async () => {
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-    data.gift = {
-      member: recipient,
-      subscription: {
-        key: 'basic_3mo',
-        months: 3,
-      },
-    };
-    await api.createSubscription(data);
-    await recipient.save();
-
-    data.gift = undefined;
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-
-    expect(updatedUser.purchased.plan.planId).to.eql('group_plan_auto');
-    expect(updatedUser.purchased.plan.customerId).to.eql('group-plan');
-    expect(updatedUser.purchased.plan.dateUpdated).to.exist;
-    expect(updatedUser.purchased.plan.gemsBought).to.eql(0);
-    expect(updatedUser.purchased.plan.paymentMethod).to.eql('Group Plan');
-    expect(updatedUser.purchased.plan.extraMonths).to.within(3, 5);
-    expect(updatedUser.purchased.plan.dateTerminated).to.eql(null);
-    expect(updatedUser.purchased.plan.lastBillingDate).to.not.exist;
-    expect(updatedUser.purchased.plan.dateCreated).to.exist;
-  });
-
-  it('adds months to members with existing recurring subscription (Stripe)', async () => {
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    plan.key = 'basic_earned';
-    plan.paymentMethod = stripePayments.constants.PAYMENT_METHOD;
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-    await recipient.save();
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-
-    expect(updatedUser.purchased.plan.extraMonths).to.within(2, 3);
-  });
-
-  it('adds months to members with existing recurring subscription (Amazon)', async () => {
-    sinon.stub(amzLib, 'getBillingAgreementDetails')
-      .returnsPromise()
-      .resolves({
-        BillingAgreementDetails: {
-          BillingAgreementStatus: {State: 'Closed'},
-        },
-      });
-
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    plan.planId = 'basic_earned';
-    plan.paymentMethod = amzLib.constants.PAYMENT_METHOD;
-    plan.lastBillingDate = moment().add(3, 'months');
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-
-    await recipient.save();
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-
-    expect(updatedUser.purchased.plan.extraMonths).to.within(3, 4);
-  });
-
-  it('adds months to members with existing recurring subscription (Paypal)', async () => {
-    sinon.stub(paypalPayments, 'paypalBillingAgreementCancel').returnsPromise().resolves({});
-    sinon.stub(paypalPayments, 'paypalBillingAgreementGet')
-      .returnsPromise().resolves({
-        agreement_details: { // eslint-disable-line camelcase
-          next_billing_date: moment().add(3, 'months').toDate(), // eslint-disable-line camelcase
-          cycles_completed: 1, // eslint-disable-line camelcase
-        },
-      });
-
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    plan.planId = 'basic_earned';
-    plan.paymentMethod = paypalPayments.constants.PAYMENT_METHOD;
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-
-    await recipient.save();
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-
-    expect(updatedUser.purchased.plan.extraMonths).to.within(2, 3);
-    paypalPayments.paypalBillingAgreementGet.restore();
-    paypalPayments.paypalBillingAgreementCancel.restore();
-  });
-
-  it('adds months to members with existing recurring subscription (Android)');
-  it('adds months to members with existing recurring subscription (iOs)');
-
-  it('adds months to members who already cancelled but not yet terminated recurring subscription', async () => {
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    plan.key = 'basic_earned';
-    plan.paymentMethod = stripePayments.constants.PAYMENT_METHOD;
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-    await recipient.save();
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await recipient.cancelSubscription();
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-
-    expect(updatedUser.purchased.plan.extraMonths).to.within(2, 3);
-  });
-
-  it('adds months to members who already cancelled but not yet terminated group plan subscription', async () => {
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    plan.key = 'basic_earned';
-    plan.paymentMethod = api.constants.GROUP_PLAN_PAYMENT_METHOD;
-    plan.extraMonths = 2.94;
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-    await recipient.save();
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await recipient.cancelSubscription();
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-    expect(updatedUser.purchased.plan.extraMonths).to.within(3, 4);
-  });
-
-  it('resets date terminated if user has old subscription', async () => {
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    plan.key = 'basic_earned';
-    plan.paymentMethod = stripePayments.constants.PAYMENT_METHOD;
-    plan.dateTerminated = moment().subtract(1, 'days').toDate();
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-    await recipient.save();
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-
-    expect(updatedUser.purchased.plan.dateTerminated).to.not.exist;
-  });
-
-  it('adds months to members with existing recurring subscription and includes existing extraMonths', async () => {
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    plan.key = 'basic_earned';
-    plan.paymentMethod = stripePayments.constants.PAYMENT_METHOD;
-    plan.extraMonths = 5;
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-    await recipient.save();
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-
-    expect(updatedUser.purchased.plan.extraMonths).to.within(7, 8);
-  });
-
-  it('adds months to members with existing recurring subscription and ignores existing negative extraMonths', async () => {
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    plan.key = 'basic_earned';
-    plan.paymentMethod = stripePayments.constants.PAYMENT_METHOD;
-    plan.extraMonths = -5;
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-    await recipient.save();
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-
-    expect(updatedUser.purchased.plan.extraMonths).to.within(2, 3);
-  });
-
-  it('does not override gemsBought, mysteryItems, dateCreated, and consective fields', async () => {
-    let planCreatedDate = moment().toDate();
-    let mysteryItem = {title: 'item'};
-    let mysteryItems = [mysteryItem];
-    let consecutive = {
-      trinkets: 3,
-      gemCapExtra: 20,
-      offset: 1,
-      count: 13,
-    };
-
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-
-    plan.key = 'basic_earned';
-    plan.gemsBought = 3;
-    plan.dateCreated = planCreatedDate;
-    plan.mysteryItems = mysteryItems;
-    plan.consecutive = consecutive;
-
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-    await recipient.save();
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-
-    expect(updatedUser.purchased.plan.gemsBought).to.equal(3);
-    expect(updatedUser.purchased.plan.mysteryItems[0]).to.eql(mysteryItem);
-    expect(updatedUser.purchased.plan.consecutive.count).to.equal(consecutive.count);
-    expect(updatedUser.purchased.plan.consecutive.offset).to.equal(consecutive.offset);
-    expect(updatedUser.purchased.plan.consecutive.gemCapExtra).to.equal(consecutive.gemCapExtra);
-    expect(updatedUser.purchased.plan.consecutive.trinkets).to.equal(consecutive.trinkets);
-    expect(updatedUser.purchased.plan.dateCreated).to.eql(planCreatedDate);
-  });
-
-  it('does not modify a user with a group subscription when they join another group', async () => {
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    plan.key = 'basic_earned';
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-    await recipient.save();
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-    let firstDateCreated = updatedUser.purchased.plan.dateCreated;
-    let extraMonthsBeforeSecond = updatedUser.purchased.plan.extraMonths;
-
-    let group2 = generateGroup({
-      name: 'test group2',
-      type: 'guild',
-      privacy: 'public',
-      leader: user._id,
-    });
-    data.groupId = group2._id;
-    await group2.save();
-    recipient.guilds.push(group2._id);
-    await recipient.save();
-
-    await api.createSubscription(data);
-
-    updatedUser = await User.findById(recipient._id).exec();
-
-    expect(updatedUser.purchased.plan.planId).to.eql('group_plan_auto');
-    expect(updatedUser.purchased.plan.customerId).to.eql('group-plan');
-    expect(updatedUser.purchased.plan.dateUpdated).to.exist;
-    expect(updatedUser.purchased.plan.gemsBought).to.eql(0);
-    expect(updatedUser.purchased.plan.paymentMethod).to.eql('Group Plan');
-    expect(updatedUser.purchased.plan.extraMonths).to.eql(extraMonthsBeforeSecond);
-    expect(updatedUser.purchased.plan.dateTerminated).to.eql(null);
-    expect(updatedUser.purchased.plan.lastBillingDate).to.not.exist;
-    expect(updatedUser.purchased.plan.dateCreated).to.eql(firstDateCreated);
-  });
-
-  it('does not remove a user who is in two groups plans and leaves one', async () => {
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    plan.key = 'basic_earned';
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-    await recipient.save();
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-    let firstDateCreated = updatedUser.purchased.plan.dateCreated;
-    let extraMonthsBeforeSecond = updatedUser.purchased.plan.extraMonths;
-
-    let group2 = generateGroup({
-      name: 'test group2',
-      type: 'guild',
-      privacy: 'public',
-      leader: user._id,
-    });
-    data.groupId = group2._id;
-    await group2.save();
-    recipient.guilds.push(group2._id);
-    await recipient.save();
-
-    await api.createSubscription(data);
-
-    let updatedGroup = await Group.findById(group._id).exec();
-    await updatedGroup.leave(recipient);
-
-    updatedUser = await User.findById(recipient._id).exec();
-
-    expect(updatedUser.purchased.plan.planId).to.eql('group_plan_auto');
-    expect(updatedUser.purchased.plan.customerId).to.eql('group-plan');
-    expect(updatedUser.purchased.plan.dateUpdated).to.exist;
-    expect(updatedUser.purchased.plan.gemsBought).to.eql(0);
-    expect(updatedUser.purchased.plan.paymentMethod).to.eql('Group Plan');
-    expect(updatedUser.purchased.plan.extraMonths).to.eql(extraMonthsBeforeSecond);
-    expect(updatedUser.purchased.plan.dateTerminated).to.eql(null);
-    expect(updatedUser.purchased.plan.lastBillingDate).to.not.exist;
-    expect(updatedUser.purchased.plan.dateCreated).to.eql(firstDateCreated);
-  });
-
-  it('does not modify a user with an unlimited subscription', async () => {
-    plan.key = 'basic_earned';
-    plan.customerId = api.constants.UNLIMITED_CUSTOMER_ID;
-
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-    await recipient.save();
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-
-    expect(updatedUser.purchased.plan.planId).to.eql('basic_3mo');
-    expect(updatedUser.purchased.plan.customerId).to.eql(api.constants.UNLIMITED_CUSTOMER_ID);
-    expect(updatedUser.purchased.plan.dateUpdated).to.exist;
-    expect(updatedUser.purchased.plan.gemsBought).to.eql(0);
-    expect(updatedUser.purchased.plan.paymentMethod).to.eql('paymentMethod');
-    expect(updatedUser.purchased.plan.extraMonths).to.eql(0);
-    expect(updatedUser.purchased.plan.dateTerminated).to.eql(null);
-    expect(updatedUser.purchased.plan.lastBillingDate).to.exist;
-    expect(updatedUser.purchased.plan.dateCreated).to.exist;
-  });
-
-  it('updates a user with a cancelled but active group subscription', async () => {
-    plan.key = 'basic_earned';
-    plan.customerId = api.constants.GROUP_PLAN_CUSTOMER_ID;
-    plan.dateTerminated = moment().add(1, 'months');
-
-    let recipient = new User();
-    recipient.profile.name = 'recipient';
-    recipient.purchased.plan = plan;
-    recipient.guilds.push(group._id);
-    await recipient.save();
-
-    user.guilds.push(group._id);
-    await user.save();
-    data.groupId = group._id;
-
-    await api.createSubscription(data);
-
-    let updatedUser = await User.findById(recipient._id).exec();
-
-    expect(updatedUser.purchased.plan.planId).to.eql('group_plan_auto');
-    expect(updatedUser.purchased.plan.customerId).to.eql(api.constants.GROUP_PLAN_CUSTOMER_ID);
-    expect(updatedUser.purchased.plan.dateUpdated).to.exist;
-    expect(updatedUser.purchased.plan.gemsBought).to.eql(0);
-    expect(updatedUser.purchased.plan.paymentMethod).to.eql('Group Plan');
-    expect(updatedUser.purchased.plan.extraMonths).to.within(0, 2);
-    expect(updatedUser.purchased.plan.dateTerminated).to.eql(null);
-    expect(updatedUser.purchased.plan.lastBillingDate).to.not.exist;
-    expect(updatedUser.purchased.plan.dateCreated).to.exist;
-  });
-});

test/api/v3/unit/libs/stripePayments.test.js

@@ -5,7 +5,6 @@ import {
   generateGroup,
 } from '../../../../helpers/api-unit.helper.js';
 import { model as User } from '../../../../../website/server/models/user';
-import { model as Group } from '../../../../../website/server/models/group';
 import { model as Coupon } from '../../../../../website/server/models/coupon';
 import stripePayments from '../../../../../website/server/libs/stripePayments';
 import payments from '../../../../../website/server/libs/payments';
@@ -659,60 +658,4 @@ describe('Stripe Payments', () => {
       });
     });
   });
-
-  describe('#upgradeGroupPlan', () => {
-    let spy, data, user, group;
-
-    beforeEach(async function () {
-      user = new User();
-      user.profile.name = 'sender';
-
-      data = {
-        user,
-        sub: {
-          key: 'basic_3mo', // @TODO: Validate that this is group
-        },
-        customerId: 'customer-id',
-        paymentMethod: 'Payment Method',
-        headers: {
-          'x-client': 'habitica-web',
-          'user-agent': '',
-        },
-      };
-
-      group = generateGroup({
-        name: 'test group',
-        type: 'guild',
-        privacy: 'public',
-        leader: user._id,
-      });
-      await group.save();
-
-      spy = sinon.stub(stripe.subscriptions, 'update');
-      spy.returnsPromise().resolves([]);
-      data.groupId = group._id;
-      data.sub.quantity = 3;
-      stripePayments.setStripeApi(stripe);
-    });
-
-    afterEach(function () {
-      sinon.restore(stripe.subscriptions.update);
-    });
-
-    it('updates a group plan quantity', async () => {
-      data.paymentMethod = 'Stripe';
-      await payments.createSubscription(data);
-
-      let updatedGroup = await Group.findById(group._id).exec();
-      expect(updatedGroup.purchased.plan.quantity).to.eql(3);
-
-      updatedGroup.memberCount += 1;
-      await updatedGroup.save();
-
-      await stripePayments.chargeForAdditionalGroupMember(updatedGroup);
-
-      expect(spy.calledOnce).to.be.true;
-      expect(updatedGroup.purchased.plan.quantity).to.eql(4);
-    });
-  });
 });

test/api/v3/unit/middlewares/cronMiddleware.js

@@ -4,6 +4,7 @@ import {
   generateTodo,
   generateDaily,
 } from '../../../../helpers/api-unit.helper';
+import { cloneDeep } from 'lodash';
 import cronMiddleware from '../../../../../website/server/middlewares/cron';
 import moment from 'moment';
 import { model as User } from '../../../../../website/server/models/user';
@@ -59,7 +60,7 @@ describe('cron middleware', () => {
     cronMiddleware(req, res, done);
   });
 
-  it('should clear todos older than 30 days for free users', async () => {
+  it('should clear todos older than 30 days for free users', async (done) => {
     user.lastCron = moment(new Date()).subtract({days: 2});
     let task = generateTodo(user);
     task.dateCompleted = moment(new Date()).subtract({days: 31});
@@ -67,21 +68,16 @@ describe('cron middleware', () => {
     await task.save();
     await user.save();
 
-    await new Promise((resolve, reject) => {
-      cronMiddleware(req, res, (err) => {
-        if (err) return reject(err);
-
-        Tasks.Task.findOne({_id: task}, function (secondErr, taskFound) {
-          if (secondErr) return reject(err);
-          expect(secondErr).to.not.exist;
-          expect(taskFound).to.not.exist;
-          resolve();
-        });
+    cronMiddleware(req, res, (err) => {
+      Tasks.Task.findOne({_id: task}, function (secondErr, taskFound) {
+        expect(secondErr).to.not.exist;
+        expect(taskFound).to.not.exist;
+        done(err);
       });
     });
   });
 
-  it('should not clear todos older than 30 days for subscribed users', async () => {
+  it('should not clear todos older than 30 days for subscribed users', async (done) => {
     user.purchased.plan.customerId = 'subscribedId';
     user.purchased.plan.dateUpdated = moment('012013', 'MMYYYY');
     user.lastCron = moment(new Date()).subtract({days: 2});
@@ -91,20 +87,16 @@ describe('cron middleware', () => {
     await task.save();
     await user.save();
 
-    await new Promise((resolve, reject) => {
-      cronMiddleware(req, res, (err) => {
-        if (err) return reject(err);
-        Tasks.Task.findOne({_id: task}, function (secondErr, taskFound) {
-          if (secondErr) return reject(secondErr);
-          expect(secondErr).to.not.exist;
-          expect(taskFound).to.exist;
-          resolve();
-        });
+    cronMiddleware(req, res, (err) => {
+      Tasks.Task.findOne({_id: task}, function (secondErr, taskFound) {
+        expect(secondErr).to.not.exist;
+        expect(taskFound).to.exist;
+        done(err);
       });
     });
   });
 
-  it('should clear todos older than 90 days for subscribed users', async () => {
+  it('should clear todos older than 90 days for subscribed users', async (done) => {
     user.purchased.plan.customerId = 'subscribedId';
     user.purchased.plan.dateUpdated = moment('012013', 'MMYYYY');
     user.lastCron = moment(new Date()).subtract({days: 2});
@@ -115,49 +107,39 @@ describe('cron middleware', () => {
     await task.save();
     await user.save();
 
-    await new Promise((resolve, reject) => {
-      cronMiddleware(req, res, (err) => {
-        if (err) return reject(err);
-        Tasks.Task.findOne({_id: task}, function (secondErr, taskFound) {
-          if (secondErr) return reject(secondErr);
-          expect(secondErr).to.not.exist;
-          expect(taskFound).to.not.exist;
-          resolve();
-        });
+    cronMiddleware(req, res, (err) => {
+      Tasks.Task.findOne({_id: task}, function (secondErr, taskFound) {
+        expect(secondErr).to.not.exist;
+        expect(taskFound).to.not.exist;
+        done(err);
       });
     });
   });
 
-  it('should call next if user was not modified after cron', async () => {
+  it('should call next if user was not modified after cron', async (done) => {
     let hpBefore = user.stats.hp;
     user.lastCron = moment(new Date()).subtract({days: 2});
     await user.save();
 
-    await new Promise((resolve, reject) => {
-      cronMiddleware(req, res, (err) => {
-        if (err) return reject(err);
-        expect(hpBefore).to.equal(user.stats.hp);
-        resolve();
-      });
+    cronMiddleware(req, res, (err) => {
+      expect(hpBefore).to.equal(user.stats.hp);
+      done(err);
     });
   });
 
-  it('updates user.auth.timestamps.loggedin and lastCron', async () => {
+  it('updates user.auth.timestamps.loggedin and lastCron', async (done) => {
     user.lastCron = moment(new Date()).subtract({days: 2});
     let now = new Date();
     await user.save();
 
-    await new Promise((resolve, reject) => {
-      cronMiddleware(req, res, (err) => {
-        if (err) return reject(err);
-        expect(moment(now).isSame(user.lastCron, 'day'));
-        expect(moment(now).isSame(user.auth.timestamps.loggedin, 'day'));
-        resolve();
-      });
+    cronMiddleware(req, res, (err) => {
+      expect(moment(now).isSame(user.lastCron, 'day'));
+      expect(moment(now).isSame(user.auth.timestamps.loggedin, 'day'));
+      done(err);
     });
   });
 
-  it('does damage for missing dailies', async () => {
+  it('does damage for missing dailies', async (done) => {
     let hpBefore = user.stats.hp;
     user.lastCron = moment(new Date()).subtract({days: 2});
     let daily = generateDaily(user);
@@ -165,34 +147,28 @@ describe('cron middleware', () => {
     await daily.save();
     await user.save();
 
-    await new Promise((resolve, reject) => {
-      cronMiddleware(req, res, (err) => {
-        if (err) return reject(err);
-        expect(user.stats.hp).to.be.lessThan(hpBefore);
-        resolve();
-      });
+    cronMiddleware(req, res, (err) => {
+      expect(user.stats.hp).to.be.lessThan(hpBefore);
+      done(err);
     });
   });
 
-  it('updates tasks', async () => {
+  it('updates tasks', async (done) => {
     user.lastCron = moment(new Date()).subtract({days: 2});
     let todo = generateTodo(user);
     let todoValueBefore = todo.value;
     await user.save();
 
-    await new Promise((resolve, reject) => {
-      cronMiddleware(req, res, (err) => {
-        if (err) return reject(err);
-        Tasks.Task.findOne({_id: todo._id}, function (secondErr, todoFound) {
-          if (secondErr) return reject(secondErr);
-          expect(todoFound.value).to.be.lessThan(todoValueBefore);
-          resolve();
-        });
+    cronMiddleware(req, res, () => {
+      Tasks.Task.findOne({_id: todo._id}, function (err, todoFound) {
+        expect(err).to.not.exist;
+        expect(todoFound.value).to.be.lessThan(todoValueBefore);
+        done();
       });
     });
   });
 
-  it('applies quest progress', async () => {
+  it('applies quest progress', async (done) => {
     let hpBefore = user.stats.hp;
     user.lastCron = moment(new Date()).subtract({days: 2});
     let daily = generateDaily(user);
@@ -216,20 +192,17 @@ describe('cron middleware', () => {
 
     party.startQuest(user);
 
-    await new Promise((resolve, reject) => {
-      cronMiddleware(req, res, (err) => {
-        if (err) return reject(err);
-        expect(user.stats.hp).to.be.lessThan(hpBefore);
-        resolve();
-      });
+    cronMiddleware(req, res, () => {
+      expect(user.stats.hp).to.be.lessThan(hpBefore);
+      done();
     });
   });
 
-  it('recovers from failed cron and does not error when user is already cronning', async () => {
+  it('recovers from failed cron and does not error when user is already cronning', async (done) => {
     user.lastCron = moment(new Date()).subtract({days: 2});
     await user.save();
 
-    let updatedUser = user.toObject();
+    let updatedUser = cloneDeep(user);
     updatedUser.nMatched = 0;
 
     sandbox.spy(cronLib, 'recoverCron');
@@ -242,13 +215,10 @@ describe('cron middleware', () => {
         },
       });
 
-    await new Promise((resolve, reject) => {
-      cronMiddleware(req, res, (err) => {
-        if (err) return reject(err);
-        expect(cronLib.recoverCron).to.be.calledOnce;
+    cronMiddleware(req, res, () => {
+      expect(cronLib.recoverCron).to.be.calledOnce;
 
-        resolve();
-      });
+      done();
     });
   });
 });

test/api/v3/unit/models/group.test.js

@@ -1,6 +1,3 @@
-import moment from 'moment';
-import { v4 as generateUUID } from 'uuid';
-import validator from 'validator';
 import { sleep } from '../../../../helpers/api-unit.helper';
 import { model as Group, INVITES_LIMIT } from '../../../../../website/server/models/group';
 import { model as User } from '../../../../../website/server/models/user';
@@ -10,7 +7,9 @@ import {
 import { quests as questScrolls } from '../../../../../website/common/script/content';
 import { groupChatReceivedWebhook } from '../../../../../website/server/libs/webhook';
 import * as email from '../../../../../website/server/libs/email';
+import validator from 'validator';
 import { TAVERN_ID } from '../../../../../website/common/script/';
+import { v4 as generateUUID } from 'uuid';
 import shared from '../../../../../website/common';
 
 describe('Group Model', () => {
@@ -668,49 +667,6 @@ describe('Group Model', () => {
         expect(party.memberCount).to.eql(1);
       });
 
-      it('does not allow a leader to leave a group with an active subscription', async () => {
-        party.memberCount = 2;
-        party.purchased.plan.customerId = '110002222333';
-
-        await expect(party.leave(questLeader))
-          .to.eventually.be.rejected.and.to.eql({
-            name: 'NotAuthorized',
-            httpCode: 401,
-            message: shared.i18n.t('leaderCannotLeaveGroupWithActiveGroup'),
-          });
-
-        party = await Group.findOne({_id: party._id});
-        expect(party).to.exist;
-        expect(party.memberCount).to.eql(1);
-      });
-
-      it('deletes a private group when the last member leaves and a subscription is cancelled', async () => {
-        let guild = new Group({
-          name: 'test guild',
-          type: 'guild',
-          memberCount: 1,
-        });
-
-        let leader = new User({
-          guilds: [guild._id],
-        });
-
-        guild.leader = leader._id;
-
-        await Promise.all([
-          guild.save(),
-          leader.save(),
-        ]);
-
-        guild.purchased.plan.customerId = '110002222333';
-        guild.purchased.plan.dateTerminated = new Date();
-
-        await guild.leave(leader);
-
-        party = await Group.findOne({_id: guild._id});
-        expect(party).to.not.exist;
-      });
-
       it('does not delete a public group when the last member leaves', async () => {
         party.privacy = 'public';
 
@@ -1089,7 +1045,7 @@ describe('Group Model', () => {
 
           expect(email.sendTxn).to.be.calledOnce;
 
-          let memberIds = _.map(email.sendTxn.args[0][0], '_id');
+          let memberIds = _.pluck(email.sendTxn.args[0][0], '_id');
           let typeOfEmail = email.sendTxn.args[0][1];
 
           expect(memberIds).to.have.a.lengthOf(2);
@@ -1112,7 +1068,7 @@ describe('Group Model', () => {
 
           expect(email.sendTxn).to.be.calledOnce;
 
-          let memberIds = _.map(email.sendTxn.args[0][0], '_id');
+          let memberIds = _.pluck(email.sendTxn.args[0][0], '_id');
 
           expect(memberIds).to.have.a.lengthOf(1);
           expect(memberIds).to.not.include(participatingMember._id);
@@ -1133,7 +1089,7 @@ describe('Group Model', () => {
 
           expect(email.sendTxn).to.be.calledOnce;
 
-          let memberIds = _.map(email.sendTxn.args[0][0], '_id');
+          let memberIds = _.pluck(email.sendTxn.args[0][0], '_id');
 
           expect(memberIds).to.have.a.lengthOf(1);
           expect(memberIds).to.not.include(participatingMember._id);
@@ -1589,57 +1545,5 @@ describe('Group Model', () => {
         expect(args.find(arg => arg[0][0].id === memberWithWebhook3.webhooks[0].id)).to.be.exist;
       });
     });
-
-    context('isSubscribed', () => {
-      it('returns false if group does not have customer id', () => {
-        expect(party.isSubscribed()).to.be.undefined;
-      });
-
-      it('returns true if group does not have plan.dateTerminated', () => {
-        party.purchased.plan.customerId = 'test-id';
-
-        expect(party.isSubscribed()).to.be.true;
-      });
-
-      it('returns true if group if plan.dateTerminated is after today', () => {
-        party.purchased.plan.customerId = 'test-id';
-        party.purchased.plan.dateTerminated = moment().add(1, 'days').toDate();
-
-        expect(party.isSubscribed()).to.be.true;
-      });
-
-      it('returns false if group if plan.dateTerminated is before today', () => {
-        party.purchased.plan.customerId = 'test-id';
-        party.purchased.plan.dateTerminated = moment().subtract(1, 'days').toDate();
-
-        expect(party.isSubscribed()).to.be.false;
-      });
-    });
-
-    context('hasNotCancelled', () => {
-      it('returns false if group does not have customer id', () => {
-        expect(party.hasNotCancelled()).to.be.undefined;
-      });
-
-      it('returns true if party does not have plan.dateTerminated', () => {
-        party.purchased.plan.customerId = 'test-id';
-
-        expect(party.hasNotCancelled()).to.be.true;
-      });
-
-      it('returns false if party if plan.dateTerminated is after today', () => {
-        party.purchased.plan.customerId = 'test-id';
-        party.purchased.plan.dateTerminated = moment().add(1, 'days').toDate();
-
-        expect(party.hasNotCancelled()).to.be.false;
-      });
-
-      it('returns false if party if plan.dateTerminated is before today', () => {
-        party.purchased.plan.customerId = 'test-id';
-        party.purchased.plan.dateTerminated = moment().subtract(1, 'days').toDate();
-
-        expect(party.hasNotCancelled()).to.be.false;
-      });
-    });
   });
 });

test/api/v3/unit/models/user.test.js

@@ -1,7 +1,6 @@
-import Bluebird from 'bluebird';
-import moment from 'moment';
 import { model as User } from '../../../../../website/server/models/user';
 import common from '../../../../../website/common';
+import Bluebird from 'bluebird';
 
 describe('User Model', () => {
   it('keeps user._tmp when calling .toJSON', () => {
@@ -146,68 +145,4 @@ describe('User Model', () => {
       });
     });
   });
-
-  context('isSubscribed', () => {
-    let user;
-    beforeEach(() => {
-      user = new User();
-    });
-
-
-    it('returns false if user does not have customer id', () => {
-      expect(user.isSubscribed()).to.be.undefined;
-    });
-
-    it('returns true if user does not have plan.dateTerminated', () => {
-      user.purchased.plan.customerId = 'test-id';
-
-      expect(user.isSubscribed()).to.be.true;
-    });
-
-    it('returns true if user if plan.dateTerminated is after today', () => {
-      user.purchased.plan.customerId = 'test-id';
-      user.purchased.plan.dateTerminated = moment().add(1, 'days').toDate();
-
-      expect(user.isSubscribed()).to.be.true;
-    });
-
-    it('returns false if user if plan.dateTerminated is before today', () => {
-      user.purchased.plan.customerId = 'test-id';
-      user.purchased.plan.dateTerminated = moment().subtract(1, 'days').toDate();
-
-      expect(user.isSubscribed()).to.be.false;
-    });
-  });
-
-  context('hasNotCancelled', () => {
-    let user;
-    beforeEach(() => {
-      user = new User();
-    });
-
-
-    it('returns false if user does not have customer id', () => {
-      expect(user.hasNotCancelled()).to.be.undefined;
-    });
-
-    it('returns true if user does not have plan.dateTerminated', () => {
-      user.purchased.plan.customerId = 'test-id';
-
-      expect(user.hasNotCancelled()).to.be.true;
-    });
-
-    it('returns false if user if plan.dateTerminated is after today', () => {
-      user.purchased.plan.customerId = 'test-id';
-      user.purchased.plan.dateTerminated = moment().add(1, 'days').toDate();
-
-      expect(user.hasNotCancelled()).to.be.false;
-    });
-
-    it('returns false if user if plan.dateTerminated is before today', () => {
-      user.purchased.plan.customerId = 'test-id';
-      user.purchased.plan.dateTerminated = moment().subtract(1, 'days').toDate();
-
-      expect(user.hasNotCancelled()).to.be.false;
-    });
-  });
 });

test/client-old/spec/controllers/chatCtrlSpec.js

@@ -1,14 +1,14 @@
 'use strict';
 
 describe("Chat Controller", function() {
-  var scope, ctrl, user, $rootScope, $controller, $httpBackend, html;
+  var scope, ctrl, user, $rootScope, $controller;
 
   beforeEach(function() {
     module(function($provide) {
       $provide.value('User', {});
     });
 
-    inject(function(_$rootScope_, _$controller_, _$compile_, _$httpBackend_){
+    inject(function(_$rootScope_, _$controller_){
       user = specHelper.newUser();
       user._id = "unique-user-id";
       $rootScope = _$rootScope_;
@@ -16,20 +16,13 @@ describe("Chat Controller", function() {
       scope = _$rootScope_.$new();
 
       $controller = _$controller_;
-      $httpBackend = _$httpBackend_;
 
       // Load RootCtrl to ensure shared behaviors are loaded
       $controller('RootCtrl',  {$scope: scope, User: {user: user}});
 
-      html = _$compile_('<div><form ng-submit="postChat(group, message.content)"><textarea submit-on-meta-enter ng-model="message.content" ng-model-options="{debounce: 250}"></textarea></form></div>')(scope);
-      document.body.appendChild(html[0]);
-      ctrl = $controller('ChatCtrl', {$scope: scope, $element: html});
+      ctrl = $controller('ChatCtrl', {$scope: scope});
     });
   });
-  
-  afterEach(function() {
-    html.remove();
-  });
 
   describe('copyToDo', function() {
     it('when copying a user message it opens modal with information from message', function() {
@@ -75,47 +68,5 @@ describe("Chat Controller", function() {
       }));
     });
   });
-  
-  it('updates model on enter key press', function() {
-    // Set initial state of the page with some dummy data.
-    scope.group = { name: 'group' };
-    
-    // The main controller is going to try to fetch the template right off. 
-    // No big deal, just return an empty string.
-    $httpBackend.when('GET', 'partials/main.html').respond('');
-
-    // Let the page settle, and the controllers set their initial state.
-    $rootScope.$digest();
-
-    // Watch for calls to postChat & make sure it doesn't do anything.
-    let postChatSpy = sandbox.stub(scope, 'postChat');
-    
-    // Pretend we typed 'aaa' into the textarea.
-    var textarea = html.find('textarea');    
-    textarea[0].value = 'aaa';
-    let inputEvent = new Event('input');
-    textarea[0].dispatchEvent(inputEvent);
-
-    // Give a change for the ng-model watchers to notice that the value in the
-    // textarea has changed.
-    $rootScope.$digest();
-    
-    // Since no time has elapsed and we debounce the model change, we should 
-    // see no model update just yet.
-    expect(scope.message.content).to.equal('');
-    
-    // Now, press the enter key in the textarea. We use jquery here to paper 
-    // over browser differences with initializing the keyboard event.
-    var keyboardEvent = jQuery.Event('keydown', {keyCode: 13, key: 'Enter', metaKey: true});
-    jQuery(textarea).trigger(keyboardEvent);
-
-    // Now, allow the model to update given the changes to the page still
-    // without letting any time elapse...
-    $rootScope.$digest();
-    
-    // ... and nevertheless seeing the desired call to postChat with the right
-    // data. Yay!
-    postChatSpy.should.have.been.calledWith(scope.group, 'aaa');
-  });
 });
 

test/client-old/spec/controllers/inventoryCtrlSpec.js

@@ -30,9 +30,7 @@ describe('Inventory Controller', function() {
           suppressModals: {}
         },
         purchased: {
-          plan: {
-            mysteryItems: [],
-          },
+          plan: {}
         },
       });
 

test/client/unit/index.js

@@ -2,17 +2,8 @@
 // and babel-runtime doesn't affect external libraries
 require('babel-polyfill');
 
-// Automatically setup SinonJS' sandbox for each test
-beforeEach(() => {
-  global.sandbox = sinon.sandbox.create();
-});
-
-afterEach(() => {
-  global.sandbox.restore();
-});
-
-// require all test files
-let testsContext = require.context('./specs', true, /\.js$/);
+// require all test files (files that ends with .spec.js)
+let testsContext = require.context('./specs', true, /\.spec$/);
 testsContext.keys().forEach(testsContext);
 
 // require all .vue and .js files except main.js for coverage.

test/client/unit/karma.conf.js

@@ -17,7 +17,7 @@ module.exports = function (config) {
     //    http://karma-runner.github.io/0.13/config/browsers.html
     // 2. add it to the `browsers` array below.
     browsers: ['PhantomJS'],
-    frameworks: ['mocha', 'sinon-stub-promise', 'sinon-chai', 'chai-as-promised', 'chai'],
+    frameworks: ['mocha', 'sinon-chai'],
     reporters: ['spec', 'coverage'],
     files: ['./index.js'],
     preprocessors: {

test/client/unit/specs/mixins/groupsUtilities.js

@@ -1,61 +0,0 @@
-import groupsUtilities from 'client/mixins/groupsUtilities';
-import { TAVERN_ID } from 'common/script/constants';
-import Vue from 'vue';
-
-describe('Groups Utilities Mixin', () => {
-  let instance, user;
-
-  before(() => {
-    instance = new Vue({
-      mixins: [groupsUtilities],
-    });
-
-    user = {
-      _id: '123',
-      party: {
-        _id: '456',
-      },
-      guilds: ['789'],
-    };
-  });
-
-  describe('isMemberOfGroup', () => {
-    it('registers as a method', () => {
-      expect(instance.isMemberOfGroup).to.be.a.function;
-    });
-
-    it('returns true when the group is the Tavern', () => {
-      expect(instance.isMemberOfGroup(user, {
-        _id: TAVERN_ID,
-      })).to.equal(true);
-    });
-
-    it('returns true when the group is the user\'s party', () => {
-      expect(instance.isMemberOfGroup(user, {
-        type: 'party',
-        _id: user.party._id,
-      })).to.equal(true);
-    });
-
-    it('returns false when the group is not the user\'s party', () => {
-      expect(instance.isMemberOfGroup(user, {
-        type: 'party',
-        _id: 'not my party',
-      })).to.equal(false);
-    });
-
-    it('returns true when the group is not a guild of which the user is a member', () => {
-      expect(instance.isMemberOfGroup(user, {
-        type: 'guild',
-        _id: user.guilds[0],
-      })).to.equal(true);
-    });
-
-    it('returns false when the group is not a guild of which the user is a member', () => {
-      expect(instance.isMemberOfGroup(user, {
-        type: 'guild',
-        _id: 'not my guild',
-      })).to.equal(false);
-    });
-  });
-});

test/client/unit/specs/store/actions/guilds.js

@@ -1,17 +0,0 @@
-import { fetchAll as fetchAllGuilds } from 'client/store/actions/guilds';
-import axios from 'axios';
-import store from 'client/store';
-
-describe('guilds actions', () => {
-  it('fetchAll', async () => {
-    const guilds = [{_id: 1}];
-    sandbox
-      .stub(axios, 'get')
-      .withArgs('/api/v3/groups?type=publicGuilds')
-      .returns(Promise.resolve({data: {data: guilds}}));
-
-    await fetchAllGuilds(store);
-
-    expect(store.state.guilds).to.equal(guilds);
-  });
-});

test/client/unit/specs/store/actions/tasks.js

@@ -1,14 +0,0 @@
-import { fetchUserTasks } from 'client/store/actions/tasks';
-import axios from 'axios';
-import store from 'client/store';
-
-describe('tasks actions', () => {
-  it('fetchUserTasks', async () => {
-    const tasks = [{_id: 1}];
-    sandbox.stub(axios, 'get').withArgs('/api/v3/tasks/user').returns(Promise.resolve({data: {data: tasks}}));
-
-    await fetchUserTasks(store);
-
-    expect(store.state.tasks).to.equal(tasks);
-  });
-});

test/client/unit/specs/store/actions/user.js

@@ -1,14 +0,0 @@
-import { fetch as fetchUser } from 'client/store/actions/user';
-import axios from 'axios';
-import store from 'client/store';
-
-describe('user actions', () => {
-  it('fetch', async () => {
-    const user = {_id: 1};
-    sandbox.stub(axios, 'get').withArgs('/api/v3/user').returns(Promise.resolve({data: {data: user}}));
-
-    await fetchUser(store);
-
-    expect(store.state.user).to.equal(user);
-  });
-});

test/common/libs/taskDefaults.test.js

@@ -12,9 +12,6 @@ describe('taskDefaults', () => {
     expect(task.up).to.eql(true);
     expect(task.down).to.eql(true);
     expect(task.history).to.eql([]);
-    expect(task.frequency).to.equal('daily');
-    expect(task.counterUp).to.equal(0);
-    expect(task.counterDown).to.equal(0);
   });
 
   it('applies defaults to a daily', () => {

test/common/ops/addTask.js

@@ -33,9 +33,6 @@ describe('shared.ops.addTask', () => {
     expect(habit.down).to.equal(false);
     expect(habit.history).to.eql([]);
     expect(habit.checklist).to.not.exist;
-    expect(habit.frequency).to.equal('daily');
-    expect(habit.counterUp).to.equal(0);
-    expect(habit.counterDown).to.equal(0);
   });
 
   it('adds an habtit when type is invalid', () => {

test/common/ops/buyArmoire.js

@@ -15,13 +15,13 @@ import i18n from '../../../website/common/script/i18n';
 function getFullArmoire () {
   let fullArmoire = {};
 
-  _.each(content.gearTypes, (type) => {
-    _.each(content.gear.tree[type].armoire, (gearObject) => {
+  _(content.gearTypes).each((type) => {
+    _(content.gear.tree[type].armoire).each((gearObject) => {
       let armoireKey = gearObject.key;
 
       fullArmoire[armoireKey] = true;
-    });
-  });
+    }).value();
+  }).value();
 
   return fullArmoire;
 }

test/common/ops/scoreTask.test.js

@@ -138,9 +138,6 @@ describe('shared.ops.scoreTask', () => {
       todo = generateTodo({ userId: ref.afterUser._id, text: 'some todo' });
 
       expect(habit.history.length).to.eql(0);
-      expect(habit.frequency).to.equal('daily');
-      expect(habit.counterUp).to.equal(0);
-      expect(habit.counterDown).to.equal(0);
 
       // before and after are the same user
       expect(ref.beforeUser._id).to.exist;
@@ -205,28 +202,17 @@ describe('shared.ops.scoreTask', () => {
 
         expect(habit.history.length).to.eql(1);
         expect(habit.value).to.be.greaterThan(0);
-        expect(habit.counterUp).to.equal(5);
 
         expect(ref.afterUser.stats.hp).to.eql(50);
         expect(ref.afterUser.stats.exp).to.be.greaterThan(ref.beforeUser.stats.exp);
         expect(ref.afterUser.stats.gp).to.be.greaterThan(ref.beforeUser.stats.gp);
       });
 
-      it('adds score notes', () => {
-        let scoreNotesString = 'scoreNotes';
-        habit.scoreNotes = scoreNotesString;
-        options = { user: ref.afterUser, task: habit, direction: 'up', times: 5, cron: false };
-        scoreTask(options);
-
-        expect(habit.history[0].scoreNotes).to.eql(scoreNotesString);
-      });
-
       it('down', () => {
         scoreTask({user: ref.afterUser, task: habit, direction: 'down', times: 5, cron: false}, {});
 
         expect(habit.history.length).to.eql(1);
         expect(habit.value).to.be.lessThan(0);
-        expect(habit.counterDown).to.equal(5);
 
         expect(ref.afterUser.stats.hp).to.be.lessThan(ref.beforeUser.stats.hp);
         expect(ref.afterUser.stats.exp).to.eql(0);

test/common/shouldDo.test.js

@@ -1,310 +0,0 @@
-// import { shouldDo, DAY_MAPPING } from '../../website/common/script/cron';
-// import moment from 'moment';
-// import 'moment-recur';
-
-// describe('shouldDo', () => {
-//   let day, dailyTask;
-//   let options = {};
-
-//   beforeEach(() => {
-//     day = new Date();
-//     dailyTask = {
-//       completed: 'false',
-//       everyX: 1,
-//       frequency: 'weekly',
-//       type: 'daily',
-//       repeat: {
-//         su: true,
-//         s: true,
-//         f: true,
-//         th: true,
-//         w: true,
-//         t: true,
-//         m: true,
-//       },
-//       startDate: new Date(),
-//     };
-//   });
-
-//   it('leaves Daily inactive before start date', () => {
-//     dailyTask.startDate = moment().add(1, 'days').toDate();
-
-//     expect(shouldDo(day, dailyTask, options)).to.equal(false);
-//   });
-
-//   context('Every X Days', () => {
-//     it('leaves Daily inactive in between X Day intervals', () => {
-//       dailyTask.startDate = moment().subtract(1, 'days').toDate();
-//       dailyTask.frequency = 'daily';
-//       dailyTask.everyX = 2;
-
-//       expect(shouldDo(day, dailyTask, options)).to.equal(false);
-//     });
-
-//     it('activates Daily on multiples of X Days', () => {
-//       dailyTask.startDate = moment().subtract(7, 'days').toDate();
-//       dailyTask.frequency = 'daily';
-//       dailyTask.everyX = 7;
-
-//       expect(shouldDo(day, dailyTask, options)).to.equal(true);
-//     });
-//   });
-
-//   context('Certain Days of the Week', () => {
-//     it('leaves Daily inactive if day of the week does not match', () => {
-//       dailyTask.repeat = {
-//         su: false,
-//         s: false,
-//         f: false,
-//         th: false,
-//         w: false,
-//         t: false,
-//         m: false,
-//       };
-
-//       for (let weekday of [0, 1, 2, 3, 4, 5, 6]) {
-//         day = moment().day(weekday).toDate();
-
-//         expect(shouldDo(day, dailyTask, options)).to.equal(false);
-//       }
-//     });
-
-//     it('leaves Daily inactive if day of the week does not match and active on the day it matches', () => {
-//       dailyTask.repeat = {
-//         su: false,
-//         s: false,
-//         f: false,
-//         th: true,
-//         w: false,
-//         t: false,
-//         m: false,
-//       };
-
-//       for (let weekday of [0, 1, 2, 3, 4, 5, 6]) {
-//         day = moment().add(1, 'weeks').day(weekday).toDate();
-
-//         if (weekday === 4) {
-//           expect(shouldDo(day, dailyTask, options)).to.equal(true);
-//         } else {
-//           expect(shouldDo(day, dailyTask, options)).to.equal(false);
-//         }
-//       }
-//     });
-
-//     it('activates Daily on matching days of the week', () => {
-//       expect(shouldDo(day, dailyTask, options)).to.equal(true);
-//     });
-//   });
-
-//   context('Every X Weeks', () => {
-//     it('leaves daily inactive if it has not been the specified number of weeks', () => {
-//       dailyTask.everyX = 3;
-//       let tomorrow = moment().add(1, 'day').toDate();
-
-//       expect(shouldDo(tomorrow, dailyTask, options)).to.equal(false);
-//     });
-
-//     it('leaves daily inactive if on every (x) week on weekday it is incorrect weekday', () => {
-//       dailyTask.repeat = {
-//         su: false,
-//         s: false,
-//         f: false,
-//         th: false,
-//         w: false,
-//         t: false,
-//         m: false,
-//       };
-
-//       day = moment();
-//       dailyTask.repeat[DAY_MAPPING[day.day()]] = true;
-//       dailyTask.everyX = 3;
-//       let threeWeeksFromTodayPlusOne = day.add(1, 'day').add(3, 'weeks').toDate();
-
-//       expect(shouldDo(threeWeeksFromTodayPlusOne, dailyTask, options)).to.equal(false);
-//     });
-
-//     it('activates Daily on matching week', () => {
-//       dailyTask.everyX = 3;
-//       let threeWeeksFromToday = moment().add(3, 'weeks').toDate();
-
-//       expect(shouldDo(threeWeeksFromToday, dailyTask, options)).to.equal(true);
-//     });
-
-//     it('activates Daily on every (x) week on weekday', () => {
-//       dailyTask.repeat = {
-//         su: false,
-//         s: false,
-//         f: false,
-//         th: false,
-//         w: false,
-//         t: false,
-//         m: false,
-//       };
-
-//       day = moment();
-//       dailyTask.repeat[DAY_MAPPING[day.day()]] = true;
-//       dailyTask.everyX = 3;
-//       let threeWeeksFromToday = day.add(6, 'weeks').day(day.day()).toDate();
-
-//       expect(shouldDo(threeWeeksFromToday, dailyTask, options)).to.equal(true);
-//     });
-//   });
-
-//   context('Monthly - Every X Months on a specified date', () => {
-//     it('leaves daily inactive if not day of the month', () => {
-//       dailyTask.everyX = 1;
-//       dailyTask.frequency = 'monthly';
-//       dailyTask.daysOfMonth = [15];
-//       let tomorrow = moment().add(1, 'day').toDate();// @TODO: make sure this is not the 15
-
-//       expect(shouldDo(tomorrow, dailyTask, options)).to.equal(false);
-//     });
-
-//     it('activates Daily on matching day of month', () => {
-//       day = moment();
-//       dailyTask.everyX = 1;
-//       dailyTask.frequency = 'monthly';
-//       dailyTask.daysOfMonth = [day.date()];
-//       day = day.add(1, 'months').date(day.date()).toDate();
-
-//       expect(shouldDo(day, dailyTask, options)).to.equal(true);
-//     });
-
-//     it('leaves daily inactive if not on date of the x month', () => {
-//       dailyTask.everyX = 2;
-//       dailyTask.frequency = 'monthly';
-//       dailyTask.daysOfMonth = [15];
-//       let tomorrow = moment().add(2, 'months').add(1, 'day').toDate();
-
-//       expect(shouldDo(tomorrow, dailyTask, options)).to.equal(false);
-//     });
-
-//     it('activates Daily if on date of the x month', () => {
-//       dailyTask.everyX = 2;
-//       dailyTask.frequency = 'monthly';
-//       dailyTask.daysOfMonth = [15];
-//       day = moment().add(2, 'months').date(15).toDate();
-//       expect(shouldDo(day, dailyTask, options)).to.equal(true);
-//     });
-//   });
-
-//   context('Monthly - Certain days of the nth Week', () => {
-//     it('leaves daily inactive if not the correct week of the month on the day of the start date', () => {
-//       dailyTask.repeat = {
-//         su: false,
-//         s: false,
-//         f: false,
-//         th: false,
-//         w: false,
-//         t: false,
-//         m: false,
-//       };
-
-//       let today = moment('01/27/2017');
-//       let week = today.monthWeek();
-//       let dayOfWeek = today.day();
-//       dailyTask.startDate = today.toDate();
-//       dailyTask.weeksOfMonth = [week];
-//       dailyTask.repeat[DAY_MAPPING[dayOfWeek]] = true;
-//       dailyTask.everyX = 1;
-//       dailyTask.frequency = 'monthly';
-//       day = moment('02/23/2017');
-
-//       expect(shouldDo(day, dailyTask, options)).to.equal(false);
-//     });
-
-//     it('activates Daily if correct week of the month on the day of the start date', () => {
-//       dailyTask.repeat = {
-//         su: false,
-//         s: false,
-//         f: false,
-//         th: false,
-//         w: false,
-//         t: false,
-//         m: false,
-//       };
-
-//       let today = moment('01/27/2017');
-//       let week = today.monthWeek();
-//       let dayOfWeek = today.day();
-//       dailyTask.startDate = today.toDate();
-//       dailyTask.weeksOfMonth = [week];
-//       dailyTask.repeat[DAY_MAPPING[dayOfWeek]] = true;
-//       dailyTask.everyX = 1;
-//       dailyTask.frequency = 'monthly';
-//       day = moment('02/24/2017');
-
-//       expect(shouldDo(day, dailyTask, options)).to.equal(true);
-//     });
-
-//     it('leaves daily inactive if not day of the month with every x month on weekday', () => {
-//       dailyTask.repeat = {
-//         su: false,
-//         s: false,
-//         f: false,
-//         th: false,
-//         w: false,
-//         t: false,
-//         m: false,
-//       };
-
-//       let today = moment('01/26/2017');
-//       let week = today.monthWeek();
-//       let dayOfWeek = today.day();
-//       dailyTask.startDate = today.toDate();
-//       dailyTask.weeksOfMonth = [week];
-//       dailyTask.repeat[DAY_MAPPING[dayOfWeek]] = true;
-//       dailyTask.everyX = 2;
-//       dailyTask.frequency = 'monthly';
-
-//       day = moment('03/24/2017');
-
-//       expect(shouldDo(day, dailyTask, options)).to.equal(false);
-//     });
-
-//     it('activates Daily if on nth weekday of the x month', () => {
-//       dailyTask.repeat = {
-//         su: false,
-//         s: false,
-//         f: false,
-//         th: false,
-//         w: false,
-//         t: false,
-//         m: false,
-//       };
-
-//       let today = moment('01/27/2017');
-//       let week = today.monthWeek();
-//       let dayOfWeek = today.day();
-//       dailyTask.startDate = today.toDate();
-//       dailyTask.weeksOfMonth = [week];
-//       dailyTask.repeat[DAY_MAPPING[dayOfWeek]] = true;
-//       dailyTask.everyX = 2;
-//       dailyTask.frequency = 'monthly';
-
-//       day = moment('03/24/2017');
-
-//       expect(shouldDo(day, dailyTask, options)).to.equal(true);
-//     });
-//   });
-
-//   context('Every X Years', () => {
-//     it('leaves daily inactive if not the correct year', () => {
-//       day = moment();
-//       dailyTask.everyX = 2;
-//       dailyTask.frequency = 'yearly';
-//       day = day.add(1, 'day').toDate();
-
-//       expect(shouldDo(day, dailyTask, options)).to.equal(false);
-//     });
-
-//     it('activates Daily on matching year', () => {
-//       day = moment();
-//       dailyTask.everyX = 2;
-//       dailyTask.frequency = 'yearly';
-//       day = day.add(2, 'years').toDate();
-
-//       expect(shouldDo(day, dailyTask, options)).to.equal(true);
-//     });
-//   });
-// });

test/content/faq.js

@@ -8,19 +8,19 @@ import {questions, stillNeedHelp} from '../../website/common/script/content/faq'
 describe('FAQ Locales', () => {
   describe('Questions', () => {
     it('has a valid questions', () => {
-      each(questions, (question) => {
+      each(questions, (question, key) => {
         expectValidTranslationString(question.question);
       });
     });
 
     it('has a valid ios answers', () => {
-      each(questions, (question) => {
+      each(questions, (question, key) => {
         expectValidTranslationString(question.ios);
       });
     });
 
     it('has a valid web answers', () => {
-      each(questions, (question) => {
+      each(questions, (question, key) => {
         expectValidTranslationString(question.web);
       });
     });

test/content/gear.js

@@ -1,4 +1,3 @@
-/* eslint-disable camelcase */
 import {
   expectValidTranslationString,
 } from '../helpers/content.helper';
@@ -69,7 +68,7 @@ describe('Gear', () => {
       weapon_special_0: 70,
       weapon_special_2: 300,
       weapon_special_3: 300,
-    };
+    }
 
     each(cases, (tierRequirement, key) => {
       context(key, () => {

test/content/mysterySets.test.js

@@ -1,13 +1,13 @@
 import {each} from 'lodash';
 import {
-  expectValidTranslationString,
+  expectValidTranslationString
 } from '../helpers/content.helper';
 
 import mysterySets from '../../website/common/script/content/mystery-sets';
 
 describe('Mystery Sets', () => {
   it('has a valid text string', () => {
-    each(mysterySets, (set) => {
+    each(mysterySets, (set, key) => {
       expectValidTranslationString(set.text);
     });
   });

test/content/time-travelers.test.js

@@ -1,28 +1,29 @@
+import _ from 'lodash';
 import {
   generateUser,
 } from '../helpers/common.helper';
 
-import timeTravelers from '../../website/common/script/content/time-travelers';
+import timeTravelers from '../../website/common/script/content/time-travelers'
 
 describe('time-travelers store', () => {
-  let user;
-  beforeEach(() => {
-    user = generateUser();
-  });
+	let user;
+	beforeEach(() => {
+		user = generateUser();
+	});
 
-  it('removes owned sets from the time travelers store', () => {
-    user.items.gear.owned.head_mystery_201602 = true; // eslint-disable-line camelcase
-    expect(timeTravelers.timeTravelerStore(user)['201602']).to.not.exist;
-    expect(timeTravelers.timeTravelerStore(user)['201603']).to.exist;
-  });
+	it('removes owned sets from the time travelers store', () => {
+		user.items.gear.owned['head_mystery_201602'] = true;
+		expect(timeTravelers.timeTravelerStore(user)['201602']).to.not.exist;
+		expect(timeTravelers.timeTravelerStore(user)['201603']).to.exist;
+	});
 
-  it('removes unopened mystery item sets from the time travelers store', () => {
-    user.purchased = {
-      plan: {
-        mysteryItems: ['head_mystery_201602'],
-      },
-    };
-    expect(timeTravelers.timeTravelerStore(user)['201602']).to.not.exist;
-    expect(timeTravelers.timeTravelerStore(user)['201603']).to.exist;
-  });
+	it('removes unopened mystery item sets from the time travelers store', () => {
+		user.purchased = {
+			plan: {
+				mysteryItems: ['head_mystery_201602'],
+			},
+		};
+		expect(timeTravelers.timeTravelerStore(user)['201602']).to.not.exist;
+		expect(timeTravelers.timeTravelerStore(user)['201603']).to.exist;
+	});
 });

test/helpers/api-integration/api-classes.js

@@ -1,5 +1,5 @@
 /* eslint-disable no-use-before-define */
-import moment from 'moment';
+
 import { requester } from './requester';
 import {
   getDocument as getDocumentFromMongo,
@@ -82,19 +82,6 @@ export class ApiGroup extends ApiObject {
 
     return await this.update(update);
   }
-
-  async createCancelledSubscription () {
-    let update = {
-      purchased: {
-        plan: {
-          customerId: 'example-customer',
-          dateTerminated: moment().add(1, 'days').toDate(),
-        },
-      },
-    };
-
-    return await this.update(update);
-  }
 }
 
 export class ApiChallenge extends ApiObject {

test/helpers/api-unit.helper.js

@@ -1,6 +1,6 @@
 import '../../website/server/libs/i18n';
 import mongoose from 'mongoose';
-import defaultsDeep from 'lodash/defaultsDeep';
+import { defaultsDeep as defaults } from 'lodash';
 import { model as User } from '../../website/server/models/user';
 import { model as Group } from '../../website/server/models/group';
 import { model as Challenge } from '../../website/server/models/challenge';
@@ -45,7 +45,7 @@ export function generateRes (options = {}) {
     },
   };
 
-  return defaultsDeep(options, defaultRes);
+  return defaults(options, defaultRes);
 }
 
 export function generateReq (options = {}) {
@@ -56,7 +56,7 @@ export function generateReq (options = {}) {
     header: sandbox.stub().returns(null),
   };
 
-  return defaultsDeep(options, defaultReq);
+  return defaults(options, defaultReq);
 }
 
 export function generateNext (func) {

website/assets/sprites/dist/spritesmith-largeSprites-0.css

@@ -1,30 +1,30 @@
 .promo_android {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1651px -180px;
+  background-position: -1547px -553px;
   width: 175px;
   height: 175px;
 }
 .promo_backgrounds_armoire_201602 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -565px -703px;
+  background-position: -845px -1041px;
   width: 141px;
   height: 294px;
 }
 .promo_backgrounds_armoire_201603 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -707px -703px;
+  background-position: -703px -1041px;
   width: 141px;
   height: 294px;
 }
 .promo_backgrounds_armoire_201604 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1265px -442px;
+  background-position: -282px -1041px;
   width: 140px;
   height: 441px;
 }
 .promo_backgrounds_armoire_201605 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1406px 0px;
+  background-position: -1124px 0px;
   width: 140px;
   height: 441px;
 }
@@ -54,37 +54,37 @@
 }
 .promo_backgrounds_armoire_201610 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -141px -1041px;
+  background-position: -1265px 0px;
   width: 140px;
   height: 441px;
 }
 .promo_backgrounds_armoire_201611 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1124px -442px;
+  background-position: -141px -1041px;
   width: 140px;
   height: 441px;
 }
 .promo_backgrounds_armoire_201612 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1265px 0px;
+  background-position: -1406px -442px;
   width: 140px;
   height: 441px;
 }
 .promo_backgrounds_armoire_201701 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1124px 0px;
+  background-position: -1124px -442px;
   width: 140px;
   height: 441px;
 }
 .promo_backgrounds_armoire_201702 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -840px 0px;
+  background-position: -982px 0px;
   width: 141px;
   height: 441px;
 }
 .promo_backgrounds_armoire_201703 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -982px 0px;
+  background-position: -840px 0px;
   width: 141px;
   height: 441px;
 }
@@ -96,79 +96,79 @@
 }
 .promo_chairs_glasses {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1757px -532px;
+  background-position: 0px -1483px;
   width: 51px;
   height: 210px;
 }
 .promo_checkin_incentives {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -423px -703px;
+  background-position: -987px -1041px;
   width: 141px;
   height: 294px;
 }
 .promo_classes_fall_2014 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1066px -1337px;
+  background-position: -1547px -452px;
   width: 321px;
   height: 100px;
 }
 .promo_classes_fall_2015 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -980px -1189px;
+  background-position: -703px -1336px;
   width: 377px;
   height: 99px;
 }
 .promo_classes_fall_2016 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1547px 0px;
+  background-position: -1547px -103px;
   width: 103px;
   height: 348px;
 }
 .promo_coffee_mug {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1651px 0px;
+  background-position: -1651px -103px;
   width: 200px;
   height: 179px;
 }
 .promo_contrib_spotlight_Keith {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1547px -1118px;
+  background-position: -90px -1694px;
   width: 87px;
   height: 111px;
 }
 .promo_contrib_spotlight_beffymaroo {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1406px -884px;
+  background-position: -1794px -729px;
   width: 114px;
   height: 147px;
 }
 .promo_contrib_spotlight_blade {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1547px -1006px;
+  background-position: 0px -1694px;
   width: 89px;
   height: 111px;
 }
 .promo_contrib_spotlight_cantras {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1547px -1230px;
+  background-position: -178px -1694px;
   width: 87px;
   height: 109px;
 }
 .promo_contrib_spotlight_megan {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1547px -894px;
+  background-position: -143px -1483px;
   width: 90px;
   height: 111px;
 }
 .promo_contrib_spotlight_shanaqui {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1547px -782px;
+  background-position: -52px -1483px;
   width: 90px;
   height: 111px;
 }
 .promo_cow {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: 0px -1041px;
+  background-position: -1406px 0px;
   width: 140px;
   height: 441px;
 }
@@ -180,67 +180,67 @@
 }
 .promo_dilatoryDistress {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1481px -1644px;
+  background-position: -1910px -970px;
   width: 90px;
   height: 90px;
 }
 .promo_egg_mounts {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -703px -1041px;
+  background-position: -824px -600px;
   width: 280px;
   height: 147px;
 }
 .promo_enchanted_armoire {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: 0px -1483px;
+  background-position: -1081px -1336px;
   width: 374px;
   height: 76px;
 }
 .promo_enchanted_armoire_201507 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -796px -1644px;
+  background-position: -853px -751px;
   width: 217px;
   height: 90px;
 }
 .promo_enchanted_armoire_201508 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1651px -1342px;
+  background-position: -1270px -1133px;
   width: 180px;
   height: 90px;
 }
 .promo_enchanted_armoire_201509 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -546px -1735px;
+  background-position: -1910px -1243px;
   width: 90px;
   height: 90px;
 }
 .promo_enchanted_armoire_201511 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -982px -442px;
+  background-position: -1270px -1224px;
   width: 122px;
   height: 90px;
 }
 .promo_enchanted_armoire_201601 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -819px -1735px;
+  background-position: -1910px -1425px;
   width: 90px;
   height: 90px;
 }
 .promo_floral_potions {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1651px -532px;
+  background-position: -1547px -729px;
   width: 105px;
   height: 273px;
 }
 .promo_ghost_potions {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1406px -442px;
+  background-position: -423px -1041px;
   width: 140px;
   height: 441px;
 }
 .promo_habitica {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1651px -356px;
+  background-position: -1723px -553px;
   width: 175px;
   height: 175px;
 }
@@ -252,13 +252,13 @@
 }
 .promo_habitoween_2016 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -282px -1041px;
+  background-position: -1265px -442px;
   width: 140px;
   height: 441px;
 }
 .promo_haunted_hair {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1547px -644px;
+  background-position: -840px -442px;
   width: 100px;
   height: 137px;
 }
@@ -270,199 +270,193 @@
 }
 .promo_item_notif {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -853px -600px;
+  background-position: -1651px -283px;
   width: 249px;
   height: 102px;
 }
-.promo_jackalope {
-  background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -703px -1189px;
-  width: 276px;
-  height: 147px;
-}
 .promo_mystery_201405 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: 0px -1735px;
+  background-position: -1910px -1698px;
   width: 90px;
   height: 90px;
 }
 .promo_mystery_201406 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1449px -1189px;
+  background-position: -1910px -418px;
   width: 90px;
   height: 96px;
 }
 .promo_mystery_201407 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1809px -669px;
+  background-position: -306px -536px;
   width: 42px;
   height: 62px;
 }
 .promo_mystery_201408 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1051px -915px;
+  background-position: -1469px -1041px;
   width: 60px;
   height: 71px;
 }
 .promo_mystery_201409 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1390px -1644px;
+  background-position: -1910px -1607px;
   width: 90px;
   height: 90px;
 }
 .promo_mystery_201410 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -982px -533px;
+  background-position: -1836px -1351px;
   width: 72px;
   height: 63px;
 }
 .promo_mystery_201411 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1663px -1644px;
+  background-position: -1910px -1516px;
   width: 90px;
   height: 90px;
 }
 .promo_mystery_201412 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1809px -602px;
+  background-position: -1071px -751px;
   width: 42px;
   height: 66px;
 }
 .promo_mystery_201501 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1792px -806px;
+  background-position: -1852px -103px;
   width: 48px;
   height: 63px;
 }
 .promo_mystery_201502 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -273px -1735px;
+  background-position: -1910px -515px;
   width: 90px;
   height: 90px;
 }
 .promo_mystery_201503 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -364px -1735px;
+  background-position: -1910px -1061px;
   width: 90px;
   height: 90px;
 }
 .promo_mystery_201504 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1482px -1337px;
+  background-position: -1456px -1336px;
   width: 60px;
   height: 69px;
 }
 .promo_mystery_201505 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -728px -1735px;
+  background-position: -1910px -788px;
   width: 90px;
   height: 90px;
 }
 .promo_mystery_201506 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1809px -532px;
+  background-position: -1852px -167px;
   width: 42px;
   height: 69px;
 }
 .promo_mystery_201507 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -990px -703px;
+  background-position: -1910px -106px;
   width: 90px;
   height: 105px;
 }
 .promo_mystery_201508 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1014px -1644px;
+  background-position: -146px -1595px;
   width: 93px;
   height: 90px;
 }
 .promo_mystery_201509 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1572px -1644px;
+  background-position: -1910px -606px;
   width: 90px;
   height: 90px;
 }
 .promo_mystery_201510 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1388px -1337px;
+  background-position: -240px -1595px;
   width: 93px;
   height: 90px;
 }
 .promo_mystery_201511 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1754px -1644px;
+  background-position: -428px -1595px;
   width: 90px;
   height: 90px;
 }
 .promo_mystery_201512 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -990px -915px;
+  background-position: -1848px -1171px;
   width: 60px;
   height: 81px;
 }
 .promo_mystery_201601 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -840px -442px;
+  background-position: -1393px -1224px;
   width: 120px;
   height: 90px;
 }
 .promo_mystery_201602 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -182px -1735px;
+  background-position: -1910px -1334px;
   width: 90px;
   height: 90px;
 }
 .promo_mystery_201603 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -91px -1735px;
+  background-position: -1910px -879px;
   width: 90px;
   height: 90px;
 }
 .promo_mystery_201604 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1296px -1644px;
+  background-position: -52px -1595px;
   width: 93px;
   height: 90px;
 }
 .promo_mystery_201605 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -455px -1735px;
+  background-position: -1910px -697px;
   width: 90px;
   height: 90px;
 }
 .promo_mystery_201606 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -699px -448px;
+  background-position: -1910px 0px;
   width: 90px;
   height: 105px;
 }
 .promo_mystery_201607 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -637px -1735px;
+  background-position: -1910px -1152px;
   width: 90px;
   height: 90px;
 }
 .promo_mystery_201608 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1108px -1644px;
+  background-position: -334px -1595px;
   width: 93px;
   height: 90px;
 }
 .promo_mystery_201609 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1202px -1644px;
+  background-position: -1451px -1133px;
   width: 93px;
   height: 90px;
 }
 .promo_mystery_201610 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1771px -1194px;
+  background-position: -1836px -1260px;
   width: 63px;
   height: 84px;
 }
 .promo_mystery_201611 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1358px -1189px;
+  background-position: -1910px -318px;
   width: 90px;
   height: 99px;
 }
@@ -474,151 +468,169 @@
 }
 .promo_mystery_201701 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -990px -809px;
+  background-position: -1910px -212px;
   width: 90px;
   height: 105px;
 }
 .promo_mystery_201702 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1264px -1041px;
+  background-position: -703px -854px;
   width: 279px;
   height: 147px;
 }
 .promo_mystery_3014 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -578px -1644px;
+  background-position: -1653px -877px;
   width: 217px;
   height: 90px;
 }
 .promo_new_hair_fall2016 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -423px -1041px;
+  background-position: 0px -1041px;
   width: 140px;
   height: 441px;
 }
 .promo_orca {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1124px -884px;
+  background-position: -983px -854px;
   width: 105px;
   height: 105px;
 }
 .promo_partyhats {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1651px -1433px;
+  background-position: -1651px -386px;
   width: 115px;
   height: 47px;
 }
 .promo_pastel_skin {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: 0px -1560px;
+  background-position: -1547px -1087px;
   width: 330px;
   height: 83px;
 }
 .customize-option.promo_pastel_skin {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -25px -1575px;
+  background-position: -1572px -1102px;
   width: 60px;
   height: 60px;
 }
 .promo_peppermint_flame {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1651px -954px;
+  background-position: -1406px -884px;
   width: 140px;
   height: 147px;
 }
 .promo_pet_skins {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1651px -806px;
+  background-position: -1653px -729px;
   width: 140px;
   height: 147px;
 }
 .customize-option.promo_pet_skins {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1676px -821px;
+  background-position: -1678px -744px;
   width: 60px;
   height: 60px;
 }
 .promo_pyromancer {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1265px -884px;
+  background-position: -699px -448px;
   width: 113px;
   height: 113px;
 }
 .promo_rainbow_armor {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1547px -1340px;
+  background-position: -266px -1694px;
   width: 92px;
   height: 103px;
 }
 .promo_seasonal_shop_fall_2016 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -984px -1041px;
+  background-position: -423px -854px;
   width: 279px;
   height: 147px;
 }
 .promo_shimmer_hair {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -331px -1560px;
+  background-position: -1547px -1003px;
   width: 330px;
   height: 83px;
 }
 .promo_splashyskins {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1651px -1102px;
+  background-position: -1270px -1041px;
   width: 198px;
   height: 91px;
 }
 .customize-option.promo_splashyskins {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1676px -1117px;
+  background-position: -1295px -1056px;
   width: 60px;
   height: 60px;
 }
 .promo_spooky_sparkles_fall_2016 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -849px -703px;
+  background-position: -1129px -1041px;
   width: 140px;
   height: 294px;
 }
 .promo_spring_classes_2016 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -703px -1337px;
+  background-position: -1547px 0px;
   width: 362px;
   height: 102px;
 }
 .promo_springclasses2014 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: 0px -1644px;
+  background-position: -1547px -1351px;
   width: 288px;
   height: 90px;
 }
 .promo_springclasses2015 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -289px -1644px;
+  background-position: -1547px -1260px;
   width: 288px;
   height: 90px;
 }
 .promo_staff_spotlight_Lemoness {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1547px -349px;
+  background-position: -1124px -884px;
   width: 102px;
   height: 146px;
 }
 .promo_staff_spotlight_Viirus {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1651px -1194px;
+  background-position: -1265px -884px;
   width: 119px;
   height: 147px;
 }
 .promo_staff_spotlight_paglias {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -1547px -496px;
+  background-position: -982px -442px;
   width: 99px;
   height: 147px;
 }
 .promo_summer_classes_2014 {
   background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
-  background-position: -423px -600px;
+  background-position: -423px -751px;
   width: 429px;
   height: 102px;
 }
+.promo_summer_classes_2015 {
+  background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
+  background-position: -1547px -1171px;
+  width: 300px;
+  height: 88px;
+}
+.promo_summer_classes_2016 {
+  background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
+  background-position: -423px -600px;
+  width: 400px;
+  height: 150px;
+}
+.promo_takethis_armor {
+  background-image: url(/static/sprites/spritesmith-largeSprites-0.png);
+  background-position: -234px -1483px;
+  width: 114px;
+  height: 87px;
+}

website/assets/sprites/dist/spritesmith-largeSprites-1.css

@@ -1,30 +1,12 @@
-.promo_summer_classes_2015 {
-  background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -553px -615px;
-  width: 300px;
-  height: 88px;
-}
-.promo_summer_classes_2016 {
-  background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -452px -145px;
-  width: 400px;
-  height: 150px;
-}
 .promo_takeThis_gear {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -441px -882px;
-  width: 114px;
-  height: 87px;
-}
-.promo_takethis_armor {
-  background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -556px -882px;
+  background-position: -643px -732px;
   width: 114px;
   height: 87px;
 }
 .promo_task_planning {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -885px -96px;
+  background-position: -282px -536px;
   width: 240px;
   height: 195px;
 }
@@ -36,31 +18,31 @@
 }
 .promo_unconventional_armor {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -1283px -666px;
+  background-position: -1108px -558px;
   width: 60px;
   height: 60px;
 }
 .promo_unconventional_armor2 {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -1283px -591px;
+  background-position: -1108px -483px;
   width: 70px;
   height: 74px;
 }
 .promo_updos {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -1195px -292px;
+  background-position: -885px -655px;
   width: 156px;
   height: 147px;
 }
 .promo_veteran_pets {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -553px -704px;
+  background-position: -1038px -803px;
   width: 146px;
   height: 75px;
 }
 .promo_winter_classes_2016 {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -452px -296px;
+  background-position: -282px -732px;
   width: 360px;
   height: 90px;
 }
@@ -72,13 +54,13 @@
 }
 .promo_winter_fireworks {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -302px -882px;
+  background-position: -1042px -655px;
   width: 138px;
   height: 147px;
 }
 .promo_winterclasses2015 {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -885px -757px;
+  background-position: -452px -326px;
   width: 325px;
   height: 110px;
 }
@@ -96,19 +78,19 @@
 }
 .promo_winteryhair {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -1211px -757px;
+  background-position: -885px -803px;
   width: 152px;
   height: 75px;
 }
 .avatar_variety {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -885px 0px;
+  background-position: -282px -440px;
   width: 498px;
   height: 95px;
 }
 .npc_viirus {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: 0px -1033px;
+  background-position: -453px -882px;
   width: 108px;
   height: 90px;
 }
@@ -120,7 +102,7 @@
 }
 .promo_backtoschool {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -151px -882px;
+  background-position: 0px -882px;
   width: 150px;
   height: 150px;
 }
@@ -132,61 +114,55 @@
 }
 .promo_startingover {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -1132px -591px;
+  background-position: -151px -882px;
   width: 150px;
   height: 150px;
 }
 .promo_valentines {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -885px -292px;
+  background-position: -523px -536px;
   width: 309px;
   height: 147px;
 }
 .promo_working_out {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -885px -440px;
+  background-position: -885px 0px;
   width: 300px;
   height: 150px;
 }
 .scene_coding {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -1186px -440px;
+  background-position: -302px -882px;
   width: 150px;
   height: 150px;
 }
 .scene_eco_friendly {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -589px -440px;
+  background-position: -885px -483px;
   width: 222px;
   height: 171px;
 }
-.scene_habits {
-  background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -282px -440px;
-  width: 306px;
-  height: 174px;
-}
 .scene_phone_peek {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: 0px -882px;
+  background-position: -723px -145px;
   width: 150px;
   height: 150px;
 }
 .welcome_basic_avatars {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -1126px -96px;
+  background-position: -885px -151px;
   width: 246px;
   height: 165px;
 }
 .welcome_promo_party {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -282px -615px;
+  background-position: -452px -145px;
   width: 270px;
   height: 180px;
 }
 .welcome_sample_tasks {
   background-image: url(/static/sprites/spritesmith-largeSprites-1.png);
-  background-position: -885px -591px;
+  background-position: -885px -317px;
   width: 246px;
   height: 165px;
 }