なるようになるブログ

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

rails commit log流し読み(2015/11/23)

rails commit log流し読み(2015/11/23)

2015/11/23分のコミットです。

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


Don't cache fake time zones.

actionview/test/template/form_options_helper_test.rbの修正です。

テスト用の偽のtime zoneの値をキャッシュしないよう修正しています。


Ditch each_with_index for each.

actionview/test/template/form_options_helper_test.rbの修正です。

indexを使用していないのにeach_with_indexを使用している箇所があったのを、eachを使用するよう修正しています。


Bring back === stubbing in time_zone_select test.

actionview/test/template/form_options_helper_test.rbの修正です。

Merge pull request #21615 from ronakjangir47/actionViewpart2 · rails/rails@58910dcで謝って削除してしまった===メソッドのスタブを戻しています。

time_zone_selectメソッドの実装にgrepメソッドが使われてない事を確認するのに必要な為、との事です。 詳細は、Revert grep to select since they are not the same by bdmac · Pull Request #9680 · rails/rails参照。 ActiveSupport::TimeZoneにはto_str定義されてなかったんですねえ。


Simplify RailsGuides::Generator#select_only a bit

guides/rails_guides/generator.rbの修正です。

RailsGuides::Generator#select_onlyメソッドで、prefixチェック処理のリファクタリングを行っています。

-        prefixes.any? { |p| guide.start_with?(p) || guide.start_with?("kindle") }
+        guide.start_with?('kindle'.freeze, *prefixes)

[ci skip] Add Translating Rails Guides to contributing_to_ruby_on_rails

rails guideのContributing to Ruby on Railsの修正です。

Contributing to Ruby on RailsにRail Guideの翻訳方法について記載したTranslating Rails Guidesの項を追加しています。

Translating Rails Guidesは、docrailsのリポジトリwikiに書かれている内容らしいのですが、docrailsのリポジトリがprivateリポジトリになり見れなくなってしまった為、Contributing to Ruby on Rails guideに内容をうつしたとの事です。


Fix race condition testing for job execution order

Active Jobのテストの修正です。

jobの実行順の確認を行うテストで、実行順の確認にテスト用のファイルのctimeを見ていたのを、RubyTime.nowの値を使用するよう修正しています。

ctimeは、ファイルシステムによっては秒単位の粒度しかなく、環境によっては複数のjobが同じ秒数と時に終わってしまい、結果テストがエラーになってしまっていた為、修正したようです。