Deploy stages/rc-2017-08-07 to staging

.eslintrc

@@ -40,6 +40,8 @@
       'app/radio-btn',
       'app/print-personal-key',
       'app/utils/ms-formatter',
+      'app/phone-internationalization',
+      'app/i18n-dropdown',
     ],
   }
 }

.reek

@@ -14,6 +14,7 @@ DuplicateMethodCall:
     - UserFlowExporter#self.massage_assets
 FeatureEnvy:
   exclude:
+    - ActiveJob::Logging::LogSubscriber#json_for
     - track_registration
     - append_info_to_payload
     - generate_slo_request
@@ -24,6 +25,8 @@ FeatureEnvy:
     - Pii::Attributes#[]=
     - OpenidConnectLogoutForm#load_identity
     - Idv::ProfileMaker#pii_from_applicant
+    - Idv::Step#vendor_validator_result
+    - IdvSession#vendor_result_timed_out?
 InstanceVariableAssumption:
   exclude:
     - User
@@ -41,6 +44,8 @@ NilCheck:
 LongParameterList:
   exclude:
     - IdentityLinker#optional_attributes
+    - VendorValidatorJob#perform
+    - Idv::VendorResult#initialize
 RepeatedConditional:
   exclude:
     - Users::ResetPasswordsController
@@ -53,6 +58,7 @@ TooManyInstanceVariables:
   exclude:
     - OpenidConnectAuthorizeForm
     - OpenidConnectRedirector
+    - Idv::VendorResult
 TooManyStatements:
   max_statements: 6
   exclude:
@@ -72,6 +78,7 @@ TooManyMethods:
     - OpenidConnect::AuthorizationController
     - Idv::Session
     - User
+    - Verify::SessionsController
 UncommunicativeMethodName:
   exclude:
     - PhoneConfirmationFlow
@@ -89,6 +96,7 @@ UtilityFunction:
   public_methods_only: true
   exclude:
     - AnalyticsEventJob#perform
+    - ApplicationController#default_url_options
     - ApplicationHelper#step_class
     - PersonalKeyFormatter#regexp
     - SessionTimeoutWarningHelper#frequency
@@ -97,6 +105,9 @@ UtilityFunction:
     - SessionDecorator
     - WorkerHealthChecker::Middleware#call
     - UserEncryptedAttributeOverrides#create_fingerprint
+    - LocaleHelper#locale_url_param
+    - Verify::Base#mock_vendor_partial
+    - IdvSession#timed_out_vendor_error
 'app/controllers':
   InstanceVariableAssumption:
     enabled: false

.rubocop.yml

@@ -47,6 +47,7 @@ Metrics/BlockLength:
     - 'config/initializers/secure_headers.rb'
     - 'config/routes.rb'
     - 'spec/**/*.rb'
+    - 'config/initializers/active_job_logger_patch.rb'
 
 Metrics/ClassLength:
   Description: Avoid classes longer than 100 lines of code.
@@ -93,6 +94,9 @@ Metrics/ModuleLength:
   - spec/**/*
   - 'app/controllers/concerns/two_factor_authenticatable.rb'
 
+Metrics/ParameterLists:
+  CountKeywordArgs: false
+
 # This is a Rails 5 feature, so it should be disabled until we upgrade
 Rails/HttpPositionalArguments:
   Description: 'Use keyword arguments instead of positional arguments in http method calls.'

Gemfile

@@ -24,8 +24,10 @@ gem 'http_accept_language'
 gem 'httparty'
 gem 'json-jwt'
 gem 'lograge'
+gem 'net-sftp'
 gem 'newrelic_rpm'
 gem 'pg'
+gem 'phonelib'
 gem 'phony_rails'
 gem 'premailer-rails'
 gem 'proofer', github: '18F/identity-proofer-gem', branch: 'master'
@@ -113,4 +115,5 @@ end
 
 group :production do
   gem 'equifax', git: 'git@github.com:18F/identity-equifax-api-client-gem.git', branch: 'master'
+  gem 'mandrill_dm'
 end

Gemfile.lock

@@ -1,6 +1,6 @@
 GIT
   remote: git@github.com:18F/identity-equifax-api-client-gem.git
