2015/08/26分のコミットです。
CHANGELOGにのったコミットは以下の通りです。
activejob/CHANGELOG.md
Merge pull request #21345 from aditya-kapoor/trim-ap
actionpack/test/abstract_unit.rb
、
actionpack/test/controller/resources_test.rb
の修正です。
resourcesのテストで、不要なテスト用クラスを削除しています。
actionpack/lib/action_dispatch/middleware/stack.rb
の修正です。
assert_index
メソッドのアクセス制御をprotected
からprivate
に修正しています。
protected
になっているものの、別にどのレシーバーからも呼び出されておらず、おそらく昔のprotected
の設定が残ったままなだけなのでは、という事でprivate
に変更したとの事です。
actionpack/lib/action_dispatch/middleware/cookies.rb
の修正です。
使用していないクラスのrequireを削除しています。
Removed mocha from Active Record Part 1
Active Recordのテストの修正です。
mock処理にmochaを使用していたのを、rails内部のヘルパーメソッドを使用するよう修正しています。Part 1なので、まだ続きます。
Remove more unused block arguments
actionpack/test/controller/resources_test.rb
、
actionpack/test/dispatch/live_response_test.rb
の修正です。
使用していないブロック変数を削除しています。
Merge pull request #21324 from Gaurav2728/gaurav-xml_serialization_doc_fix
activemodel/lib/active_model/serialization.rb
のdocの修正です。
シリアライズ処理について説明している箇所で、デフォルトでJSONとXMLのSerialization classがある旨説明されていたのですが、 Remove XML Serialization from core. · rails/rails@f7ebdb1でXMLのSerialization classはrails本体から削除されてしまったので、その旨説明を修正しています。
Initial implementation of ActiveJob AsyncAdapter.
Active Jobの修正です。
非同期にジョブ処理を行うAsyncJob
クラス、及びそのadapterを新規に追加しています。 active_job.queue_adapter
に:async
を指定する事で使用可能です。
処理的にはconcurrent-ruby
のthread poolを使ってjobのキューイングを行っています。
ちなみにこのadapterは、実際のキューイング処理で使う用、というよりは、テストの為に使う用、という事のようです。実際の本番環境ではもっとちゃんとしたバックエンドシステムを使用する事が推奨されています。
Merge pull request #21281 from ronakjangir47/added_docs
activesupport/lib/active_support/time_with_zone.rb
のdocの修正です。
TimeWithZone
クラスの各メソッドに、docを追加しています。
Merge pull request #21368 from rodzyn/remove_mocha
ActionPackのテストの修正です。
mock処理にminitestのmockクラスを使用していたのを、rails内部のヘルパーメソッドを使用するよう修正しています。Part 2。minitest/mock
のrequireが削除されたので、これで対応完了のようです。
always return a controller class from the controller_class
method
actionpack/lib/action_dispatch/http/request.rb
、
actionpack/lib/action_dispatch/routing/route_set.rb
の修正です。
Request#controller_class
メソッドが、必ずcontroller classを返すよう修正しています。
元々は、controllerが見つからなかった場合、呼び出し元のRouteSet#serve
メソッドで404を返すようにしていたのですが、
Request#controller_class
メソッドの方で、controllerが見つからなかった場合に404を返すエラー用のcontroller classを返すようにし、呼び出し元でエラー時の処理を意識しないで良いようにしています。
always dispatch to controllers the same way
ActionPackの修正です。
controllerのdispatch処理を行う場合、同じ処理(StaticDispatcher
クラス経由)で処理を行うよう処理を整理しています。
adding a direct dispatch method to controller classes
ActionPackの修正です。
controllerのdispatchを行う際、action経由ではなく、controllerから直接dispatch行えるよう、AcitonController::Metal
クラスに、dispatch
メソッドを追加しています。
move response allocation to the class level
actionpack/lib/action_controller/metal/live.rb
、
actionpack/lib/action_controller/metal/rack_delegation.rb
の修正です。
Live::Response
、ActionDispatch::Response
クラスで行っているresponse objectの生成処理を、各クラスのクラスメソッドに移動しています。
response objectの生成処理を行うのに上記クラスのインスタンスは不要な為、移動したとの事です。
rack SPEC doesn't allow first
on the body
actionpack/test/controller/new_base/bare_metal_test.rb
の修正です。
head
メソッドのテストで、bodyがcall
した結果の先頭にある事を想定したテストが書いてあったのですが、rakcの使用上、bodyが結果の先頭に格納される事は無いため、適切な箇所から値を取得するよう修正しています。
Fix another rack SPEC violation
actionpack/test/controller/new_base/middleware_test.rb
の修正です。
rackをcall
した際の、bodyの値のとり方がおかしい箇所があったのを修正しています。
actionpack/lib/action_controller/metal/head.rb
の修正です。
不要な空白を削除しています。
provide a request and response to all controllers
Action Packの修正です。
controllerが応答処理を行う際、必ずrequest、responseオブジェクトのデータを保持するよう修正しています。
元々は、条件によりrequest、responseオブジェクトを保持していたりしていなかったり、という状態で、値を見る際にresponseオブジェクトなのかrackのresponseなのかをチェックする必要があったのですが、そのチェックをしなくて済むよう、修正したようです。
Fixed to_datetime docs [ci skip]
activesupport/lib/active_support/time_with_zone.rb
のdocの修正です。
to_datetime
メソッドのdocに、UTC timezoneのDateTimeインスタンスを返す、と記載してあったのですが、
実際は使用しているtimezoneのUTCとのoffsetを返す、という挙動の為、適切にdocを修正しています。
Fix the unused variable warning
activerecord/test/cases/invertible_migration_test.rb
の修正です。
migrateをrevertするテストで、使用していない変数を削除しています。
Removed duplicate requiring minitest/mock as it is already required in method_call_assertions
各テストファイルで、不要なminitest/mock
のrequireを削除しています。