なるようになるブログ

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

rails commit log流し読み(2023/05/08)

2023/05/08分のコミットです。

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


Merge pull request #48153 from JoeDupuis/fix-arel-doc-code-block

activerecord/lib/arel.rbのdocの修正です。

Arelのdocのフォーマットを修正しています。


Test write_multi directly

activesupport/test/cache/behaviors/cache_store_behavior.rbの修正です。

write_multiメソッドを直接呼び出すテストを追加しています。


Move fetch_multi test to appropriate location

activesupport/test/cache/behaviors/cache_instrumentation_behavior.rbactivesupport/test/cache/behaviors/cache_store_behavior.rbの修正です。

fetch_multiメソッドに関するテストを適切なファイルに移動しています。


Use "library.name" format in initializer examples

activesupport/lib/active_support/deprecation.rbのdoc、及び、rails guideのError Reporting in Rails Applicationsの修正です。

initializerメソッドのexampleで、"library.name" というフォーマットを使用するよう統一しています。


Fix a race condition in Action Text's test suite

actiontext/test/test_helper.rbactiontext/test/unit/model_test.rbの修正です。

queryが実行されたかどうかをチェックするassertionで、Action Textで独自のassertionを定義していたのを、共通で使用されているhelperを使用するよう修正しています。


Merge pull request #48159 from jonathanhefner/cache-delete_multi-empty-key-list

activesupport/lib/active_support/cache.rbの修正です。

delete_multiメソッドで、引数のkey listが空の場合にエラーにならないよう修正しています。


Fix body stream test on Rack 3.0 by rewinding the body stream.

`actionpack/test/controller/test_case_test.rbの修正です。

request bodyをreadする前に、明示的にrewindをするよう修正しています。Rack 3.0から、body read時の自動でのrewind処理は保証されなくなったため。


Merge pull request #48163 from Shopify/view-lookup-alloc

Action Pack、Action Viewの修正です。

ActionView::LookupContextmime type validationについて、details hashのdupとfilter処理の前に不正なmime typeがあるかどうかをチェックするよう修正しています。多くのリクエストでは不正なmime typeが来ることは少なく、これにより不要なオブジェクトの生成を減らせられる筈の為。合わせて、symbolが有効な値かどうかをチェックする際の生成数も減らせるようリファクタリングを行っています。


Merge pull request #48157 from shivamsinghchahar/improve-aj-queue-adapter-test

activejob/test/cases/queue_adapter_test.rbの修正です。

queue adapterをsubclassで設定した場合のテストで、queue adapter名がnilかどうかだけをチェックしていたのを、subclassで設定した内容が影響無いことまでチェックするよう修正しています。


Test String#blank? with every non-dummy encoding

activesupport/test/core_ext/object/blank_test.rbの修正です。

Rubyにbundleされているnon-dummy以外のencodingのStringに対して、String#blank?が動作する事を確認するテストを追加しています。


Merge pull request #48150 from jonathanhefner/cache-summarize-logged-multi-keys

Acitve Supportの修正です。

cache operationのログの修正として、

  • read_multifetch_multi、及び、delete_multiに複数のkeyを指定した場合のログフォーマットの修正
  • write_multiのログ出力処理で引数を全て1つのcomposite keyにまとめて出力するよう修正
  • MemoryStore#cleanupのログのpayloadeにcleanupしたデータのsizeを含むよう修正

を行っています。


Merge pull request #48148 from tindron/arbitrary-hash-arrays

actionpack/lib/action_controller/metal/strong_parameters.rbの修正です。

arbitrary Hashをpermitしている、かつ、parameterがnestしたArrayだった場合に、permit後にArrayがflatになった状態になっていたのを、nestを維持するよう修正しています。


Merge pull request #48127 from rubys/create-home

railties/lib/rails/generators/rails/app/templates/Dockerfile.ttの修正です。

Dockerfileでhome directoryを作成するよう修正しています。IRBhistory fileを保持出来るようにするため。