-  revision: 4308a502baf7b65e8b463ecafc2d428d530b4349
+  revision: 889aad815bda2ff2a41cd2b108e2afae7f50d8b8
   branch: master
   specs:
     equifax (1.0.0)
@@ -148,7 +148,7 @@ GEM
     bullet (5.5.1)
       activesupport (>= 3.0.0)
       uniform_notifier (~> 1.10.0)
-    bummr (0.2.0)
+    bummr (0.1.8)
       rainbow
       thor
     byebug (9.0.6)
@@ -247,6 +247,7 @@ GEM
     errbase (0.0.3)
     erubis (2.7.0)
     eventmachine (1.0.9.1)
+    excon (0.57.0)
     execjs (2.7.0)
     factory_girl (4.8.0)
       activesupport (>= 3.0.0)
@@ -300,7 +301,7 @@ GEM
     httpi (2.4.2)
       rack
       socksify
-    i18n (0.8.4)
+    i18n (0.8.6)
     i18n-tasks (0.9.15)
       activesupport (>= 4.0.2)
       ast (>= 2.1.0)
@@ -314,7 +315,7 @@ GEM
     ice_nine (0.11.2)
     iniparse (1.4.2)
     jmespath (1.3.1)
-    json (2.1.0)
+    json (1.8.6)
     json-jwt (1.7.2)
       activesupport
       bindata
@@ -348,6 +349,12 @@ GEM
       skinny (~> 0.2.3)
       sqlite3 (~> 1.3)
       thin (~> 1.5.0)
+    mandrill-api (1.0.53)
+      excon (>= 0.16.0, < 1.0)
+      json (>= 1.7.7, < 2.0)
+    mandrill_dm (1.3.4)
+      mail (~> 2.6)
+      mandrill-api (~> 1.0.53)
     memory_profiler (0.9.8)
     method_source (0.8.2)
     mime-types (3.1)
@@ -360,6 +367,8 @@ GEM
     nenv (0.3.0)
     net-scp (1.2.1)
       net-ssh (>= 2.6.5)
+    net-sftp (2.1.2)
+      net-ssh (>= 2.6.5)
     net-ssh (4.1.0)
     newrelic_rpm (4.2.0.334)
     nokogiri (1.8.0)
@@ -376,6 +385,7 @@ GEM
     parser (2.4.0.0)
       ast (~> 2.2)
     pg (0.21.0)
+    phonelib (0.6.12)
     phony (2.15.44)
     phony_rails (0.14.6)
       activesupport (>= 3.0)
@@ -696,9 +706,12 @@ DEPENDENCIES
   json-jwt
   lograge
   mailcatcher
+  mandrill_dm
+  net-sftp
   newrelic_rpm
   overcommit
   pg
+  phonelib
   phony_rails
   poltergeist
   premailer-rails

app/assets/javascripts/app/components/accordion.js

@@ -72,6 +72,7 @@ class Accordion extends Events {
     this.content.classList.add('shown');
     this.content.classList.remove('animate-out');
     this.content.classList.add('animate-in');
+    this.content.setAttribute('aria-hidden', 'false');
     this.emit('accordion.show');
   }
 
@@ -81,6 +82,7 @@ class Accordion extends Events {
     this.shownIcon.classList.add('display-none');
     this.content.classList.remove('animate-in');
     this.content.classList.add('animate-out');
+    this.content.setAttribute('aria-hidden', 'true');
     this.emit('accordion.hide');
     this.header.focus();
   }

app/assets/javascripts/app/form-field-format.js

@@ -1,5 +1,6 @@
-import { PhoneFormatter, SocialSecurityNumberFormatter, TextField } from 'field-kit';
+import { SocialSecurityNumberFormatter, TextField } from 'field-kit';
 import DateFormatter from './modules/date-formatter';
+import InternationalPhoneFormatter from './modules/international-phone-formatter';
 import NumericFormatter from './modules/numeric-formatter';
 import PersonalKeyFormatter from './modules/personal-key-formatter';
 import ZipCodeFormatter from './modules/zip-code-formatter';
