なるようになるブログ

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

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

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

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


Fix broken RDoc formatting

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

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

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


Fix random CI failure DirtyTest:

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ファイルを作成、及び、GemfileRubyバージョンを指定するよう修正した、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内でredirectredirectにタイポしている箇所があったのを修正しています。


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によってはテストが失敗してしまう為。