Commit ba2b6e9f

@@ -6,8 +6,14 @@ module My
 
     def create
       image = params[:photo][:image]
-      ProcessAvatarJob.perform_later(current_user, TemporaryStorage.new.store(image))
+      ProcessAvatarJob.perform_later(current_user, storage.store(image))
       redirect_to new_my_avatar_path, notice: t(:avatar_uploaded)
     end
+
+    private
+
+    def storage
+      TemporaryStorage.new
+    end
   end
 end

@@ -25,7 +25,7 @@ class Photo < ActiveRecord::Base
   end
 
   def version(key)
-    versions.find { |version| version.for?(key) }
+    versions.detect { |version| version.for?(key) }
   end
 
   private

@@ -3,8 +3,8 @@ class Tutorial < ActiveRecord::Base
   belongs_to :user
   acts_as_taggable
   default_scope -> { order('tutorials.created_at desc') }
-  scope :search, ->(query) do 
-    query.blank? ? all : where(["UPPER(heading) LIKE :query OR UPPER(description) LIKE :query", { query: "%#{query.upcase}%" }]) 
+  scope :search, ->(query) do
+    query.blank? ? all : where(["UPPER(heading) LIKE :query OR UPPER(description) LIKE :query", { query: "%#{query.upcase}%" }])
   end
 
   def to_param

@@ -7,7 +7,7 @@ class Version
     @photo = photo
   end
 
-  def adjust(image)
+  def adjust(_image)
     fail "Please override with version specific behaviours"
   end
 

@@ -23,7 +23,9 @@ describe ReProcessPhotoJob, type: :job do
 
       subject.perform(photo)
 
-      expect(ProcessPhotoJob).to have_received(:perform_later).with(photo, temp_file)
+      expect(ProcessPhotoJob).
+        to have_received(:perform_later).
+        with(photo, temp_file)
     end
   end
 end

@@ -6,7 +6,8 @@ describe PasswordReset do
     let(:mailer) { double(deliver_later: true) }
 
     before :each do
-      allow(PasswordResetMailer).to receive(:send_password_reset_instructions_to).
+      allow(PasswordResetMailer).
+        to receive(:send_password_reset_instructions_to).
         with(user).
         and_return(mailer)
     end