なるようになるブログ

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

rails commit log流し読み(2018/04/28)

2018/04/28分のコミットです。

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


Adding precision about which letter case to use for controller names (#32740)

rails guideのRails Routing from the Outside Inの修正です。

Connecting URLs to Codeの項、toオプションで指定するコントローラ名はスネークケースで記載する必要がある旨説明を追加しています。


Merge pull request #31956 from fatkodima/has_attached-presence-validation

activestorage/lib/active_storage/attached/one.rbの修正です。

Attached::Oneクラスにblank?メソッドを追加しています。実装はattachment.blank?の結果を返すようになっています。

attachmentsにvalidationを出来るようにする為との事です。

class User < ActiveRecord::Base
  has_one_attached :avatar
  has_many_attached :highlights

  validates :avatar, presence: true
  validates :highlights, presence: true
end

↑のようにpresenceチェックが出来るようになっています。


Merge pull request #32733 from Edouard-chin/ec-setupand-teardown

activesupport/lib/active_support/test_case.rbactivesupport/lib/active_support/testing/setup_and_teardown.rbの修正です。

ActiveSupport::Testing::SetupAndTeardown moduleをincludeActiveSupport::TestCaseに追加していたのを、prependを使うようにし、継承チェインの先頭にはいるよう修正しています。

先頭にいないと、after_teardownでexceptionが発生した場合のエラーのハンドリング(exceptionが発生しても後処理を継続する処理)が期待通りに動作しない為。


Merge pull request #32727 from utilum/assert_dont_expects

Action Cable、Active Recordのテストの修正です。

mock処理にmochaを使用していたのを、rails内部のヘルパーメソッド(assert_calledassert_called_with)を使用するよう修正しています


Merge pull request #30647 from droptheplot/render-partials-string-locals

actionview/lib/action_view/renderer/partial_renderer.rbの修正です。

partial rendererのlocalsオプションに渡すHashのkeyにStringを使用出来なかったのを、使用出来るよう修正しています。


[ci skip] Fix typo in testing.md

rails guideのTesting Rails Applicationsの修正です。

methodmetodにタイポしている箇所があったのを修正しています。