IntegrationTestでsave_and_open_pageメソッドを使用出来るよう対応しています。IntegrationTestの場合、レスポンスをファイルに保存し、それをブラウザで開く(オープン処理はcopiousfreetime/launchy gemを使用)よう対応されています。

Merge pull request #51036 from zzak/railties/asto-integration

railties/test/application/active_storage/uploads_integration_test.rbの修正です。

Active Storageのfile upload処理のintegrationテストを追加しています。

Don't silently execute statements on migrations when they can't be reversed

activerecord/lib/active_record/migration/command_recorder.rbの修正です。

migrationファイル内でupdateメソッドを使用している場合に、そのmigrationをrevertした際に、updateメソッドが暗黙的に実行されてしまうバグがあったのを修正しています。

が、修正内容に問題があったため、後ほどRevertされています。

Remove unused variable assignment in test

activerecord/test/cases/migration/command_recorder_test.rbの修正です。

使用していない変数を削除しています。

Action View Tests: Use #with_routing helper

actionview/test/abstract_unit.rb、 actionview/test/actionpack/controller/view_paths_test.rbの修正です。

Action Viewのテストで、Make with_routing test helper work for integration testsで追加されたwith_routing helperを使うよう修正しています。

Revert "Don't silently execute statements on migrations when they can't be reversed"

そのmigrationをrevertした際に、migration内のupdateメソッドが暗黙的に実行されてしまうバグがあったのを修正した、Don't silently execute statements on migrations when they can't be re…をRevertしています。修正というより既存の挙動を変更してしまう対応だったため。

Strict loading using :n_plus_one_only does not eagerly load child associations.

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

strict_loadingに:n_plus_one_onlyオプションを指定していた場合に、指定したmodelの子associationをfirstやlastでloadした場合にActiveRecord::StrictLoadingViolationErrorがraiseされてしまうバグがあったのを修正しています。

Fix some more ignored block warnings

blockを使用していないメソッドにblockを渡していた箇所があったのを、blockを渡さないよう修正しています。

Arel: only wrap SELECT statements in UNION if they involve ORDER BY, LIMIT or OFFSET

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

ArelでUNIQO用のSQLを生成する際にSELECTを括弧で括るようFixes/union select parenthesesで修正したのですが、、括らないよう戻しています。SQLiteでエラーになるSQLが生成されるようになってしまった為。

Fix indentation on save_and_open_page tests

actionpack/test/controller/integration_test.rbの修正です

save_and_open_pageメソッドのテストのインデントを修正しています。

Fix typo in page_dump_helper.rb

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

defaultをdefaulにタイポしていたのを修正しています。

2024-04-19

rails commit log流し読み(2024/04/18)

2024/04/18分のコミットです。

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

actionpack/CHANGELOG.md

Fix a regression in 7.1.3 passing a to: option without a controller when the controller is already defined by a scope.

activerecord/CHANGELOG.md

Allow Sqlite3Adapter to use sqlite3 gem version 2.x

Merge pull request #51523 from etiennebarrie/route-to-implicit-controller

actionpack/lib/action_dispatch/routing/mapper.rbの修正です。

Rails 7.1.3で、routeでcontroller scope無いでtoオプションをcontrollerの指定無しで定義した場合にエラーになるリグレッションが発生していたのを修正しています。

Rails.application.routes.draw do
  controller :home do
    get "recent", to: "recent_posts"
  end
end

Merge pull request #51445 from shouichi/remove-node-modules-from-the-final-image

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

Dockerの最終imageに不要にnode_modulesを含まないよう修正しています。

Merge pull request #51440 from CodingItWrong/libmysql2-fix-install-error

Gemfile.lockの修正です。

mysql2 gemを最新バージョンに更新しています。mysql-client 8.3で動作するようにするため。

Pin sqlite3 version in bug_report_templates

bug report templatesの修正です。

sqlite3 gemのバージョンを1系にロックしています。最新の2系だとbundlerでのロード時にエラーになってしまうため。

Correct typo for ActiveJob::Enqueuing::ClassMethods doc

activejob/lib/active_job/enqueuing.rbのdocの修正です。

ActiveJob::Enqueuing::ClassMethods moduleのdoc内のタイポを修正しています。

Allow sqlite3 to float to version 2

activerecord/lib/active_record/connection_adapters/sqlite3_adapter.rb、 railties/lib/rails/generators/database.rbの修正です。

sqlite3 gemの2系を使用出来るよう、コード内のgemのバージョン指定を修正しています。

Fix unused block warnings for template methods

actionview/lib/action_view/template.rbの修正です。

