なるようになるブログ

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

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

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

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


Merge pull request #17184 from fillman/master

activesupport/test/autoload_test.rbの修正です。

$LOADED_FEATURESのチェックのテストが漏れていたので追加しています。


Fix grammar

guides/source/upgrading_ruby_on_rails.mdの修正です。

グラマーの修正です。


remove un-needed sort

actionview/lib/action_view/helpers/tag_helper.rb、及びActionViewのテストの修正です。

tag_optionsメソッド内でHTMLのsortを行う処理があったのですが、sort処理は要らないのでは、という事で削除しています。

合わせて、ActionViewのテストでassert_dom_equalメソッドを使用するよう修正しています。


fix tests on action_mailer

ActionMailerのテストの修正です。

テストでassert_equalを使用していたのをassert_dom_equalを使用するよう修正しています。


Only include the needed assertions

actionmailer/lib/action_mailer/test_case.rbactionmailer/test/base_test.rbの修正です。

上記ActionMailerのテストの修正でRails::Dom::Testing::Assertionsをincludeするよう修正していたのですが、 Rails::Dom::Testing::Assertionsだと、不要なクラスまでincludeしてしまうので、テストに必要なクラスのみincludeするよう修正しています。


update error message when InlineAdapter is used

activejob/lib/active_job/queue_adapters/inline_adapter.rbの修正です。

ActiveJobにInlineAdapterクラスを使用していた際のエラーメッセージを修正しています。

エラーメッセージの中に、ActiveJobの参考URLとしてhttps://github.com/rails/activejobが含まれていたのですが、左のURLは既に中身が無い状態なので、rails guideを参考URLに修正しています。


Warning about middleware not auto-reloading

rails guideのRails on Rackの修正です。

rackのMiddlewares は一回しか読み込まれず、変更のモニタリングもされないので、変更した場合はサーバの再起動を行う必要がある、旨説明を追加しています。


Point to a specific version of guide to make the redirect work

activejob/lib/active_job/queue_adapters/inline_adapter.rbの修正です。

先に修正したActiveJobのエラーメッセージに含まれるURLをhttp://guides.rubyonrails.org/v4.2.0/active_job_basics.htmlに修正しています。

バージョン情報が含まれていると、guidesにページが無い時に、自動でedgeguidesの方にリダイレクトしてくれるようです。知らなかった。