tests: fix tests for Rails.cache.

app/logical/cache.rb

@@ -37,6 +37,10 @@ class Cache
     nil
   end
 
+  def self.clear
+    Rails.cache.clear
+  end
+
   def self.sanitize(key)
     key.gsub(/\W/) {|x| "%#{x.ord}"}.slice(0, 230)
   end

test/functional/moderator/invitations_controller_test.rb

@@ -7,7 +7,6 @@ module Moderator
         @mod = FactoryGirl.create(:moderator_user)
         CurrentUser.user = @mod
         CurrentUser.ip_addr = "127.0.0.1"
-        MEMCACHE.flush_all
 
         @user_1 = FactoryGirl.create(:user)
         @user_2 = FactoryGirl.create(:user, :inviter_id => @mod.id)

test/functional/moderator/ip_addrs_controller_test.rb

@@ -8,7 +8,6 @@ module Moderator
         CurrentUser.user = @user
         CurrentUser.ip_addr = "127.0.0.1"
         FactoryGirl.create(:comment)
-        MEMCACHE.flush_all
       end
 
       should "find by ip addr" do

test/functional/posts_controller_test.rb

@@ -8,7 +8,6 @@ class PostsControllerTest < ActionController::TestCase
       CurrentUser.user = @user
       CurrentUser.ip_addr = "127.0.0.1"
       @post = FactoryGirl.create(:post, :uploader_id => @user.id, :tag_string => "aaaa")
-      MEMCACHE.flush_all
     end
 
     teardown do

test/functional/tag_alias_corrections_controller_test.rb

@@ -6,7 +6,6 @@ class TagAliasCorrectionsControllerTest < ActionController::TestCase
       @admin = FactoryGirl.create(:admin_user)
       CurrentUser.user = @admin
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
       @tag_alias = FactoryGirl.create(:tag_alias, :antecedent_name => "aaa", :consequent_name => "bbb")
     end
 

test/functional/tag_alias_requests_controller_test.rb

@@ -6,7 +6,6 @@ class TagAliasRequestsControllerTest < ActionController::TestCase
       @user = FactoryGirl.create(:user)
       CurrentUser.user = @user
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do

test/functional/tag_aliases_controller_test.rb

@@ -6,7 +6,6 @@ class TagAliasesControllerTest < ActionController::TestCase
       @user = FactoryGirl.create(:admin_user)
       CurrentUser.user = @user
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do

test/functional/tag_implication_requests_controller_test.rb

@@ -8,7 +8,6 @@ class TagImplicationRequestsControllerTest < ActionController::TestCase
       end
       CurrentUser.user = @user
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do

test/functional/tag_implications_controller_test.rb

@@ -6,7 +6,6 @@ class TagImplicationsControllerTest < ActionController::TestCase
       @user = FactoryGirl.create(:admin_user)
       CurrentUser.user = @user
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do

test/test_helper.rb

@@ -21,12 +21,12 @@ Shoulda::Matchers.configure do |config|
   end
 end
 
-if defined?(MEMCACHE)
-  Object.send(:remove_const, :MEMCACHE)
-end
-
 class ActiveSupport::TestCase
   include PostArchiveTestHelper
+
+  teardown do
+    Cache.clear
+  end
 end
 
 class ActionController::TestCase
@@ -41,9 +41,12 @@ class ActionController::TestCase
     __send__(http_method, action, params, session.merge(:user_id => @users[role].id))
     assert_redirected_to(new_sessions_path)
   end
+
+  teardown do
+    Cache.clear
+  end
 end
 
-MEMCACHE = MemcacheMock.new
 Delayed::Worker.delay_jobs = false
 
 require "helpers/reportbooru_helper"

test/unit/artist_test.rb

@@ -18,7 +18,6 @@ class ArtistTest < ActiveSupport::TestCase
       user = Timecop.travel(1.month.ago) {FactoryGirl.create(:user)}
       CurrentUser.user = user
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do

test/unit/artist_url_test.rb

@@ -3,7 +3,6 @@ require 'test_helper'
 class ArtistUrlTest < ActiveSupport::TestCase
   context "An artist url" do
     setup do
-      MEMCACHE.flush_all
       CurrentUser.user = FactoryGirl.create(:user)
       CurrentUser.ip_addr = "127.0.0.1"
     end

