Skip to content

Commit

Permalink
Fixes execution of configuration default tests
Browse files Browse the repository at this point in the history
* defaults were never tested because of invalid include?
* corrected defaults to reflect current configuration object
  • Loading branch information
dn committed Mar 21, 2012
1 parent e98a6cc commit 13056b3
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions spec/copycopter_client/configuration_spec.rb
Expand Up @@ -5,7 +5,7 @@
match do |config|
config.should respond_to(option)

if instance_variables.include?('@default')
if instance_variables.include?(:'@default')
config.send(option).should == @default
end

Expand All @@ -32,8 +32,8 @@
it { should have_config_option(:environment_name).overridable.default(nil) }
it { should have_config_option(:client_version).overridable.default(CopycopterClient::VERSION) }
it { should have_config_option(:client_name).overridable.default('Copycopter Client') }
it { should have_config_option(:client_url).overridable.default('http://copycopter.com') }
it { should have_config_option(:secure).overridable.default(true) }
it { should have_config_option(:client_url).overridable.default('https://rubygems.org/gems/copycopter_client') }
it { should have_config_option(:secure).overridable.default(false) }
it { should have_config_option(:host).overridable.default('copycopter.com') }
it { should have_config_option(:http_open_timeout).overridable.default(2) }
it { should have_config_option(:http_read_timeout).overridable.default(5) }
Expand Down

0 comments on commit 13056b3

Please sign in to comment.