From ed7c051fcc1bb183ca4c4138d4a328c2a5da2ace Mon Sep 17 00:00:00 2001
From: Sam <sam.saffron@gmail.com>
Date: Thu, 27 Feb 2014 12:15:41 +1100
Subject: [PATCH] BUGFIX: no need for mock central here

---
 spec/jobs/periodical_updates_spec.rb | 26 ++++----------------------
 1 file changed, 4 insertions(+), 22 deletions(-)

diff --git a/spec/jobs/periodical_updates_spec.rb b/spec/jobs/periodical_updates_spec.rb
index f15f7498e6..d91a8ac779 100644
--- a/spec/jobs/periodical_updates_spec.rb
+++ b/spec/jobs/periodical_updates_spec.rb
@@ -3,30 +3,12 @@ require_dependency 'jobs/scheduled/periodical_updates'
 
 describe Jobs::PeriodicalUpdates do
 
-  after do
+
+  it "works" do
+
+    # does not blow up, no mocks, everything is called
     Jobs::PeriodicalUpdates.new.execute(nil)
   end
 
-  it "calculates avg post time" do
-    Post.expects(:calculate_avg_time).once
-  end
-
-  it "calculates avg topic time" do
-    Topic.expects(:calculate_avg_time).once
-  end
-
-  it "features topics" do
-    CategoryFeaturedTopic.expects(:feature_topics).once
-  end
-
-  it "updates view counts" do
-    UserStat.expects(:update_view_counts).once
-  end
-
-  it "calculates scores" do
-    calculator = mock()
-    ScoreCalculator.expects(:new).once.returns(calculator)
-    calculator.expects(:calculate)
-  end
 
 end