なるようになるブログ

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

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

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

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


Inherit all actioncable's test classes from ActionCable::TestCase

Action Cableのテストの修正です。

各テストクラスの親クラスにActiveSupport::TestCaseを指定している箇所テストがあったのを、ActionCable::TestCaseを使用するよう統一しています。


Include ActiveSupport::Testing::MethodCallAssertions to ActionCable::TestCase

Action Cableのテストの修正です。

テストクラス毎にActiveSupport::Testing::MethodCallAssertionsをincludeしていたのを、ActionCable::TestCaseでincludeするよう修正しています。


Remove extra include ActiveSupport::Testing::MethodCallAssertions

activesupport/test/testing/method_call_assertions_test.rbの修正です。

ActiveSupport::Testing::MethodCallAssertionsのincludeを削除しています。

テスト実行前にActiveSupport::TestCaseActiveSupport::Testing::MethodCallAssertionsをinclude済みの為。


Include ActiveSupport::Testing::MethodCallAssertions in railties/test/isolation/abstract_unit.rb

railtiesのテストの修正です。

ActiveSupport::TestCaseActiveSupport::Testing::MethodCallAssertionsをincludeするよう修正しています。


Merge pull request #27577 from maclover7/jm-fix-27547

actioncable/lib/action_cable/subscription_adapter/postgresql.rbの修正です。

Action CableでadapterにPostgreSQL adapterを使用している場合に、Active Recordで管理しているDBのconnectionをそのまま使いまわしていたのを、connectionを取得した際そのconnectionをAction Recordの管理から外し、Action Cable側で管理するよう修正しています。

Active Recordで管理しているconnectionをそのまま利用していると、Active Record側でconnectionを破棄した際に、Aciton Cable側のconnectionも予期せず破棄されてしまう為。


Merge pull request #33087 from bogdanvlviv/document-CompareWithRange

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

CompareWithRange moduleで定義されているメソッド(===include?cover?)がAPI docに表示されるよう:nodoc:を削除しています。これらのメソッドは既にguideに記載されており、publicメソッドな為。


Avoid Node.js v10.4.0 for now

.travis.ymlの修正です。

rails-ujsのテストで使用するNode.jsのバージョンを10.3.0に固定しています。10.4.0だと何故かエラーになってしまう為。


Merge pull request #33078 from bogdanvlviv/add-remove-require

Active Supportの修正です。

不要なrequireの削除、及び、不足していたrequireの追加を行っています。