fix(metrics): remove the forgot_sent amplitude event (#2078) r=vladikoff

Author: philbooth

lib/metrics/amplitude.js

@@ -61,14 +61,6 @@ const EVENTS = {
     group: (request, data, metricsContext) => GROUPS[metricsContext.flowType],
     event: 'complete'
   },
-  'password.forgot.resend_code.completed': {
-    group: GROUPS.login,
-    event: 'forgot_sent'
-  },
-  'password.forgot.send_code.completed': {
-    group: GROUPS.login,
-    event: 'forgot_sent'
-  },
   'sms.installFirefox.sent': {
     group: GROUPS.sms,
     event: 'sent'

test/local/metrics/amplitude.js

@@ -316,38 +316,6 @@ describe('metrics/amplitude', () => {
       })
     })
 
-    describe('password.forgot.resend_code.completed', () => {
-      beforeEach(() => {
-        amplitude('password.forgot.resend_code.completed', mocks.mockRequest({}))
-      })
-
-      it('did not call log.error', () => {
-        assert.equal(log.error.callCount, 0)
-      })
-
-      it('called log.amplitudeEvent correctly', () => {
-        assert.equal(log.amplitudeEvent.callCount, 1)
-        const args = log.amplitudeEvent.args[0]
-        assert.equal(args[0].event_type, 'fxa_login - forgot_sent')
-      })
-    })
-
-    describe('password.forgot.send_code.completed', () => {
-      beforeEach(() => {
-        amplitude('password.forgot.send_code.completed', mocks.mockRequest({}))
-      })
-
-      it('did not call log.error', () => {
-        assert.equal(log.error.callCount, 0)
-      })
-
-      it('called log.amplitudeEvent correctly', () => {
-        assert.equal(log.amplitudeEvent.callCount, 1)
-        const args = log.amplitudeEvent.args[0]
-        assert.equal(args[0].event_type, 'fxa_login - forgot_sent')
-      })
-    })
-
     describe('sms.installFirefox.sent', () => {
       beforeEach(() => {
         amplitude('sms.installFirefox.sent', mocks.mockRequest({}))