なるようになるブログ

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

rails commit log流し読み(2017/01/20)

2017/01/20分のコミットです。

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

activerecord/CHANGELOG.md


Deprecate initialize_schema_migrations_table and initialize_internal_metadata_table

Active Recordの修正です。

ActiveRecord::ConnectionAdapters::SchemaStatements moduleからinitialize_schema_migrations_tableinitialize_internal_metadata_tableメソッドをdeprecateにしています。

それぞれActiveRecord::SchemaMigration.create_tableActiveRecord::InternalMetadata.create_tableを呼び出しているだけで、メソッド経由ではなく直接呼びだせば良いだろう、という事でdeprecateになっています。Revert "Merge pull request #27718 from kamipo/remove_internal_public_methods"の続き。


Merge pull request #27733 from kenta-s/add-missing-argument-to-find_templates

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

ActionView::Resolver#find_templatesメソッドの引数が足りてなかった(引数は5つ渡されるのに、4つしか受け取れるようになってなかった)のを修正しています。


Merge pull request #27734 from y-yagi/update_create_table_statements_in_sqlite3

railties/test/application/rake/dbs_test.rbの修正です。

sqlite3コマンドでdumpされたSQLを確認するテストで、CREATE TABLEIF NOT EXISTSが含まれていてもテストが通るよう修正しています。

SQLite 3.16.0からCREATE TABLE文にIF NOT EXISTSが含まれるようになった為。

参考:SQLite: Check-in [c7021960]


Mention ActiveRecord's config.example.yml

rails guideのContributing to Ruby on Railsの修正です。

Testing Active Recordの項に、activerecord/test/config.example.ymlに書いてあるテーブル名やユーザを準備する必要がある旨説明を追加しています。


Add missing tests for ActionView::Template::Text

actionview/test/template/text_test.rbの修正です。

ActionView::Template::Textクラスのメソッドでテストが無かったメソッドについてテストを追加しています。


Merge pull request #27624 from elfassy/assert_enqueued_jobs_with_queue_level

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

assert_enqueued_jobsメソッドの引数にqueueを指定出来るよう修正しています。


add missing comment out [ci skip]

activesupport/lib/active_support/core_ext/enumerable.rbのdocの修正です。

Enumerable module配下の各メソッドのdoc内のメソッド実行例に#が不足していたのを追加しています。


add missing comment out [ci skip]

activesupport/lib/active_support/core_ext/array/access.rbのdocの修正です。

Array#withoutメソッドのdoc内のメソッド実行例に#が不足していたのを追加しています。