なるようになるブログ

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

rails commit log流し読み(2018/07/10)

2018/07/10分のコミットです。

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


Add deprecations related to rails notes command to Changelog

railties/CHANGELOG.mdの修正です。

rake taskとして実装されていたrails notesrails commandとして実装した、Merge pull request #33220 from anniecodes/notes-commandの対応によりdeprecateになった機能(rake notesSOURCE_ANNOTATION_DIRECTORIES env等々)についてCHANGELOGにエントリーを追加しています。


Merge pull request #33309 from utilum/remove_unnecessary_mocha_stubs

Active Recordのテストの修正です。

不要なMochaのstub処理を削除しています。


Reduce mocking by testing value instead of method call

Active Recordのテストの修正です。

テストでstderr / stdoutをmockしている箇所があったのを、stderr / stdoutをStringIOで置き換えて、mockせずに値のチェックを行うよう修正しています。


Avoid unneeded expanded column aliases array caching

activerecord/lib/active_record/associations/join_dependency/join_part.rbactiverecord/lib/active_record/associations/join_dependency.rbの修正です。

JoinDependency::Aliases#initializeでcolumn cacheを作成する際に、column + aliasを保持していたのを、columnの値だけ保持するよう修正しています。aliasはcolumnから取得出来る為。


Replace shallow mocks with Ruby classes

Active Recordの修正です。

mockを使っていた箇所で、RubyClassで置き換えが可能な箇所を、Classで置き換えるよう修正しています。