test/unit/comment_test.rb

@@ -6,7 +6,6 @@ class CommentTest < ActiveSupport::TestCase
       user = FactoryGirl.create(:user)
       CurrentUser.user = user
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do

test/unit/dmail_test.rb

@@ -3,7 +3,6 @@ require 'test_helper'
 class DmailTest < ActiveSupport::TestCase
   context "A dmail" do
     setup do
-      MEMCACHE.flush_all
       @user = FactoryGirl.create(:user)
       CurrentUser.user = @user
       CurrentUser.ip_addr = "1.2.3.4"

test/unit/favorite_test.rb

@@ -5,7 +5,6 @@ class FavoriteTest < ActiveSupport::TestCase
     user = FactoryGirl.create(:user)
     CurrentUser.user = user
     CurrentUser.ip_addr = "127.0.0.1"
-    MEMCACHE.flush_all
     Favorite # need to force loading the favorite model
   end
 

test/unit/ip_ban_test.rb

@@ -5,7 +5,6 @@ class IpBanTest < ActiveSupport::TestCase
     @user = FactoryGirl.create(:user)
     CurrentUser.user = @user
     CurrentUser.ip_addr = "127.0.0.1"
-    MEMCACHE.flush_all
     Danbooru.config.stubs(:member_comment_time_threshold).returns(1.week.from_now)
   end
 

test/unit/janitor_trial_test.rb

@@ -7,7 +7,6 @@ class JanitorTrialTest < ActiveSupport::TestCase
       @user = FactoryGirl.create(:user)
       CurrentUser.user = @admin
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do

test/unit/moderator/ip_addr_search_test.rb

@@ -9,7 +9,6 @@ module Moderator
         CurrentUser.ip_addr = "127.0.0.1"
         Danbooru.config.stubs(:member_comment_time_threshold).returns(1.week.from_now)
         FactoryGirl.create(:comment)
-        MEMCACHE.flush_all
       end
 
       teardown do

test/unit/note_test.rb

@@ -6,7 +6,6 @@ class NoteTest < ActiveSupport::TestCase
       @user = FactoryGirl.create(:user)
       CurrentUser.user = @user
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do

test/unit/pool_test.rb

@@ -13,7 +13,6 @@ class PoolTest < ActiveSupport::TestCase
     end
 
     CurrentUser.ip_addr = "127.0.0.1"
-    MEMCACHE.flush_all
 
     mock_pool_archive_service!
     PoolArchive.sqs_service.stubs(:merge?).returns(false)

test/unit/post_appeal_test.rb

@@ -6,7 +6,6 @@ class PostAppealTest < ActiveSupport::TestCase
       @alice = FactoryGirl.create(:user)
       CurrentUser.user = @alice
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
       Danbooru.config.stubs(:max_appeals_per_day).returns(5)
     end
 

test/unit/post_archive_test.rb

@@ -8,7 +8,6 @@ class PostArchiveTest < ActiveSupport::TestCase
       end
       CurrentUser.user = @user
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do

test/unit/post_disapproval_test.rb

@@ -6,7 +6,6 @@ class PostDisapprovalTest < ActiveSupport::TestCase
       @alice = FactoryGirl.create(:moderator_user)
       CurrentUser.user = @alice
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do

test/unit/post_flag_test.rb

@@ -8,7 +8,6 @@ class PostFlagTest < ActiveSupport::TestCase
       end
       CurrentUser.user = @alice
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
       @post = FactoryGirl.create(:post, :tag_string => "aaa")
     end
 

test/unit/post_pruner_test.rb

@@ -5,7 +5,6 @@ class PostPrunerTest < ActiveSupport::TestCase
     @user = FactoryGirl.create(:admin_user)
     CurrentUser.user = @user
     CurrentUser.ip_addr = "127.0.0.1"
-    MEMCACHE.flush_all
 
     Timecop.travel(2.weeks.ago) do
       @flagger = FactoryGirl.create(:gold_user)

test/unit/post_sets/favorite_test.rb

@@ -7,7 +7,6 @@ module PostSets
         @user = FactoryGirl.create(:user)
         CurrentUser.user = @user
         CurrentUser.ip_addr = "127.0.0.1"
