読者です 読者をやめる 読者になる 読者になる

なるようになるブログ

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

rails commit log流し読み(2016/10/08)

2016/10/08分のコミットです。

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


Tweak a test so the queries match

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

limitに関するテストで、テスト結果の確認にfirstを使用していたのを、to_aを使用するよう修正しています。

-    assert_equal Post.limit(1).to_a.first, all_posts.first
+    assert_equal Post.limit(1).to_a, all_posts.to_a

生成されるqueryを合わせるようにする為、との事です。


explain why autosave= disables inverse_of [ci skip]

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

autosave=メソッド内の@automatic_inverse_ofにfalseを設定している箇所に、何故autosaveinverse_ofが一緒に動作しないのかの説明を追加しています。


Undefine assings in IntegrationTest

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

IntegrationTestクラスでassignsをundefするよう修正しています。

assingsはcontrollerのtestでのみ使用すべきで値で、integration testで使用すべてきでは無い為、との事です。


Silence a warning

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

使用していない定数の参照によりRubyのwarningが出ている箇所があったので、silence_warningで該当部分を囲む、warningが表示されないよう修正しています。

-        ::LoadMe
+        silence_warning do
+          ::LoadMe
+        end

定数参照自体はテストの都合上必要な為、warningを抑止する方法で対応したようです。


Fixnum and Bignum are deprecated in Ruby trunk

コード内でFixnumBignum定数を使用していた箇所を、使用しないよう修正でしています。

Ruby trunkで、FixnumBignum定数を使用するとdeprecate warningを出すよう対応が入った為との事です。参照: Feature #12739: deprecate_constant :Fixnum, :Bignum - Ruby trunk - Ruby Issue Tracking System


Revert "Temporarily make ACa tests noiser and more predictable"

Action Cableのテストを実行する際、verbose、及びseedオプションを指定するようにした、Temporarily make ACa tests noiser and more predictableをrevertしています。テストが安定化したから、ですかねえ。