bunch of rubocop fixes

Author: tansengming

app/admin/admin_users.rb

@@ -18,7 +18,7 @@
   controller do
     def update
       admin_user_params = params[:admin_user]
-      admin_user_params.delete_if {|param_name,_v| param_name.to_sym == :password } if admin_user_params.fetch(:password).blank?
+      admin_user_params.delete_if { |param_name,_v| param_name.to_sym == :password } if admin_user_params.fetch(:password).blank?
       update!
     end
   end

app/controllers/ups_controller.rb

@@ -5,4 +5,4 @@ def show
     ok = {status: :ok}
     respond_with ok
   end
-end
+end

app/services/event_tracker.rb

@@ -26,6 +26,7 @@ def identify
   end
 
   private
+
   def segment
     @segment ||= Segment::Analytics.new({
       write_key: ENV['SEGMENT_WRITE_KEY']

lib/ext/inet_input.rb

@@ -1,2 +1,2 @@
 class InetInput < Formtastic::Inputs::StringInput
-end
+end

lib/tasks/adhoc.rake

@@ -3,4 +3,4 @@
 # - make it idempotent. I.e. running it twice should create twice the trouble.
 
 namespace :adhoc do
-end
+end

lib/tasks/cron.rake

@@ -6,4 +6,4 @@ namespace :cron do
   task :daily
   task :weekly
   task :monthly
-end
+end

spec/controllers/payments_controller_spec.rb

@@ -8,7 +8,7 @@
     subject { get :new }
 
     context 'when there is a selected plan' do
-      subject { get :new, params: {plan: 'nice_tip'} }
+      subject { get :new, params: { plan: 'nice_tip' } }
       it 'should be success' do
         expect(subject).to be_successful
       end
@@ -24,7 +24,7 @@
       before { sign_out user }
 
       it 'should be success' do
-        expect(subject).to_not be_successful  
+        expect(subject).to_not be_successful
       end
     end
   end
@@ -35,8 +35,7 @@
     subject { post :create, params: params }
     before  { stripe_helper.create_plan(id: 'good_tip') }
     let(:stripe_helper)  { StripeMock.create_test_helper }
-    let(:default_params) { {stripeToken: stripe_helper.generate_card_token, plan: 'good_tip'} }
-
+    let(:default_params) { { stripeToken: stripe_helper.generate_card_token, plan: 'good_tip' } }
 
     context 'when there is a valid stripe token and plan' do
       let(:params) { default_params }
@@ -45,7 +44,7 @@
         expect { subject }.to change { RemoteKey.count }.by(1)
         expect(user.reload.stripe_customer).not_to be_nil
         expect(subject).to redirect_to page_path('thanks')
-        expect(user.stripe_customer.retrieve.subscriptions.data.map{|s| s.plan[:id]}).to eq ['good_tip']
+        expect(user.stripe_customer.retrieve.subscriptions.data.map { |s| s.plan[:id] }).to eq ['good_tip']
       end
     end
 

spec/controllers/users_controller_spec.rb

@@ -13,4 +13,4 @@
       end
     end
   end
-end
+end

spec/factories.rb

@@ -1,16 +1,16 @@
 FactoryBot.define do
   factory :admin_user do
     sequence(:email) { |n| "admin-#{n}@example.com" }
-    password { "foo12345" }
+    password { 'foo12345' }
   end
 
   factory :super_admin do
     sequence(:email) { |n| "super-admin-#{n}@example.com" }
-    password { "foo12345" }
+    password { 'foo12345' }
   end
 
   factory :user do
     sequence(:email) { |n| "user-#{n}@example.com" }
-    password { "foo12345" }
+    password { 'foo12345' }
   end
 end

spec/features/admin_spec.rb

@@ -7,7 +7,7 @@
   describe 'login' do
     before do
       sign_in admin
-      visit "/admin"
+      visit '/admin'
     end
     its(:current_path) { should == '/admin' }
 
@@ -19,8 +19,8 @@
       end
 
       it 'should not have an exception' do
-         expect { subject }.not_to raise_error
+        expect { subject }.not_to raise_error
       end
     end
   end
-end
+end