blockを使用していないメソッドにblockを渡さないよう修正しています。

Remove outdated mathn related test

mathn gemがロードされている場合の為のテストを削除しています。mathn gemはRuby本体から削除済み、かつ、gemとしてもdeprecatedになっており、考慮する必要がなくなっている為。

Action View Caching code sample syntax [ci skip]

actionview/lib/action_view/helpers/cache_helper.rbのdocの修正です。

ActionView::CacheHelper.cacheメソッドのdoc内のexampleコードで内のコメント部分の先頭に#を追加し、コメントである事がわかるよう修正しています。

2024-04-18

rails commit log流し読み(2024/04/17)

2024/04/17分のコミットです。

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

Merge pull request #51572 from Bodacious/patch-1

railties/lib/rails/generators/rails/app/templates/config/initializers/enable_yjit.rb.ttの修正です。

RubyVM::YJITがenableかどうかのチェックを括弧で括るよう修正しています。チェックに&&等の条件を独自に追加した場合にチェックが動作するようにするため。

test: fix flaky schema dump test to prefer YAML to Marshal

activerecord/test/cases/connection_adapters/schema_cache_test.rbの修正です。

schema dumpのテストで使用するファイルのフォーマットをMarshalからYAMLに変更しています。Marshalの場合、ファイルの中身が完全に毎回同じ内容になる事を保証しておらず、時折テストが失敗してしまうため。

Merge pull request #51476 from marvinthepa/no_anonymous_block_parameter

activerecord/lib/active_record/connection_adapters/abstract/query_cache.rbの修正です。

古いRubyでno anonymous block parameterでエラーになるコードがあったのを修正しています。

ruby-head: handle unused block warnings

activesupport/lib/active_support/testing/strict_warnings.rb、 activesupport/test/time_travel_test.rbの修正です。

Ruby 3.4でblockを使用していないメソッドにblockを渡した場合にwarningが出るようになるのですが、そのwarningについては発生してもテストをfailしないよう修正しています。false positiveが少しあるようで、まだ微調整中と思われる為。

参考：Feature #15554: warn/error passing a block to a method which never use a block

TestCase#create_fixtures no longer takes a block

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

ActiveRecord::FixtureSet.create_fixturesメソッドにうけつけていないblockを渡している箇所があったのを修正しています。

2024-04-17

rails commit log流し読み(2024/04/16)

2024/04/16分のコミットです。

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

Run devcontainer smoke tests in one single job

.github/workflows/devcontainer-smoke-test.ymlの修正です。

devcontainerのsmoke testを一つのjobでまとめて実行するよう修正しています。結果の確認をしやすくする為。

Remove rubocop and lint workflows

GitHub Actionsからrubocopとlint実行用のworkflowを削除しています。Buildkiteで実行するよう変更した為。

Add node and yarn to devcontainer when creating a project with Javascript

railties/lib/rails/generators/devcontainer.rb、 railties/lib/rails/generators/rails/app/templates/.devcontainer/devcontainer.json.ttの修正です。

Javascriptを使用するapplicationを作成した際に、Node.jsとyarnをdevcontainerに追加するよ修正しています。

2024-04-16

rails commit log流し読み(2024/04/15)

2024/04/15分のコミットです。

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

Fix remove_prefix_and_suffix in ActiveRecord::SchemaDumper

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

schemaファイル生成時の処理で、table_name_prefixとtable_name_suffixの削除処理が正しく動作しなくなってしまっていたのを修正しています。

Fix devcontainer smoke test for forked repos

.github/workflows/devcontainer-smoke-test.ymlの修正です。

forkしたリポジトリでdevcontainerのsmokeテストが実行されないよう修正しています。テストにdevcontainerのimageのpush処理が含まれており、forkリポジトリだとその処理がエラーになる為。

2024-04-15

rails commit log流し読み(2024/04/14)

2024/04/14分のコミットです。

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

activerecord/CHANGELOG.md

Allow ActiveRecord::Base#pluck to accept hash values

Fix typo in Feature Policy for idle-detection

actionpack/lib/action_dispatch/http/permissions_policy.rbの修正です。

Feature Policyのidle-detection directive名をタイポしていたのを修正しています。

Allow ActiveRecord::Base#pluck to accept hash values

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

ActiveRecord::Base#pluckでtableとcolumnの組み合わせをHashで指定出来るよう修正しています。

# Before
Post.joins(:comments).pluck("posts.id", "comments.id", "comments.body")

# After
Post.joins(:comments).pluck(posts: [:id], comments: [:id, :body])