Fix styling

app/controllers/matches_controller.rb

@@ -17,8 +17,8 @@ class MatchesController < ApplicationController
   end
 
   def admin
-    @matches = Match.active.includes(:contest, :contester1, :contester2, :map1, :map2, :referee).
-               all.group_by { |t| t.week.to_s }.to_a.reverse
+    @matches = Match.active.includes(:contest, :contester1, :contester2, :map1, :map2, :referee)
+               .all.group_by { |t| t.week.to_s }.to_a.reverse
     render layout: "full"
   end
 

lib/verification.rb

@@ -56,7 +56,7 @@ module Verification
   # TODO: rikki?
   # Returns true if params (or its keys if a hash) are a subset of filter
   def self.contain(params, filter)
-    keys = (params.instance_of?(Array) ? params : params.keys).map { |k| k.to_sym }
+    keys = (params.instance_of?(Array) ? params : params.keys).map(&:to_sym)
     (keys - filter).empty?
   end
 end

spec/controllers/api/v1/maps_controller_spec.rb

@@ -1,19 +1,19 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Api::V1::MapsController do
   before do
-    request.accept = 'application/json'
+    request.accept = "application/json"
   end
 
-  describe '#index' do
+  describe "#index" do
     let!(:map) { create :map }
 
-    it 'returns a list of maps' do
+    it "returns a list of maps" do
       get :index
       expect(response).to be_success
-      expect(json['maps'].length).to eq(1)
-      json_map = json['maps'][0]
-      expect(json_map['id']).to eq(map.id)
+      expect(json["maps"].length).to eq(1)
+      json_map = json["maps"][0]
+      expect(json_map["id"]).to eq(map.id)
     end
   end
 end

spec/controllers/api/v1/servers_controller_spec.rb

@@ -1,20 +1,20 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Api::V1::ServersController do
   before do
-    request.accept = 'application/json'
+    request.accept = "application/json"
   end
 
-  describe '#index' do
+  describe "#index" do
     let!(:server) { create :server, :active }
     let!(:inactive_server) { create :server, :inactive }
 
-    it 'returns a list of servers' do
+    it "returns a list of servers" do
       get :index
       expect(response).to be_success
-      expect(json['servers'].length).to eq(1)
-      json_server = json['servers'][0]
-      expect(json_server['id']).to eq(server.id)
+      expect(json["servers"].length).to eq(1)
+      json_server = json["servers"][0]
+      expect(json_server["id"]).to eq(server.id)
     end
   end
 end

spec/controllers/plugin/plugin_controller_spec.rb

@@ -1,9 +1,9 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe PluginController do
   render_views
 
-  describe '#user' do
+  describe "#user" do
     before do
       create :group, :donors
       create :group, :champions
@@ -11,14 +11,14 @@ describe PluginController do
 
     let!(:user) { create :user_with_team }
 
-    it 'returns user data' do
+    it "returns user data" do
       get :user, id: user.steamid
       expect(response).to be_success
       expect(response.body).to include(user.username)
     end
 
-    it 'definitely does not return IP address' do
-      last_ip = '127.0.0.1'
+    it "definitely does not return IP address" do
+      last_ip = "127.0.0.1"
       user.lastip = last_ip
       user.save!
       get :user, id: user.steamid

spec/features/shoutbox/shoutbox_spec.rb

@@ -1,33 +1,33 @@
-require 'spec_helper'
+require "spec_helper"
 
-feature 'Shoutbox', js: true do
+feature "Shoutbox", js: true do
   let!(:user) { create :user }
 
   background do
     sign_in_as user
   end
 
-	scenario 'creating a valid shout' do
+  scenario "creating a valid shout" do
     visit root_path
-		shout = rand(100000).to_s
-		fill_in 'shoutbox_text', with: shout
-		click_button 'Shout!'
+    shout = rand(100_000).to_s
+    fill_in "shoutbox_text", with: shout
+    click_button "Shout!"
     expect(page).to have_content(shout)
   end
 
-	scenario 'enter more than 100 characters' do
+  scenario "enter more than 100 characters" do
     valid_shout = 100.times.map { "a" }.join
     invalid_shout = 101.times.map { "a" }.join
     visit root_path
     expect(page).to_not have_content("Maximum shout length exceeded")
-		fill_in 'shoutbox_text', with: invalid_shout
+    fill_in "shoutbox_text", with: invalid_shout
     expect(page).to have_content("Maximum shout length exceeded")
-		fill_in 'shoutbox_text', with: valid_shout
+    fill_in "shoutbox_text", with: valid_shout
     expect(page).to_not have_content("Maximum shout length exceeded")
   end
 
-	scenario 'creating shout while banned' do
-		Ban.create! ban_type: Ban::TYPE_MUTE, expiry: Time.now + 10.days, user_name: user.username
+  scenario "creating shout while banned" do
+    Ban.create! ban_type: Ban::TYPE_MUTE, expiry: Time.now.utc + 10.days, user_name: user.username
     visit root_path
     expect(find("#sidebar")).to have_content "You have been muted."
   end

spec/features/users/case_insensitive_login_spec.rb

