diff options
author | claudiob <claudiob@gmail.com> | 2014-12-24 23:45:45 +0100 |
---|---|---|
committer | claudiob <claudiob@gmail.com> | 2015-01-02 15:31:33 -0800 |
commit | 93dd5028a0cd0363d9f4bfc97d9ce70f0f3e88c8 (patch) | |
tree | 2cbf2fcf7420a4c5213a3fa7e315158e99679c09 /railties | |
parent | 2ade177b37c079666efa7705d57519072b8d8215 (diff) | |
download | rails-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')
-rw-r--r-- | railties/test/railties/engine_test.rb | 15 |
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 |