なるようになるブログ

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

rails commit log流し読み(2017/12/06)

2017/12/06分のコミットです。

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


only install ffmpeg and mupdf on activestorage builds

.travis.ymlの修正です。

Active Storageのテストのbuildでのみffmpegmupdfをインストールするよう修正しています。


Execute JsonAttributeTest only if supports_json? returns true

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

supports_jsonがtrueを返す時だけJSON attributeのテストを行うよう修正しています。

rails本体でサポートしているadapterではすべてJSON data typeをサポート済みなのですが、third partyのadapter(Oracle enhanced adapter)ではJSON data typeをサポートしていない為。

が、これは間違いで、JsonAttributeTestJSON attributeに関するテストで、これはJSON data typeをサポート済みかどうかとは別な為、後ほどrevertされています。


Add assert_in_epsilon to Testing guide [ci skip]

rails guideのA Guide to Testing Rails Applicationsの修正です。

Available Assertionsの項にあるassetionsの一覧にassert_in_epsilonassert_not_in_epsilonを追加しています。


Yield array from AC::Parameters#each for block with one arg

actionpack/lib/action_controller/metal/strong_parameters.rbの修正です。

ActionController::Parameters#each#each_pairメソッドにblockを渡した場合に、block実行時の引数をArrayにするよう修正しています。

    def each_pair(&block)
       @parameters.each_pair do |key, value|
-        yield key, convert_hashes_to_parameters(key, value)
+        yield [key, convert_hashes_to_parameters(key, value)]
       end
     end

Hash#each#each_pairと挙動を合わせる為。