@@ -1,6 +1,6 @@
-require 'spec_helper'
+require "spec_helper"
 
-feature 'Case insensitive login', js: :true do
+feature "Case insensitive login", js: :true do
   let(:username) { "CaSe_InSeNsItIvE" }
   let(:password) { "passwordABC123" }
   let!(:user) { create(:user, username: username, raw_password: password) }
@@ -9,23 +9,23 @@ feature 'Case insensitive login', js: :true do
     visit root_path
   end
 
-  feature 'when a user with mixed-case username signs in' do
-    scenario 'with a matching case allows the user to sign in' do
+  feature "when a user with mixed-case username signs in" do
+    scenario "with a matching case allows the user to sign in" do
       fill_login_form(username)
       click_button submit(:user, :login)
 
-      expect(page).to have_content(I18n.t('login_successful'))
+      expect(page).to have_content(I18n.t("login_successful"))
 
       within user_status do
         expect(page).to have_content(account_link)
       end
     end
 
-    scenario 'with a non-matching case allows the user to sign in' do
+    scenario "with a non-matching case allows the user to sign in" do
       fill_login_form("CASE_INSENSITIVE")
       click_button submit(:user, :login)
 
-      expect(page).to have_content(I18n.t('login_successful'))
+      expect(page).to have_content(I18n.t("login_successful"))
 
       within user_status do
         expect(page).to have_content(account_link)
@@ -39,6 +39,6 @@ feature 'Case insensitive login', js: :true do
   end
 
   def account_link
-    'ACCOUNT'
+    "ACCOUNT"
   end
 end

spec/support/features/form_helpers.rb

@@ -2,7 +2,7 @@ module Features
   module FormHelpers
     def fill_form(model, hash)
       hash.each do |attribute, value|
-        fill_in attribute_translation(model, attribute), :with => value
+        fill_in attribute_translation(model, attribute), with: value
       end
     end
 

spec/support/features/server_helpers.rb

@@ -1,24 +1,24 @@
 module Features
   module ServerHelpers
     def test_server_creation_and_editing
-      dns = 'ServerDns.com'
-      ip = '192.168.1.1'
-      port = '8000'
-      password = 'secret'
-      name = 'MyNsServer'
-      description = 'My NS Server'
-      irc = '#some_channel'
+      dns = "ServerDns.com"
+      ip = "192.168.1.1"
+      port = "8000"
+      password = "secret"
+      name = "MyNsServer"
+      description = "My NS Server"
+      irc = "#some_channel"
 
       visit new_server_path
-      fill_in 'Dns', with: dns
-      fill_in 'server_ip', with: ip
-      fill_in 'server_port', with: port
-      fill_in 'Password', with: password
-      fill_in 'Name', with: name
-      fill_in 'Description', with: description
-      fill_in 'Irc', with: irc
-      check 'Available for officials?'
-      click_button 'Save'
+      fill_in "Dns", with: dns
+      fill_in "server_ip", with: ip
+      fill_in "server_port", with: port
+      fill_in "Password", with: password
+      fill_in "Name", with: name
+      fill_in "Description", with: description
+      fill_in "Irc", with: irc
+      check "Available for officials?"
+      click_button "Save"
 
       expect(page).to have_content(dns)
       expect(page).to have_content("#{ip}:#{port}")
@@ -26,17 +26,17 @@ module Features
       expect(page).to have_content(irc)
       expect(page).to have_content(description)
 
-      click_link 'Edit Server'
+      click_link "Edit Server"
 
-      fill_in 'Dns', with: "#{dns}2"
-      fill_in 'server_ip', with: "192.168.1.2"
-      fill_in 'server_port', with: "8001"
-      fill_in 'Password', with: "#{password}2"
-      fill_in 'Name', with: "#{name}2"
-      fill_in 'Description', with: "#{description}2"
-      fill_in 'Irc', with: "#{irc}2"
-      check 'Available for officials?'
-      click_button 'Save'
+      fill_in "Dns", with: "#{dns}2"
+      fill_in "server_ip", with: "192.168.1.2"
+      fill_in "server_port", with: "8001"
+      fill_in "Password", with: "#{password}2"
+      fill_in "Name", with: "#{name}2"
+      fill_in "Description", with: "#{description}2"
+      fill_in "Irc", with: "#{irc}2"
+      check "Available for officials?"
+      click_button "Save"
 
       expect(page).to have_content("192.168.1.2:8001")
       expect(page).to have_content("#{dns}2")

spec/support/features/session_helpers.rb

@@ -6,16 +6,16 @@ module Features
       fill_in "login_username", with: user.username
       fill_in "login_password", with: user.raw_password
 
-      click_button I18n.t('helpers.submit.user.login')
+      click_button I18n.t("helpers.submit.user.login")
     end
 
     def change_timezone_for(user, timezone)
       visit edit_user_path(user.id)
 
-      click_link I18n.t('profile.locals')
+      click_link I18n.t("profile.locals")
       find("option[value='#{timezone}']").select_option
 
-      click_button I18n.t('helpers.submit.user.update')
+      click_button I18n.t("helpers.submit.user.update")
     end
 
     def user_status