aboutsummaryrefslogtreecommitdiffstats
path: root/railties/test
diff options
context:
space:
mode:
authorclaudiob <claudiob@gmail.com>2014-12-24 23:45:45 +0100
committerclaudiob <claudiob@gmail.com>2015-01-02 15:31:33 -0800
commit93dd5028a0cd0363d9f4bfc97d9ce70f0f3e88c8 (patch)
tree2cbf2fcf7420a4c5213a3fa7e315158e99679c09 /railties/test
parent2ade177b37c079666efa7705d57519072b8d8215 (diff)
downloadrails-93dd5028a0cd0363d9f4bfc97d9ce70f0f3e88c8.tar.gz
rails-93dd5028a0cd0363d9f4bfc97d9ce70f0f3e88c8.tar.bz2
rails-93dd5028a0cd0363d9f4bfc97d9ce70f0f3e88c8.zip
Loosen test about order of initializers
This commit modifies the code (but not the purpose) of a test that checks that > initializers are executed after application configuration initializers Currently the test hard-codes the *exact* initializers that are expected to occur before a custom one. This can cause the test to fail even if the expectation still passes. This commit loosens the test by simply checking that, in the array of initializers, the custom initializers (called `dummy_initializer` in the example) is executed after the last occurrence of `load_config_initializers`.
Diffstat (limited to 'railties/test')
-rw-r--r--railties/test/railties/engine_test.rb15
1 files changed, 5 insertions, 10 deletions
diff --git a/railties/test/railties/engine_test.rb b/railties/test/railties/engine_test.rb
index 91cdc60bd1..4f8cf4a41c 100644
--- a/railties/test/railties/engine_test.rb
+++ b/railties/test/railties/engine_test.rb
@@ -498,17 +498,12 @@ YAML
boot_rails
initializers = Rails.application.initializers.tsort
- index = initializers.index { |i| i.name == "dummy_initializer" }
- selection = initializers[(index-3)..(index)].map(&:name).map(&:to_s)
+ dummy_index = initializers.index {|i| i.name == "dummy_initializer"}
+ config_index = initializers.rindex {|i| i.name == :load_config_initializers}
+ stack_index = initializers.index {|i| i.name == :build_middleware_stack}
- assert_equal %w(
- load_config_initializers
- load_config_initializers
- engines_blank_point
- dummy_initializer
- ), selection
-
- assert index < initializers.index { |i| i.name == :build_middleware_stack }
+ assert config_index < dummy_index
+ assert dummy_index < stack_index
end
class Upcaser