-        MEMCACHE.flush_all
 
         @post_1 = FactoryGirl.create(:post)
         @post_2 = FactoryGirl.create(:post)

test/unit/post_sets/intro_test.rb

@@ -8,7 +8,6 @@ module PostSets
         @user = FactoryGirl.create(:user)
         CurrentUser.user = @user
         CurrentUser.ip_addr = "127.0.0.1"
-        MEMCACHE.flush_all
 
         @post_1 = FactoryGirl.create(:post, :tag_string => "a")
         @post_2 = FactoryGirl.create(:post, :tag_string => "b")

test/unit/post_sets/pool_test.rb

@@ -10,7 +10,6 @@ module PostSets
         @user = FactoryGirl.create(:user)
         CurrentUser.user = @user
         CurrentUser.ip_addr = "127.0.0.1"
-        MEMCACHE.flush_all
 
         mock_pool_archive_service!
         start_pool_archive_transaction

test/unit/post_sets/post_test.rb

@@ -8,7 +8,6 @@ module PostSets
         @user = FactoryGirl.create(:user)
         CurrentUser.user = @user
         CurrentUser.ip_addr = "127.0.0.1"
-        MEMCACHE.flush_all
 
         @post_1 = FactoryGirl.create(:post, :tag_string => "a")
         @post_2 = FactoryGirl.create(:post, :tag_string => "b")

test/unit/post_test.rb

@@ -18,7 +18,6 @@ class PostTest < ActiveSupport::TestCase
     end
     CurrentUser.user = @user
     CurrentUser.ip_addr = "127.0.0.1"
-    MEMCACHE.flush_all
     mock_saved_search_service!
   end
 

test/unit/post_version_test.rb

@@ -8,7 +8,6 @@ class PostVersionTest < ActiveSupport::TestCase
       end
       CurrentUser.user = @user
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do

test/unit/post_vote_test.rb

@@ -8,7 +8,6 @@ class PostVoteTest < ActiveSupport::TestCase
     @user = FactoryGirl.create(:user)
     CurrentUser.user = @user
     CurrentUser.ip_addr = "127.0.0.1"
-    MEMCACHE.flush_all
 
     @post = FactoryGirl.create(:post)
   end

test/unit/related_tag_calculator_test.rb

@@ -5,7 +5,6 @@ class RelatedTagCalculatorTest < ActiveSupport::TestCase
     user = FactoryGirl.create(:user)
     CurrentUser.user = user
     CurrentUser.ip_addr = "127.0.0.1"
-    MEMCACHE.flush_all
   end
 
   teardown do

test/unit/related_tag_query_test.rb

@@ -5,7 +5,6 @@ class RelatedTagQueryTest < ActiveSupport::TestCase
     user = FactoryGirl.create(:user)
     CurrentUser.user = user
     CurrentUser.ip_addr = "127.0.0.1"
-    MEMCACHE.flush_all
   end
 
   context "a related tag query without a category constraint" do

test/unit/saved_search_test.rb

@@ -41,10 +41,6 @@ class SavedSearchTest < ActiveSupport::TestCase
   end
 
   context "Fetching the post ids for a search" do
-    setup do
-      MEMCACHE.expects(:get).returns(nil)
-    end
-
     teardown do
       FakeWeb.clean_registry
     end

test/unit/tag_alias_correction_test.rb

@@ -6,13 +6,11 @@ class TagAliasCorrectionTest < ActiveSupport::TestCase
       @mod = FactoryGirl.create(:moderator_user)
       CurrentUser.user = @mod
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
       @post = FactoryGirl.create(:post, :tag_string => "aaa")
       @tag_alias = FactoryGirl.create(:tag_alias, :antecedent_name => "aaa", :consequent_name => "bbb")
     end
 
     teardown do
-      MEMCACHE.flush_all
       CurrentUser.user = nil
       CurrentUser.ip_addr = nil
     end

test/unit/tag_alias_request_test.rb

@@ -6,11 +6,9 @@ class TagAliasRequestTest < ActiveSupport::TestCase
       @user = FactoryGirl.create(:user)
       CurrentUser.user = @user
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do
-      MEMCACHE.flush_all
       CurrentUser.user = nil
       CurrentUser.ip_addr = nil
     end

test/unit/tag_alias_test.rb

