Skip to content
Browse files

silence warning: possibly useless use of a constant in void context

  • Loading branch information...
1 parent 6ed4ad1 commit 180e2e5e9e9aac7123f94eda61144241566b445d @kennyj committed
Showing with 7 additions and 3 deletions.
  1. +7 −3 activesupport/test/dependencies_test.rb
View
10 activesupport/test/dependencies_test.rb
@@ -923,8 +923,10 @@ def test_remove_constant_does_not_trigger_loading_autoloads
def test_remove_constant_does_not_autoload_already_removed_parents_as_a_side_effect
with_autoloading_fixtures do
- ::A
- ::A::B
+ silence_warnings do
+ ::A
+ ::A::B
+ end
ActiveSupport::Dependencies.remove_constant('A')
ActiveSupport::Dependencies.remove_constant('A::B')
assert !defined?(A)
@@ -934,7 +936,9 @@ def test_remove_constant_does_not_autoload_already_removed_parents_as_a_side_eff
def test_load_once_constants_should_not_be_unloaded
with_autoloading_fixtures do
ActiveSupport::Dependencies.autoload_once_paths = ActiveSupport::Dependencies.autoload_paths
- ::A.to_s
+ silence_warnings do
+ ::A
+ end
assert defined?(A)
ActiveSupport::Dependencies.clear
assert defined?(A)

0 comments on commit 180e2e5

Please sign in to comment.
Something went wrong with that request. Please try again.