fix(project): move mailer files into proper directories (#1676) r=vladikoff

Author: philbooth

.gitignore

@@ -17,5 +17,5 @@ secret*
 /.vagrant
 .nyc_output
 npm-debug.log
-mailer/server.pot
-mailer/.mail_output
+server.pot
+.mail_output

.nsprc

@@ -1,4 +1,7 @@
 {
   "exceptions": [
+    // i18n-abide@0.0.25
+    "https://nodesecurity.io/advisories/39",
+    "https://nodesecurity.io/advisories/48"
   ]
 }

.travis.yml

@@ -46,10 +46,7 @@ script:
   - COVERALLS_REPO_TOKEN=vKN3jjhAOwxkv9HG0VBX4EYIlWLPwiJ9d npm test
   - npm run test-e2e
   # Test fxa-auth-mailer
-  - cd mailer
   - grunt templates
-  - npm test
   - grunt l10n-extract
-  - cd ..
   # NSP check
   - grunt nsp

Gruntfile.js

@@ -5,7 +5,12 @@
 module.exports = function (grunt) {
   require('load-grunt-tasks')(grunt)
 
+  grunt.initConfig({
+    pkg: grunt.file.readJSON('package.json')
+  })
+
   grunt.loadTasks('grunttasks')
 
   grunt.registerTask('default', ['eslint', 'copyright'])
+  grunt.registerTask('mailer', ['templates', 'copy:strings', 'l10n-extract'])
 }

mailer/bin/server.js renamed to bin/mailer_server.js

@@ -3,26 +3,26 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 var restify = require('restify')
-var config = require('../config')
+var config = require('../mailer/config')
 
-var log = require('../log')('server')
+var log = require('../lib/senders/log')('server')
 var mailConfig = config.get('mail')
 
 var packageJson = require('../package.json')
 var P = require('bluebird')
 
 // NOTE: Mailer is also used by fxa-auth-server directly with an old logging interface
 // the legacy log module provides an interface to convert old logs to new mozlog logging.
-var mailerLog = require('../log')('mailer')
-var legacyMailerLog = require('../legacy_log')(mailerLog)
-var Mailer = require('../mailer')(legacyMailerLog)
+var mailerLog = require('../lib/senders/log')('mailer')
+var legacyMailerLog = require('../lib/senders/legacy_log')(mailerLog)
+var Mailer = require('../lib/senders/email')(legacyMailerLog)
 
-var dbConnect = require('../lib/db_connect')()
+var dbConnect = require('../lib/senders/db_connect')()
 
 P.all(
   [
-    require('../translator')(config.get('locales'), config.get('defaultLanguage')),
-    require('../templates')()
+    require('../lib/senders/translator')(config.get('locales'), config.get('defaultLanguage')),
+    require('../lib/senders/templates')()
   ]
 )
 .spread(
@@ -35,7 +35,7 @@ P.all(
     dbConnect()
       .then(function (db) {
         // fetch and process verification reminders
-        var verificationReminders = require('../lib/verification-reminders')(mailer, db)
+        var verificationReminders = require('../lib/senders/verification-reminders')(mailer, db)
         verificationReminders.poll()
       })
       .catch(function (err) {

grunttasks/bump.js

@@ -13,7 +13,7 @@ module.exports = function (grunt) {
       bumpVersion: true,
       commit: true,
       commitMessage: 'Release v%VERSION%',
-      commitFiles: ['package.json', 'npm-shrinkwrap.json', 'CHANGELOG.md'],
+      commitFiles: ['package.json', 'npm-shrinkwrap.json', 'CHANGELOG.md', 'AUTHORS'],
       createTag: true,
       tagName: 'v%VERSION%',
       tagMessage: 'Version %VERSION%',

mailer/grunttasks/l10n-extract.js renamed to grunttasks/l10n-extract.js

@@ -30,7 +30,7 @@ module.exports = function (grunt) {
     var done = this.async()
 
     var walker = extract({
-      'input-dir': path.join(pkgroot, 'templates'),
+      'input-dir': path.join(pkgroot, 'lib/senders/templates'),
       'output-dir': pkgroot,
       'output': 'server.pot',
       'join-existing': true,
@@ -43,12 +43,7 @@ module.exports = function (grunt) {
 
     walker.on('end', function () {
       var jsWalker = extract({
-        'input-dir': pkgroot,
-        /* node_modules and test should not contain any strings
-         * Gruntfile causes an error and should contain no strings
-         * bin/server.js extracts "/", so it is excluded.
-         */
-        exclude: /(node_modules|test|Gruntfile|grunttasks|bin|scripts|\.git|config)/,
+        'input-dir': path.join(pkgroot, 'lib/senders'),
         'output-dir': pkgroot,
         'output': 'server.pot',
         'join-existing': true,

mailer/grunttasks/templates.js renamed to grunttasks/templates.js

@@ -9,6 +9,7 @@ module.exports = function (grunt) {
 
   grunt.config('nunjucks', {
     options: {
+      paths: 'lib/senders/',
       tags: {
         blockStart: '<%',
         blockEnd: '%>',
@@ -23,9 +24,9 @@ module.exports = function (grunt) {
       files: [
         {
           expand: true,
-          cwd: 'partials/',
+          cwd: 'lib/senders/partials/',
           src: '*.html',
-          dest: 'templates/',
+          dest: 'lib/senders/templates/',
           ext: '.html'
         }
       ]

mailer/lib/db.js renamed to lib/senders/db.js

@@ -2,9 +2,9 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
-var butil = require('./crypto/butil')
-var log = require('../log')('db')
-var P = require('./promise')
+var butil = require('../crypto/butil')
+var log = require('./log')('db')
+var P = require('../promise')
 var Pool = require('./pool')
 var qs = require('querystring')
 
@@ -40,7 +40,7 @@ module.exports = function () {
       .then(
         function (body) {
           var data = bufferize(body)
-          data.emailVerified = !!data.emailVerified
+          data.emailVerified = !! data.emailVerified
           return data
         },
         function (err) {
@@ -55,7 +55,7 @@ module.exports = function () {
       .then(
         function (body) {
           var data = bufferize(body)
-          data.emailVerified = !!data.emailVerified
+          data.emailVerified = !! data.emailVerified
           return data
         },
         function (err) {

mailer/lib/db_connect.js renamed to lib/senders/db_connect.js

@@ -3,8 +3,8 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 var P = require('bluebird')
-var config = require('../config')
-var log = require('../log')('db')
+var config = require('../../mailer/config')
+var log = require('./log')('db')
 
 var DB = require('./db')()
 
@@ -29,10 +29,10 @@ module.exports = function () {
           function (err) {
             if (err && err.message && err.message.indexOf('ECONNREFUSED') > -1) {
               log.warn('db', {message: 'Failed to connect to database, retrying...'})
-              if (!cancelled) {
+              if (! cancelled) {
                 return P.delay(dbConnectionRetry)
                   .then(function () {
-                    if (!cancelled) {
+                    if (! cancelled) {
                       return dbConnectPoll()
                     }
                   })
@@ -51,7 +51,7 @@ module.exports = function () {
         // report any errors to the db log and rethrow it to the consumer.
         log.error('db', {err: err})
         throw err
-      });
+      })
   }
 
   return dbConnect