@@ -13,7 +14,7 @@ function formatForm() {
     ['.home_equity_line', new NumericFormatter()],
     ['.mfa', new NumericFormatter()],
     ['.mortgage', new NumericFormatter()],
-    ['.phone', new PhoneFormatter()],
+    ['.phone', new InternationalPhoneFormatter()],
     ['.personal-key', new PersonalKeyFormatter()],
     ['.ssn', new SocialSecurityNumberFormatter()],
     ['.zipcode', new ZipCodeFormatter()],

app/assets/javascripts/app/form-validation.js

@@ -10,7 +10,7 @@ document.addEventListener('DOMContentLoaded', () => {
       if (input) {
         input.addEventListener('input', () => {
           if (input.validity.patternMismatch) {
-            input.setCustomValidity(I18n.t(`idv.errors.pattern_mismatch.${f}`));
+            input.setCustomValidity(I18n.t(`idv.errors.pattern_mismatch.${I18n.key(f)}`));
           } else {
             input.setCustomValidity('');
           }

app/assets/javascripts/app/i18n-dropdown.js

@@ -0,0 +1,18 @@
+import 'classlist.js';
+
+document.addEventListener('DOMContentLoaded', () => {
+  const mobileLink = document.querySelector('.i18n-mobile-toggle');
+  const mobileDropdown = document.querySelector('.i18n-mobile-dropdown');
+  const desktopLink = document.querySelector('.i18n-desktop-toggle');
+  const desktopDropdown = document.querySelector('.i18n-desktop-dropdown');
+
+  function initDropdown (trigger, dropdown) {
+    trigger.addEventListener('click', function() {
+      this.classList.toggle('focused');
+      dropdown.classList.toggle('focused');
+    });
+  }
+
+  if (mobileLink) initDropdown(mobileLink, mobileDropdown);
+  if (desktopLink) initDropdown(desktopLink, desktopDropdown);
+});

app/assets/javascripts/app/modules/international-phone-formatter.js

@@ -0,0 +1,69 @@
+import { Formatter } from 'field-kit';
+import { asYouType as AsYouType } from 'libphonenumber-js';
+
+const fixCountryCodeSpacing = (text, countryCode) => {
+  // If the text is `+123456`, make it `+123 456`
+  if (text[countryCode.length + 1] !== ' ') {
+    return text.replace(`+${countryCode}`, `+${countryCode} `);
+  }
+  return text;
+};
+
+const getFormattedTextData = (text) => {
+  if (text === '1') {
+    text = '+1';
+  }
+
+  const asYouType = new AsYouType('US');
+  let formattedText = asYouType.input(text);
+  const countryCode = asYouType.country_phone_code;
+
+  if (asYouType.country_phone_code) {
+    formattedText = fixCountryCodeSpacing(formattedText, countryCode);
+  }
+
+  return {
+    text: formattedText,
+    template: asYouType.template,
+    countryCode,
+  };
+};
+
+const cursorPosition = (formattedTextData) => {
+  // If the text is `(23 )` the cursor goes after the 3
+  const match = formattedTextData.text.match(/\d[^\d]*$/);
+  if (match) {
+    return match.index + 1;
+  }
+  return formattedTextData.text.length + 1;
+};
+
+class InternationalPhoneFormatter extends Formatter {
+  format(text) {
+    const formattedTextData = getFormattedTextData(text);
+    return super.format(formattedTextData.text);
+  }
+
+  // eslint-disable-next-line class-methods-use-this
+  parse(text) {
+    return text.replace(/[^\d+]/g, '');
+  }
+
+  isChangeValid(change, error) {
+    const formattedTextData = getFormattedTextData(change.proposed.text);
+    const previousFormattedTextData = getFormattedTextData(change.current.text);
+
+    if (previousFormattedTextData.template &&
+      !formattedTextData.template &&
+      change.inserted.text.length === 1
+    ) {
+      return false;
+    }
+
+    change.proposed.text = formattedTextData.text;
+    change.proposed.selectedRange.start = cursorPosition(formattedTextData);
+    return super.isChangeValid(change, error);
+  }
+}
+
+export default InternationalPhoneFormatter;