Fix style

app/controllers/api/v1/users_controller.rb

@@ -15,7 +15,7 @@ class Api::V1::UsersController < Api::V1::BaseController
       avatar: @user.profile.avatar.url,
       admin: @user.admin?,
       steam: {
-        url: @steam.nil? ? nil : @steam.base_url, 
+        url: @steam.nil? ? nil : @steam.base_url,
         nickname: @steam.nil? ? nil : @steam.nickname
       },
       bans: {
@@ -31,9 +31,9 @@ class Api::V1::UsersController < Api::V1::BaseController
 
   private
 
-  def steam_profile user
+  def steam_profile(user)
     SteamCondenser::Community::SteamId.from_steam_id("STEAM_#{user.steamid}")
-  rescue SteamCondenser::Error => e
-    return nil
+  rescue SteamCondenser::Error
+    nil
   end
 end

spec/controllers/api/v1/users_controller_spec.rb

@@ -1,87 +1,87 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Api::V1::UsersController do
   before do
-    request.accept = 'application/json'
+    request.accept = "application/json"
   end
 
-  describe '#show' do
+  describe "#show" do
     before(:each) do
       @user = create :user, :chris
     end
 
-    it 'returns user data' do
+    it "returns user data" do
       get :show, id: @user.id
 
       expect(response).to be_success
-      expect(json['id']).to eq(@user.id)
-      expect(json['username']).to eq(@user.username)
-      expect(json['country']).to eq(@user.country)
-      expect(json['time_zone']).to eq(@user.time_zone)
-      expect(json['admin']).to eq(@user.admin?)
+      expect(json["id"]).to eq(@user.id)
+      expect(json["username"]).to eq(@user.username)
+      expect(json["country"]).to eq(@user.country)
+      expect(json["time_zone"]).to eq(@user.time_zone)
+      expect(json["admin"]).to eq(@user.admin?)
       expect(json).to have_key("steam")
-      expect(json['steam']).to have_key("url")
-      expect(json['steam']).to have_key("nickname")
-      expect(json['bans']['mute']).to eq(false)
-      expect(json['bans']['gather']).to eq(false)
-      expect(json['bans']['site']).to eq(false)
-      expect(json['team']).to be_nil
+      expect(json["steam"]).to have_key("url")
+      expect(json["steam"]).to have_key("nickname")
+      expect(json["bans"]["mute"]).to eq(false)
+      expect(json["bans"]["gather"]).to eq(false)
+      expect(json["bans"]["site"]).to eq(false)
+      expect(json["team"]).to be_nil
     end
 
-    it 'returns data for users with invalid steam ids' do
-      @user.steamid = '0:0:000'
+    it "returns data for users with invalid steam ids" do
+      @user.steamid = "0:0:000"
       @user.save!
 
       get :show, id: @user.id
 
       expect(response).to be_success
-      expect(json['steam']['url']).to be_nil
-      expect(json['steam']['nickname']).to be_nil
+      expect(json["steam"]["url"]).to be_nil
+      expect(json["steam"]["nickname"]).to be_nil
     end
 
-    it 'returns 404 if user does not exist' do
+    it "returns 404 if user does not exist" do
       expect {
         get :show, id: -1
       }.to raise_error(ActionController::RoutingError)
     end
 
-    it 'returns correct ban if user muted' do
+    it "returns correct ban if user muted" do
       create :ban, :mute, user: @user
       get :show, id: @user.id
       expect(response).to be_success
-      expect(json['bans']['mute']).to eq(true)
+      expect(json["bans"]["mute"]).to eq(true)
     end
 
-    it 'returns correct ban if user gather banned' do
+    it "returns correct ban if user gather banned" do
       create :ban, :gather, user: @user
       get :show, id: @user.id
       expect(response).to be_success
-      expect(json['bans']['gather']).to eq(true)
+      expect(json["bans"]["gather"]).to eq(true)
     end
 
-    it 'returns correct ban if user site banned' do
+    it "returns correct ban if user site banned" do
       create :ban, :site, user: @user
       get :show, id: @user.id
       expect(response).to be_success
-      expect(json['bans']['site']).to eq(true)
+      expect(json["bans"]["site"]).to eq(true)
     end
 
-    it 'returns team information' do
+    it "returns team information" do
       @user.destroy
       @user_with_team = create :user_with_team, :chris
       get :show, id: @user_with_team.id
       expect(response).to be_success
-      expect(json['team']['id']).to eq(@user_with_team.team.id)
-      expect(json['team']['name']).to eq(@user_with_team.team.name)
+      expect(json["team"]["id"]).to eq(@user_with_team.team.id)
+      expect(json["team"]["name"]).to eq(@user_with_team.team.name)
     end
   end
 
-  describe '#index' do
+  describe "#index" do
     before do
       5.times { create(:user_with_team) }
     end
 
-    it 'returns all users and associated teams' do
+    it "returns all users and associated teams" do
       users = User.all
 
       get :index
@@ -90,7 +90,7 @@ describe Api::V1::UsersController do
       expect(json["users"].size).to eq(users.size)
     end
 
-    it 'returns the excpected JSON keys' do
+    it "returns the excpected JSON keys" do
       get :index
       user_json = json["users"].first
       nested_team_json = user_json["team"]

spec/services/api/v1/users_collection_spec.rb

@@ -1,29 +1,29 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Api::V1::UsersCollection do
   let(:collection) { Api::V1::UsersCollection.new }
 
-  describe '#execute_query' do
-    describe 'when there are users with no teams' do
+  describe "#execute_query" do
+    describe "when there are users with no teams" do
       before do
         3.times { create(:user) }
       end
 
-      it 'returns 0 results' do
+      it "returns 0 results" do
         expect(collection.execute_query.size).to eq(0)
       end
     end
 
-    describe 'when there are some users with teams' do
+    describe "when there are some users with teams" do
       before do
         3.times { create(:user_with_team) }
       end
 
-      it 'returns 3 results' do
+      it "returns 3 results" do
         expect(collection.execute_query.size).to eq(3)
       end
 
-      it 'returns 6 columns' do
+      it "returns 6 columns" do
         expect(collection.execute_query.first.size).to eq(6)
       end
     end