読者です 読者をやめる 読者になる 読者になる

なるようになるブログ

読書感想文かrailsについてかrubyについてか

rails commit log流し読み(2016/09/04)

2016/09/04分のコミットです。

CHANGELOGにのったコミットは以下の通りです。

actionpack/CHANGELOG.md


Update docs for change in transaction callback exceptions

rails guideのRuby on Rails 5.0 Release NotesActive Record Callbacksの修正です。

transaction callbacksでエラーがおきた際にエラーを握りつぶさないよう修正した、Change transaction callbacks to not swallowing errors. · rails/rails@07d3d40の対応についてRelease Notesにリンクを追加、及び対応内容に合わせてActive Record Callbacks guideのTransaction Callbacksの項の説明を修正しています。


Remove unnecessary count method for collection proxy

activerecord/lib/active_record/associations/collection_association.rbactiverecord/lib/active_record/associations/collection_proxy.rbの修正です。

CollectionAssociation#countメソッドを削除しています。

元々はCollectionProxyからCollectionAssociationのメソッドを呼ぶようになっていたのですが、今はCollectionProxyRelationを継承しており、直接メソッドを参照出来る為、不要になったCollectionAssociationからメソッドを削除したとの事です。


make fixture_file_upload work in integration tests

actionpack/lib/action_dispatch/testing/integration.rbの修正です。

ActionDispatch::Integration::SessionクラスでTestProcess moduleをincludeしていたのを、ActionDispatch::IntegrationTestクラスでincludeするよう修正しています。

TestProcessfixture_file_upload method等のhelper methodが定義されているmoduleなのですが、fixture_file_uploadではfixture_pathを取得するのに、self.classfixture_pathを取得するようになっています。

が、Sessionクラスからではfixture_pathは取得出来ないので、fixture_pathを正しく取得出来るようにする為に、ActionDispatch::IntegrationTestにincludeするよう修正しています。


Merge pull request #26377 from y-yagi/rename_test_method_to_avoid_overriding

actionpack/test/dispatch/routing_test.rbの修正です。

テストメソッド名が被ってしまい、元のテストが動作しなくなってしまっていたので、テスト名が被らないよう修正しています。