rails commit log流し読み(2017/08/10) - なるようになるブログ

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

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

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

FormBuilderクラスのdoc内の@personが+で囲まれていたのを、ttタグで囲むよう修正しています。

@マークが入っていると、+では正しく等幅フォントで表示されない為。

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

test_save_should_not_save_serialized_attribute_with_partial_writes_if_not_presentでデータをDBから取得する際、firstメソッドを使用していたのをfindメソッドでidを指定するよう修正しています。テストの実行順によってはfirstで期待どおりのレコードが取得出来ない為。

Revert “Set Ruby version in Gemfile and .ruby-version by default”

rails newで新規にアプリを作成する際、.ruby-versionファイルを作成、及び、GemfileにRubyバージョンを指定するよう修正した、Set Ruby version in Gemfile and .ruby-version by default をrevertしています。

Gemfile.lockに既にRubyバージョンは定義されており、本来それを使うべきだ
.ruby-versionにはJRubyやrubiniusを使用している場合のengineが指定出来ない
そもそもrailsを使っている人が全てrbenvを使用している訳ではない

等々の理由でrevertされたようです。議論はこちら。

Revert “Revert "Set Ruby version in Gemfile and .ruby-version by default”"

とまあ上記色々な理由があったのですが、それらを踏まえても今の段階では上記ファイル及び設定はがあった方が良いだろう、という事で上記revertはDHHによりrevertされました。入っている状態に戻っています。

Fix a typo

activestorage/app/models/active_storage/variant.rbのdocの修正です。

ActiveStorage::Variant#service_urlメソッドのdoc内でredirectをredirectにタイポしている箇所があったのを修正しています。

Merge pull request #30152 from bogdanvlviv/fix_random_ci_failure

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

assertionで値を比較する際に、値をidでsortするよう修正しています。

  def test_has_many_through_polymorphic_has_one
-    assert_equal Tagging.find(1, 2).sort_by(&:id), authors(:david).taggings_2
+    assert_equal Tagging.find(1, 2).sort_by(&:id), authors(:david).taggings_2.sort_by(&:id)
   end

sortを指定していと結果が不定になり、seedによってはテストが失敗してしまう為。