From e90ca9cf48b5adee293a660167a66f5737cf0f60 Mon Sep 17 00:00:00 2001
From: farhatahmad <ahmad.af.farhat@gmail.com>
Date: Wed, 22 May 2019 13:51:55 -0400
Subject: [PATCH] Rspec and rubocop fixes

---
 app/controllers/admins_controller.rb         | 3 ++-
 spec/controllers/rooms_controller_spec.rb    | 8 ++++----
 spec/factories.rb                            | 2 +-
 test/mailers/previews/user_mailer_preview.rb | 2 +-
 4 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/app/controllers/admins_controller.rb b/app/controllers/admins_controller.rb
index c118e6c5..fcbe4af2 100644
--- a/app/controllers/admins_controller.rb
+++ b/app/controllers/admins_controller.rb
@@ -22,7 +22,8 @@ class AdminsController < ApplicationController
   include Emailer
 
   manage_users = [:edit_user, :promote, :demote, :ban_user, :unban_user, :approve]
-  site_settings = [:branding, :coloring, :coloring_lighten, :coloring_darken, :registration_method, :room_authentication]
+  site_settings = [:branding, :coloring, :coloring_lighten, :coloring_darken,
+                   :registration_method, :room_authentication]
 
   authorize_resource class: false
   before_action :find_user, only: manage_users
diff --git a/spec/controllers/rooms_controller_spec.rb b/spec/controllers/rooms_controller_spec.rb
index e9afc767..1f32e293 100644
--- a/spec/controllers/rooms_controller_spec.rb
+++ b/spec/controllers/rooms_controller_spec.rb
@@ -84,7 +84,7 @@ describe RoomsController, type: :controller do
     end
 
     it "sets the join name to cookie[:greenlight_name] if it exists" do
-      name = Faker::Pokemon.name
+      name = Faker::Games::Pokemon.name
       @request.cookies[:greenlight_name] = name
 
       get :show, params: { room_uid: @owner.main_room }
@@ -115,7 +115,7 @@ describe RoomsController, type: :controller do
 
     it "should create room with name and correct settings" do
       @request.session[:user_id] = @owner.id
-      name = Faker::Pokemon.name
+      name = Faker::Games::Pokemon.name
 
       room_params = { name: name, "client": "html5", "mute_on_join": "1" }
       json_room_settings = "{\"muteOnStart\":true,\"joinViaHtml5\":true}"
@@ -131,7 +131,7 @@ describe RoomsController, type: :controller do
 
     it "it should redirect to root if not logged in" do
       expect do
-        name = Faker::Pokemon.name
+        name = Faker::Games::Pokemon.name
         post :create, params: { room: { name: name } }
       end.to change { Room.count }.by(0)
 
@@ -288,7 +288,7 @@ describe RoomsController, type: :controller do
 
     it "properly updates room name through the room settings modal and redirects to current page" do
       @request.session[:user_id] = @user.id
-      name = Faker::Pokemon.name
+      name = Faker::Games::Pokemon.name
 
       room_params = { room_uid: @secondary_room.uid, room: { "name": name } }
 
diff --git a/spec/factories.rb b/spec/factories.rb
index 0b3aa58a..70515f6a 100644
--- a/spec/factories.rb
+++ b/spec/factories.rb
@@ -32,7 +32,7 @@ FactoryBot.define do
   end
 
   factory :room do
-    name { Faker::Pokemon.name }
+    name { Faker::Games::Pokemon.name }
     owner { create(:user) }
   end
 end
diff --git a/test/mailers/previews/user_mailer_preview.rb b/test/mailers/previews/user_mailer_preview.rb
index 2daf7eb6..7254af50 100644
--- a/test/mailers/previews/user_mailer_preview.rb
+++ b/test/mailers/previews/user_mailer_preview.rb
@@ -49,7 +49,7 @@ class UserMailerPreview < ActionMailer::Preview
     user = User.first
     UserMailer.invite_user_signup(user, "http://example.com/", @logo, @color, "test@example.com")
   end
-  
+
   # http://localhost:3000/rails/mailers/user_mailer/user_promoted
   def user_promoted
     user = User.first