2010-09-17 17:57:33 +02:00
|
|
|
require 'test_helper'
|
|
|
|
require 'application_checker'
|
|
|
|
require 'fileutils'
|
|
|
|
|
|
|
|
tmp_dir = "#{File.dirname(__FILE__)}/fixtures/tmp"
|
|
|
|
|
|
|
|
if defined? BASE_ROOT
|
|
|
|
BASE_ROOT.replace tmp_dir
|
|
|
|
else
|
|
|
|
BASE_ROOT = tmp_dir
|
|
|
|
end
|
|
|
|
FileUtils.mkdir_p BASE_ROOT
|
|
|
|
|
|
|
|
# Stub out methods on upgrader class
|
|
|
|
module Rails
|
|
|
|
module Upgrading
|
|
|
|
class ApplicationChecker
|
|
|
|
attr_reader :alerts
|
|
|
|
|
|
|
|
def base_path
|
|
|
|
BASE_ROOT + "/"
|
|
|
|
end
|
|
|
|
|
|
|
|
def in_rails_app?
|
|
|
|
true
|
|
|
|
end
|
|
|
|
|
|
|
|
def initialize
|
|
|
|
@alerts = {}
|
|
|
|
end
|
|
|
|
|
|
|
|
def alert(title, text, more_info_url, culprits)
|
|
|
|
@alerts[title] = [text, more_info_url, culprits]
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
class ApplicationCheckerTest < ActiveSupport::TestCase
|
|
|
|
def setup
|
|
|
|
@checker = Rails::Upgrading::ApplicationChecker.new
|
|
|
|
@old_dir = Dir.pwd
|
|
|
|
|
|
|
|
Dir.chdir(BASE_ROOT)
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_ar_methods_in_controller
|
|
|
|
make_file("app/controllers", "post_controller.rb", "Post.find(:all)")
|
|
|
|
@checker.check_ar_methods
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Soon-to-be-deprecated ActiveRecord calls")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_ar_methods_in_models
|
|
|
|
make_file("app/models", "post.rb", "Post.find(:all)")
|
|
|
|
@checker.check_ar_methods
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Soon-to-be-deprecated ActiveRecord calls")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_validation_on_methods
|
|
|
|
make_file("app/models", "post.rb", "validate_on_create :comments_valid?")
|
|
|
|
@checker.check_validation_on_methods
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Updated syntax for validate_on_* methods")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_before_validation_on_methods
|
|
|
|
make_file("app/models", "post.rb", "before_validation_on_create :comments_valid?")
|
|
|
|
@checker.check_before_validation_on_methods
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Updated syntax for before_validation_on_* methods")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_named_scope_left_over
|
|
|
|
make_file("app/models", "post.rb", "named_scope :failure")
|
|
|
|
@checker.check_ar_methods
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("named_scope is now just scope")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_routes
|
|
|
|
make_file("config/", "routes.rb", " map.connect 'fail'")
|
|
|
|
@checker.check_routes
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Old router API")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_for_old_test_help
|
|
|
|
make_file("test/", "test_helper.rb", " require 'test_help'")
|
|
|
|
@checker.check_test_help
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Deprecated test_help path")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_for_old_test_help_with_double_quotes
|
|
|
|
make_file("test/", "test_helper.rb", " require \"test_help\"")
|
|
|
|
@checker.check_test_help
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Deprecated test_help path")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_for_old_test_help_doesnt_see_test_helper
|
|
|
|
make_file("test/", "test_helper.rb", " require 'test_helper'")
|
|
|
|
@checker.check_test_help
|
|
|
|
|
|
|
|
assert !@checker.alerts.has_key?("Deprecated test_help path")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_lack_of_app_dot_rb
|
|
|
|
@checker.check_environment
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("New file needed: config/application.rb")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_environment_syntax
|
|
|
|
make_file("config/", "environment.rb", "config.frameworks = []")
|
|
|
|
@checker.check_environment
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Old environment.rb")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_gems
|
|
|
|
make_file("config/", "environment.rb", "config.gem 'rails'")
|
|
|
|
@checker.check_gems
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Old gem bundling (config.gems)")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_mailer_syntax
|
|
|
|
make_file("app/models/", "notifications.rb", "def signup\nrecipients @users\n end")
|
|
|
|
@checker.check_mailers
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Old ActionMailer class API")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_mailer_syntax_from
|
|
|
|
make_file("app/models/", "notifications.rb", "def signup\nfrom @user\n end")
|
|
|
|
@checker.check_mailers
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Old ActionMailer class API")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_mailer_syntax_subject
|
|
|
|
make_file("app/models/", "notifications.rb", "def signup\nsubject @subject\n end")
|
|
|
|
@checker.check_mailers
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Old ActionMailer class API")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_mailer_syntax_attachment
|
|
|
|
make_file("app/models/", "notifications.rb", "def signup\nattachment 'application/pdf' do |a|\n end")
|
|
|
|
@checker.check_mailers
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Old ActionMailer class API")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_new_check_mailer_syntax_from
|
|
|
|
make_file("app/models/", "notifications.rb", "def signup\n:from => @users\n end")
|
|
|
|
@checker.check_mailers
|
|
|
|
|
|
|
|
assert ! @checker.alerts.has_key?("Old ActionMailer class API")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_new_check_mailer_syntax_subject
|
|
|
|
make_file("app/models/", "notifications.rb", "def signup\n:subject => @users\n end")
|
|
|
|
@checker.check_mailers
|
|
|
|
|
|
|
|
assert ! @checker.alerts.has_key?("Old ActionMailer class API")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_new_check_mailer_syntax_attachments
|
|
|
|
make_file("app/models/", "notifications.rb", "def signup\nattachments['an-image.jp'] = File.read('an-image.jpg')\n end")
|
|
|
|
@checker.check_mailers
|
|
|
|
|
|
|
|
assert ! @checker.alerts.has_key?("Old ActionMailer class API")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_mailer_api
|
|
|
|
make_file("app/controllers/", "thing_controller.rb", "def signup\n Notifications.deliver_signup\n end")
|
|
|
|
@checker.check_mailers
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Deprecated ActionMailer API")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_generators
|
|
|
|
make_file("vendor/plugins/thing/generators/thing/", "thing_generator.rb", "def manifest\n m.whatever\n end")
|
|
|
|
@checker.check_generators
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Old Rails generator API")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_plugins
|
|
|
|
make_file("vendor/plugins/rspec-rails/", "whatever.rb", "def rspec; end")
|
|
|
|
@checker.check_plugins
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Known broken plugins")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_ignoring_comments
|
|
|
|
make_file("config/", "routes.rb", "# map.connect 'fail'")
|
|
|
|
@checker.check_routes
|
|
|
|
|
|
|
|
assert !@checker.alerts.has_key?("Old router API")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_deprecated_constants_in_app_code
|
|
|
|
make_file("app/controllers/", "thing_controller.rb", "class ThingController; THING = RAILS_ENV; end;")
|
|
|
|
@checker.check_deprecated_constants
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Deprecated constant(s)")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_deprecated_constants_in_lib
|
|
|
|
make_file("lib/", "extra_thing.rb", "class ExtraThing; THING = RAILS_ENV; end;")
|
|
|
|
@checker.check_deprecated_constants
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Deprecated constant(s)")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_deprecated_cookie_settings
|
|
|
|
make_file("config/initializers/", "more_settings.rb", "ActionController::Base.cookie_verifier_secret = 'OMG'")
|
|
|
|
@checker.check_old_cookie_secret
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Deprecated cookie secret setting")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_deprecated_session_secret
|
|
|
|
make_file("config/initializers/", "more_settings.rb", "ActionController::Base.session = {\n:whatever => 'woot'\n}")
|
|
|
|
@checker.check_old_session_secret
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Deprecated session secret setting")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_deprecated_session_settings
|
|
|
|
make_file("config/initializers/", "more_settings.rb", "ActionController::Base.session_store = :cookie\nthings.awesome(:whatever)")
|
|
|
|
@checker.check_old_session_setting
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Old session store setting")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_helpers
|
2010-09-17 18:52:59 +02:00
|
|
|
make_file("app/views/users/", "test.html.erb", "<b>blah blah blah</b><%= form_for(:thing) do |f| %> <label>doo dah</label> <%= f.whatever %> <% end %>")
|
2010-09-17 17:57:33 +02:00
|
|
|
@checker.check_old_helpers
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Deprecated ERb helper calls")
|
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_old_helpers_lets_regular_blocks_pass
|
|
|
|
make_file("app/views/users/", "another_test.html.erb", "<b>blah blah blah</b><% @some_items.each do |item| %> <label>doo dah</label> <%= item %> <% end %>")
|
|
|
|
@checker.check_old_helpers
|
|
|
|
|
|
|
|
assert_equal @checker.alerts.has_key?("Deprecated ERb helper calls"), false
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_old_helpers_lets_regular_blocks_pass
|
|
|
|
make_file("app/views/users/", "another_test.html.erb", "<b>blah blah blah</b><% @some_items.each do |item| %> <label>doo dah</label> <%= item %> <% end %>")
|
|
|
|
@checker.check_old_helpers
|
|
|
|
|
|
|
|
assert_equal @checker.alerts.has_key?("Deprecated ERb helper calls"), false
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_old_ajax_helpers
|
|
|
|
make_file("app/views/sections", "section.js", "<%= link_to_remote 'section-', :update => 'sections', :url => {:action => :destroy, :controller => 'sections', :id => @section.id } %>")
|
|
|
|
@checker.check_old_ajax_helpers
|
|
|
|
|
|
|
|
assert @checker.alerts.has_key?("Deprecated AJAX helper calls")
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_check_old_ajax_helpers_empty
|
|
|
|
@checker.check_old_ajax_helpers
|
|
|
|
|
|
|
|
assert ! @checker.alerts.has_key?("Deprecated AJAX helper calls")
|
|
|
|
end
|
|
|
|
|
|
|
|
def teardown
|
|
|
|
clear_files
|
|
|
|
|
|
|
|
Dir.chdir(@old_dir)
|
|
|
|
end
|
|
|
|
|
|
|
|
def make_file(where, name=nil, contents=nil)
|
|
|
|
FileUtils.mkdir_p "#{BASE_ROOT}/#{where}"
|
|
|
|
File.open("#{BASE_ROOT}/#{where}/#{name}", "w+") do |f|
|
|
|
|
f.write(contents)
|
|
|
|
end if name
|
|
|
|
end
|
|
|
|
|
|
|
|
def clear_files
|
|
|
|
FileUtils.rm_rf(Dir.glob("#{BASE_ROOT}/*"))
|
|
|
|
end
|
|
|
|
end
|