fixes for rubocop

app/controllers/profiles/emails_controller.rb

 class Profiles::EmailsController < Profiles::ApplicationController
-
   before_action :find_email, only: [:destroy, :resend_confirmation_instructions]
 
   def index

app/services/emails/create_service.rb

 module Emails
   class CreateService < ::Emails::BaseService
     def execute(options = {})
-      @user.emails.create({email: @email}.merge(options))
+      @user.emails.create({ email: @email }.merge(options))
     end
   end
 end

spec/controllers/profiles/emails_controller_spec.rb

@@ -8,7 +8,7 @@ describe Profiles::EmailsController do
   end
 
   describe '#create' do
-    let(:email_params) { {email: "add_email@example.com"} }
+    let(:email_params) { { email: "add_email@example.com" } }
 
     it 'sends an email confirmation' do
       expect {post(:create, { email: email_params })}.to change { ActionMailer::Base.deliveries.size }
@@ -18,7 +18,7 @@ describe Profiles::EmailsController do
   end
 
   describe '#resend_confirmation_instructions' do
-    let(:email_params) { {email: "add_email@example.com"} }
+    let(:email_params) { { email: "add_email@example.com" } }
 
     it 'resends an email confirmation' do
       email = user.emails.create(email: 'add_email@example.com')

spec/models/gpg_key_spec.rb

@@ -90,7 +90,7 @@ describe GpgKey do
     it 'email is verified if the user has the matching email' do
       user = create :user, email: 'bette.cartwright@example.com'
       gpg_key = create :gpg_key, key: GpgHelpers::User2.public_key, user: user
-      email_unconfirmed = create :email, user: user
+      create :email, user: user
       user.reload
 
       expect(gpg_key.emails_with_verified_status).to eq(
@@ -98,7 +98,7 @@ describe GpgKey do
         'bette.cartwright@example.net' => false
       )
 
-      email_confirmed = create :email, :confirmed, user: user, email: 'bette.cartwright@example.net'
+      create :email, :confirmed, user: user, email: 'bette.cartwright@example.net'
       user.reload
       expect(gpg_key.emails_with_verified_status).to eq(
         'bette.cartwright@example.com' => true,

spec/models/user_spec.rb

@@ -409,14 +409,12 @@ describe User do
       end
 
       it 'transfers old confirmation values into new secondary' do
-        org_user = @user
         @user.update_attributes!(email: @secondary.email)
         @user.reload
         expect(@user.emails.count).to eq 1
         expect(@user.emails.first.confirmed_at).not_to eq nil
       end
     end
-
   end
 
   describe '#update_tracked_fields!', :clean_gitlab_redis_shared_state do
@@ -1147,7 +1145,7 @@ describe User do
 
     it 'returns only confirmed emails' do
       email_confirmed = create :email, user: user, confirmed_at: Time.now
-      email_unconfirmed = create :email, user: user
+      create :email, user: user
       user.reload
       expect(user.verified_emails).to match_array([user.email, email_confirmed.email])
     end
@@ -1158,7 +1156,7 @@ describe User do
 
     it 'returns true when the email is verified/confirmed' do
       email_confirmed = create :email, user: user, confirmed_at: Time.now
-      email_unconfirmed = create :email, user: user
+      create :email, user: user
       user.reload
 
       expect(user.verified_email?(user.email)).to be_truthy