@@ -11,7 +11,6 @@ class TagAliasTest < ActiveSupport::TestCase
         CurrentUser.user = user
       end
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
       mock_saved_search_service!
     end
 

test/unit/tag_implication_request_test.rb

@@ -6,11 +6,9 @@ class TagImplicationRequestTest < ActiveSupport::TestCase
       @user = FactoryGirl.create(:user)
       CurrentUser.user = @user
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do
-      MEMCACHE.flush_all
       CurrentUser.user = nil
       CurrentUser.ip_addr = nil
     end

test/unit/tag_implication_test.rb

@@ -7,7 +7,6 @@ class TagImplicationTest < ActiveSupport::TestCase
       CurrentUser.user = user
       CurrentUser.ip_addr = "127.0.0.1"
       @user = FactoryGirl.create(:user)
-      MEMCACHE.flush_all
     end
 
     teardown do

test/unit/tag_subscription_test.rb

@@ -5,7 +5,6 @@ class TagSubscriptionTest < ActiveSupport::TestCase
     user = FactoryGirl.create(:user)
     CurrentUser.user = user
     CurrentUser.ip_addr = "127.0.0.1"
-    MEMCACHE.flush_all
   end
 
   teardown do

test/unit/tag_test.rb

@@ -5,7 +5,6 @@ class TagTest < ActiveSupport::TestCase
     user = FactoryGirl.create(:builder_user)
     CurrentUser.user = user
     CurrentUser.ip_addr = "127.0.0.1"
-    MEMCACHE.flush_all
   end
 
   teardown do
@@ -32,10 +31,6 @@ class TagTest < ActiveSupport::TestCase
   end
 
   context "A tag category fetcher" do
-    setup do
-      MEMCACHE.flush_all
-    end
-
     should "fetch for a single tag" do
       FactoryGirl.create(:artist_tag, :name => "test")
       assert_equal(Tag.categories.artist, Tag.category_for("test"))
@@ -57,10 +52,6 @@ class TagTest < ActiveSupport::TestCase
   end
 
   context "A tag category mapping" do
-    setup do
-      MEMCACHE.flush_all
-    end
-
     should "exist" do
       assert_nothing_raised {Tag.categories}
     end
@@ -97,10 +88,6 @@ class TagTest < ActiveSupport::TestCase
   end
 
   context "A tag" do
-    setup do
-      MEMCACHE.flush_all
-    end
-
     should "be lockable by a moderator" do
       @tag = FactoryGirl.create(:tag)
       @tag.update_attributes({:is_locked => true}, :as => :moderator)
@@ -123,11 +110,11 @@ class TagTest < ActiveSupport::TestCase
     should "reset its category after updating" do
       tag = FactoryGirl.create(:artist_tag)
       tag.update_category_cache_for_all
-      assert_equal(Tag.categories.artist, MEMCACHE.get("tc:#{tag.name}"))
+      assert_equal(Tag.categories.artist, Cache.get("tc:#{tag.name}"))
 
       tag.update_attribute(:category, Tag.categories.copyright)
       tag.update_category_cache_for_all
-      assert_equal(Tag.categories.copyright, MEMCACHE.get("tc:#{tag.name}"))
+      assert_equal(Tag.categories.copyright, Cache.get("tc:#{tag.name}"))
     end
   end
 

test/unit/upload_test.rb

@@ -17,7 +17,6 @@ class UploadTest < ActiveSupport::TestCase
       user = FactoryGirl.create(:contributor_user)
       CurrentUser.user = user
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do

test/unit/user_feedback_test.rb

@@ -4,7 +4,6 @@ class UserFeedbackTest < ActiveSupport::TestCase
   context "A user's feedback" do
     setup do
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do

test/unit/user_test.rb

@@ -6,7 +6,6 @@ class UserTest < ActiveSupport::TestCase
       @user = FactoryGirl.create(:user)
       CurrentUser.user = @user
       CurrentUser.ip_addr = "127.0.0.1"
-      MEMCACHE.flush_all
     end
 
     teardown do

test/unit/wiki_page_test.rb

@@ -2,7 +2,6 @@ require 'test_helper'
 
 class WikiPageTest < ActiveSupport::TestCase
   setup do
-    MEMCACHE.flush_all
     CurrentUser.ip_addr = "127.0.0.1"
   end