2016/09/11分のコミットです。
CHANGELOGへの追加はありませんでした。
Use :git for specifying dependencies from github with https protocol
Gemfile
の修正です。
Gemfile
でgem先の指定にgithub
ショートハンドを使用していたのを、明示的にhttps
のURLを指定するよう修正しています。
-gem "sass", github: "sass/sass", branch: "stable", require: false +gem "sass", git: "https://github.com/sass/sass", branch: "stable", require: false
bundlerはgithub
ショートハンドを使用すると、https
プロトコルではなく、git
プロトコルを使用するようになっているのですが、bundler 1.13.0からgit
プロトコルを使用するとwarningメッセージが出るようになりました。
そのため、warningが表示されないようにする為に、https
プロトコルを使用するよう修正しています。
Do not mutate AC::TestRequest::DEFAULT_OPTIONS
actionpack/lib/action_controller/test_case.rb
の修正です。
ActionController::TestRequest#initialize
でsession_options
にTestSession::DEFAULT_OPTIONS
の値を設定する際に、dup
した値を渡すよう修正しています。
テスト内でsession_options
の値を更新した際に、TestRequest::DEFAULT_OPTIONS
の値も変わってしまうのを防ぐため。
Remove outdated "#TODO: Fix for binds." comment [ci skip]
activerecord/lib/active_record/relation.rb
のdocの修正です。
ActiveRecord::Relation#explain
メソッド内にあったFix for binds
というコメントを削除しています。既に対応済みなため、との事です。
use correct Gemfile in bin/setup
test
railties/test/application/bin_setup_test.rb
の修正です。
bin/setup
のテストで、テスト用RailsアプリにGemfileを追加し、そのGemfileをテスト内で使用するよう修正しています。
bin/setup
のテストでは、実際にbin/setup
コマンドを実行し、出力結果が想定通りか確認するようになっていたのですが、元々はGemfileが追加されず、rails自体のGemfileが使われるようになってしまっていました。
今まではそれで問題が無かったのですが、先に説明した通り、bundler 1.13.0からwarningが出るようになり、その影響でテストがコケるようになっていました。
そのため、rails自体のGemfileを使用するのではなく、テスト用にGemfileを追加し、そちらを使用するようにして対応しています。
Remove unused require "thread"
in test/cases/attribute_methods/read_test.rb
activerecord/test/cases/attribute_methods/read_test.rb
の修正です。
使用していないthread
のrequireを削除しています。
Remove duplicated elsif
branch
activerecord/test/cases/migration/column_attributes_test.rb
の修正です。
decimal型についてのテストから、不要な分岐処理を削除しています。