Merge pull request #144 from robotmay/score-adjustments

Scores decrement automatically after 2 days

app/models/photograph.rb

@@ -267,9 +267,9 @@ def set_highest_rank
   end
 
   def increment_score(by = 1)
-    Photograph.rankings.increment(id, by) do
-      set_highest_rank
-    end
+    Photograph.rankings.increment(id, by)
+    set_highest_rank
+    Worker.perform_in(2.days, 'Photograph', id, :decrement_score, by)
   end
 
   def decrement_score(by = 1)

app/workers/photograph_worker.rb

@@ -0,0 +1,8 @@
+class PhotographWorker
+  include Sidekiq::Worker
+
+  def perform(photograph_id, method, *args)
+    photo = Photograph.find(photograph_id)
+    photo.send(method, *args)
+  end
+end

app/workers/worker.rb

@@ -0,0 +1,8 @@
+class Worker
+  include Sidekiq::Worker
+
+  def perform(klass, id, method, *args)
+    model = klass.constantize.find(id)
+    model.send(method, *args)
+  end
+end

spec/models/favourite_spec.rb

@@ -7,6 +7,8 @@
   let(:user) { favourite.user }
   let(:counter) { double("counter", increment: true, decrement: true) }
 
+  before { Photograph.any_instance.stub(:set_highest_rank) { true } }
+
   it { should belong_to(:user) }
   it { should belong_to(:photograph) }
   it { should have_many(:notifications) }

spec/models/photograph_spec.rb

@@ -2,7 +2,7 @@
 
 describe Photograph do
   let(:user) { User.make! }
-  let(:photograph) { Photograph.make(user: user) }
+  let(:photograph) { Photograph.make!(user: user) }
   subject { photograph }
 
   describe "associations" do
@@ -104,6 +104,16 @@
     end
   end
 
+  describe "score" do
+    describe "#increment_score" do
+      it "enqueues a score decrement" do
+        expect {
+          photograph.increment_score(1)
+        }.to change(Worker.jobs, :size).by(1)
+      end
+    end
+  end
+
   describe "defaults" do
     let(:photograph) { Photograph.make!(user: user) }
     let(:user) { User.make! }

spec/support/sidekiq.rb

@@ -0,0 +1 @@
+require 'sidekiq/testing'

spec/workers/worker_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+
+describe Worker do
+  subject { Worker.new }
+  let(:photograph) { Photograph.make }
+  before { Photograph.stub(:find) { photograph } }
+  before { Photograph.stub(:increment_score) { true } }
+
+  it "runs any method" do
+    photograph.should_receive(:increment_score)
+    subject.perform('Photograph', 1, :increment_score, 1)
+  end
+end