Refactor

Gemfile

@@ -1,4 +1,4 @@
-source :rubygems
+source 'https://rubygems.org'
 gemspec
 
-gem 'rspec'
+gem 'rspec'

Gemfile.lock

@@ -4,7 +4,7 @@ PATH
     simple-decorator (0.2.0)
 
 GEM
-  remote: http://rubygems.org/
+  remote: https://rubygems.org/
   specs:
     diff-lcs (1.1.3)
     rspec (2.10.0)

spec/simple_decorator_spec.rb

@@ -1,49 +1,53 @@
 require 'spec_helper'
 
 describe SimpleDecorator do
+  # User mock object
+  let(:user) { mock('user') }
+
+  # Interface
+  let(:decorator) { SimpleDecorator.new(user) }
+
+  # An other decorator
+  let(:other_decorator) { SimpleDecorator.new(decorator) }
+
   before do
-    # User mock object
-    @user = mock('user')
-    @user.stub('first_name') { 'Foo' }
-    @user.stub('last_name') { 'Bar' }
-    @user.stub('count') { 1 }
-
-    # Interface
-    @decorator = SimpleDecorator.new(@user)
-    def @decorator.first_name; 'Geronimo' end
-    def @decorator.count; @component.count + 1 end
-
-    # An other decorator
-    @other_decorator = SimpleDecorator.new(@decorator)
-    def @other_decorator.count; @component.count + 2 end
+    user.stub('first_name') { 'Foo' }
+    user.stub('last_name') { 'Bar' }
+    user.stub('count') { 1 }
+
+    def decorator.first_name; 'Geronimo' end
+    def decorator.count; @component.count + 1 end
+
+    def other_decorator.count; @component.count + 2 end
   end
+
   describe "is a true decorator" do
     describe 'Object behavior' do
       it 'should take an object as an argument for instanciation' do
-        SimpleDecorator.new(@user).should_not raise_error(ArgumentError)
+        SimpleDecorator.new(user).should_not raise_error(ArgumentError)
       end
 
       describe 'should be fully transparent' do
-        it { @decorator.should be_an_instance_of(@user.class) }
-        it { @decorator.should == @user }
+        it { decorator.should be_an_instance_of(user.class) }
+        it { decorator.should == user }
       end
     end # Object behavior
 
     describe 'Delegation behavior' do
       it 'should receive any known method an not pass them to the component' do
-        @decorator.first_name.should == 'Geronimo'
-        @decorator.first_name.should_not == 'Foo'
+        decorator.first_name.should == 'Geronimo'
+        decorator.first_name.should_not == 'Foo'
       end
 
       it 'should delegate unknown methods to the underlying component' do
-        @decorator.last_name.should == 'Bar'
+        decorator.last_name.should == 'Bar'
       end
 
       it 'should be able to decorate a decorator' do
-        @user.count.should      == 1
-        @decorator.count.should == 2
-        @other_decorator.count.should  == 4
+        user.count.should      == 1
+        decorator.count.should == 2
+        other_decorator.count.should  == 4
       end
     end # Delegation behavior
   end # is a true decorator
-end
+end