tests: move test/helpers to test/test_helpers.

The Rails convention is for test/helpers to be used for testing the view
helpers in app/helpers. We were using it to store certain utility
methods instead. Move these to test/test_helpers so that test/helpers
can be used for its intended purpose.

test/functional/iqdb_queries_controller_test.rb

@@ -1,9 +1,6 @@
 require 'test_helper'
-require 'helpers/iqdb_test_helper'
 
 class IqdbQueriesControllerTest < ActionController::TestCase
-  include IqdbTestHelper
-
   context "The iqdb controller" do
     setup do
       @user = FactoryGirl.create(:user)

test/functional/pool_elements_controller_test.rb

@@ -1,9 +1,6 @@
 require 'test_helper'
-require 'helpers/pool_archive_test_helper'
 
 class PoolElementsControllerTest < ActionController::TestCase
-  include PoolArchiveTestHelper
-
   context "The pools posts controller" do
     setup do
       mock_pool_archive_service!

test/functional/pool_versions_controller_test.rb

@@ -1,9 +1,6 @@
 require 'test_helper'
-require 'helpers/pool_archive_test_helper'
 
 class PoolVersionsControllerTest < ActionController::TestCase
-  include PoolArchiveTestHelper
-
   context "The pool versions controller" do
     setup do
       mock_pool_archive_service!

test/functional/pools_controller_test.rb

@@ -1,9 +1,6 @@
 require 'test_helper'
-require 'helpers/pool_archive_test_helper'
 
 class PoolsControllerTest < ActionController::TestCase
-  include PoolArchiveTestHelper
-
   context "The pools controller" do
     setup do
       Timecop.travel(1.month.ago) do

test/functional/post_versions_controller_test.rb

@@ -1,11 +1,9 @@
 require 'test_helper'
-require 'helpers/post_archive_test_helper'
 
 class PostVersionsControllerTest < ActionController::TestCase
-  include PostArchiveTestHelper
-
   def setup
     super
+
     @user = FactoryGirl.create(:user)
     CurrentUser.user = @user
     CurrentUser.ip_addr = "127.0.0.1"
@@ -13,6 +11,7 @@ class PostVersionsControllerTest < ActionController::TestCase
 
   def teardown
     super
+
     CurrentUser.user = nil
     CurrentUser.ip_addr = nil
   end

test/functional/reports_controller_test.rb

@@ -1,9 +1,6 @@
 require 'test_helper'
-require 'helpers/post_archive_test_helper'
 
 class ReportsControllerTest < ActionController::TestCase
-  include PostArchiveTestHelper
-
   def setup
     super
 

test/functional/saved_searches_controller_test.rb

@@ -1,9 +1,6 @@
 require 'test_helper'
-require 'helpers/saved_search_test_helper'
 
 class SavedSearchesControllerTest < ActionController::TestCase
-  include SavedSearchTestHelper
-
   context "The saved searches controller" do
     setup do
       @user = FactoryGirl.create(:user)

test/functional/uploads_controller_test.rb

@@ -1,9 +1,6 @@
 require 'test_helper'
-require 'helpers/iqdb_test_helper'
 
 class UploadsControllerTest < ActionController::TestCase
-  include IqdbTestHelper
-
   def setup
     super
     mock_iqdb_service!

test/test_helper.rb

@@ -14,7 +14,7 @@ require 'cache'
 require 'webmock/minitest'
 
 Dir[File.expand_path(File.dirname(__FILE__) + "/factories/*.rb")].each {|file| require file}
-Dir[File.expand_path(File.dirname(__FILE__) + "/helpers/*.rb")].each {|file| require file}
+Dir[File.expand_path(File.dirname(__FILE__) + "/test_helpers/*.rb")].each {|file| require file}
 
 Shoulda::Matchers.configure do |config|
   config.integrate do |with|
@@ -24,8 +24,12 @@ end
 
 class ActiveSupport::TestCase
   include PostArchiveTestHelper
+  include PoolArchiveTestHelper
   include ReportbooruHelper
   include DownloadTestHelper
+  include IqdbTestHelper
+  include SavedSearchTestHelper
+  include UploadTestHelper
 
   setup do
     mock_popular_search_service!
@@ -40,8 +44,6 @@ class ActiveSupport::TestCase
 end
 
 class ActionController::TestCase
-  include PostArchiveTestHelper
-
   def assert_authentication_passes(action, http_method, role, params, session)
     __send__(http_method, action, params, session.merge(:user_id => @users[role].id))
     assert_response :success

test/unit/moderator/tag_batch_change_test.rb

@@ -1,10 +1,7 @@
 require "test_helper"
-require 'helpers/saved_search_test_helper'
 
 module Moderator
   class TagBatchChangeTest < ActiveSupport::TestCase
-    include SavedSearchTestHelper
-
     def setup
       super
       mock_saved_search_service!

test/unit/pool_test.rb

@@ -1,11 +1,8 @@
 # encoding: utf-8
 
 require 'test_helper'
-require 'helpers/pool_archive_test_helper'
 
 class PoolTest < ActiveSupport::TestCase
-  include PoolArchiveTestHelper
-
   setup do
     Timecop.travel(1.month.ago) do
       @user = FactoryGirl.create(:user)

test/unit/post_replacement_test.rb

@@ -1,9 +1,6 @@
 require 'test_helper'
-require 'helpers/iqdb_test_helper'
 
 class PostReplacementTest < ActiveSupport::TestCase
-  include IqdbTestHelper
-
   def upload_file(path, filename, &block)
     Tempfile.open do |file|
       file.write(File.read(path))

test/unit/post_sets/pool_test.rb

@@ -1,10 +1,7 @@
 require 'test_helper'
-require 'helpers/pool_archive_test_helper'
 
 module PostSets
   class PoolTest < ActiveSupport::TestCase
-    include PoolArchiveTestHelper
-
     context "In all cases" do
       setup do
         @user = FactoryGirl.create(:user)

test/unit/post_test.rb

@@ -1,13 +1,6 @@
 require 'test_helper'
-require 'helpers/pool_archive_test_helper'
-require 'helpers/saved_search_test_helper'
-require 'helpers/iqdb_test_helper'
 
 class PostTest < ActiveSupport::TestCase
-  include PoolArchiveTestHelper
-  include SavedSearchTestHelper
-  include IqdbTestHelper
-
   def assert_tag_match(posts, query)
     assert_equal(posts.map(&:id), Post.tag_match(query).pluck(:id))
   end

test/unit/saved_search_test.rb

@@ -1,9 +1,6 @@
 require 'test_helper'
-require 'helpers/saved_search_test_helper'
 
 class SavedSearchTest < ActiveSupport::TestCase
-  include SavedSearchTestHelper
-
   def setup
     super
     @user = FactoryGirl.create(:user)

test/unit/tag_alias_test.rb

@@ -1,9 +1,6 @@
 require 'test_helper'
-require 'helpers/saved_search_test_helper'
 
 class TagAliasTest < ActiveSupport::TestCase
-  include SavedSearchTestHelper
-
   context "A tag alias" do
     setup do
       Timecop.travel(1.month.ago) do

test/unit/upload_test.rb

@@ -1,11 +1,6 @@
 require 'test_helper'
-require 'helpers/iqdb_test_helper'
-require 'helpers/upload_test_helper'
 
 class UploadTest < ActiveSupport::TestCase
-  include IqdbTestHelper
-  include UploadTestHelper
-
   def setup
     super