From 4547e894e9f924221f1ec4ff8e71fa750bab3595 Mon Sep 17 00:00:00 2001 From: Kasper Timm Hansen Date: Sun, 22 Nov 2015 16:15:46 +0100 Subject: Bring back `===` stubbing in `time_zone_select` test. Erroneously removed in 58910dc7. The stubbing was a regression test to ensure `time_zone_select` wasn't implemented with `grep`. Rename the test and add a comment to make the intent clearer. --- actionview/test/template/form_options_helper_test.rb | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) (limited to 'actionview') diff --git a/actionview/test/template/form_options_helper_test.rb b/actionview/test/template/form_options_helper_test.rb index 06985b5a41..6b97cec34c 100644 --- a/actionview/test/template/form_options_helper_test.rb +++ b/actionview/test/template/form_options_helper_test.rb @@ -1203,9 +1203,15 @@ class FormOptionsHelperTest < ActionView::TestCase html end - def test_time_zone_select_with_priority_zones_as_regexp_using_grep_finds_no_zones + def test_time_zone_select_with_priority_zones_is_not_implemented_with_grep @firm = Firm.new("D") + # `time_zone_select` can't be written with `grep` because Active Support + # time zones don't support implicit string coercion with `to_str`. + @fake_timezones.each do |tz| + def tz.===(zone); raise Exception; end + end + html = time_zone_select("firm", "time_zone", /A|D/) assert_dom_equal "