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

なるようになるブログ

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

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

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

CHANGELOGへの追加はありませんでした。


Merge pull request #26370 from yahonda/rails26368

activerecord/test/cases/autosave_association_test.rbactiverecord/test/cases/relation/where_test.rbの修正です。

テスト用modelファイルの読み込み順を調整(models/postmodels/commentより先に読み込むよう対応)しています。

テストの実行順により時折failになってしまうテスト(BasicsTest#test_readonly_attributes)があり、テストを安定化させる為に対応したとの事です。

Postクラスの方が先に読み込まれてないと、associationが正しく認識されないからですかねえ。


Remove redundant !loaded? condition

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

ActiveRecord::Associations::CollectionAssociation#sizeメソッドでloaded?のチェックが複数回行われていたので、片方削除しています。


Remove not used alias method

activerecord/lib/active_record/relation/query_methods.rbの修正です。

使用していないleft_joins!メソッドを削除しています。


Fix Remaining Case-In-Assignment Statement Formatting

Rubocopのフォーマット修正対応で、Case-In-Assignmentのフォーマット対応が残っていたのをまとめて修正しています。