memoize helpers

this does not instantiate new instance of HelpersWrapper every time "#helpers" is accessed
drapergem · Oct 12, 2012 · e2a757d · e2a757d
1 parent 02add97
commit e2a757d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/lib/draper/decorator.rb b/lib/draper/decorator.rb
@@ -196,7 +196,7 @@ def ==(other)
     #
     # @return [Object] proxy
     def helpers
-      HelpersWrapper.new self.class.helpers
+      @helpers ||= HelpersWrapper.new self.class.helpers
     end
     alias :h :helpers
 

diff --git a/spec/draper/decorator_spec.rb b/spec/draper/decorator_spec.rb
@@ -28,6 +28,13 @@
     it "is aliased to .h" do
       subject.h.should == subject.helpers
     end
+
+    it "initializes the wrapper only once" do
+      helper_proxy = subject.helpers
+      helper_proxy.stub(:test_method) { "test_method" }
+      subject.helpers.test_method.should eq("test_method")
+      subject.helpers.test_method.should eq("test_method")
+    end
   end
 
   context("#helpers") do