Bugs: Browse | Submit New | Admin

[#16979] 'rake spec' generates warning

Date:
2008-01-10 05:18		 Priority:
3
Submitted By:
Antonio Cangiano (acangiano) 		Assigned To:
Nobody (None)
Category:
None		 State:
Open
Summary:
'rake spec' generates warning

Detailed description 
Running 'rake spec' generates a series of warnings due to the constant ANVIL_ROOT being reassigned within a block of
before(:each):

(in /Users/Antonio/Code/anvil/anvilties)
....................../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT
../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT
../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT
../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT
../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT
../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT
../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT
../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT
../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT
../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT
../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT
../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT
../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT
../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT
../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT
../spec/initializer_spec.rb:80: warning: already initialized constant ANVIL_ROOT


This could be solved by using a variable rather than a constant or by moving the constant assignment outside of the
block. In the attached (super-tiny) patch, I took the latter approach.

Add A Comment: Notepad

Please login


Followup

Message 
Date: 2008-01-10 14:28
Sender: Lance Carlson

How about emulating how Merb handles constants?

http://pastie.caboo.se/137455

Attached Files:

Name Description Download
remove_warnings_from_specs.diff Patch that removes spec warnings Download

Changes:

Field Old Value Date By
File Added3093: remove_warnings_from_specs.diff2008-01-10 05:18acangiano