curlybars 1.3.0 → 1.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3c5f1fe84620e10957d38e663d3883158c00b6db25395c61e9628ddd3d19d110
-  data.tar.gz: 5eac82875ef4fca6a4c0aa728626615c643089ed6cc24de91232d793753f8f5b
+  metadata.gz: f83af0f1fc951b9a35dd68ae1b3bd181799031cf37913100574c65d98de5e870
+  data.tar.gz: 954af845f827c80b1fced94ad10ed5f1bcaf99dc872f3b54435ab9e23c3d82e3
 SHA512:
-  metadata.gz: 3697082fceb22d1d868d4de0fcf1054d8facc0e8dee7da0e58704bf34bb303a0cd287b460baa5d40ffcbf17ed7d0e3235643b7d855e57f821ed63223f1d737e3
-  data.tar.gz: dfba149b55705dbc9567a763dbc12b3514296a43ec576a683642f83f727d6d7ec4c49ffc514a4c2a9a4d1ec91740365dd144d2a87bde96d59b2632b8c24ed4df
+  metadata.gz: f75d2a62e3c4dec26b01bf04928d624e70a0b807cf99c40ac0b5b6a03e421e5f296f6a5eaed609bb7bd4679bbe402ebfd0e8626c6f2527a8118afd71aa937522
+  data.tar.gz: 748d32d09437bdf7bd02b5d607aee846cabd02ab015a837ec95ccc00e451099f21cf71f3b5a7cb305e8d4f8bfe18145b7401531db55cb8ba4fc0ad66906e259e

data/lib/curlybars/method_whitelist.rb

@@ -14,19 +14,21 @@ module Curlybars
       methods_with_type_validator.call(methods_with_type)
 
       define_method(:allowed_methods) do
-        methods_list = methods_without_type + methods_with_type.keys
+        @method_whitelist_allowed_methods ||= begin
+          methods_list = methods_without_type + methods_with_type.keys
 
-        # Adds methods to the list of allowed methods
-        method_adder = lambda do |*more_methods, **more_methods_with_type|
-          methods_with_type_validator.call(more_methods_with_type)
+          # Adds methods to the list of allowed methods
+          method_adder = lambda do |*more_methods, **more_methods_with_type|
+            methods_with_type_validator.call(more_methods_with_type)
 
-          methods_list += more_methods
-          methods_list += more_methods_with_type.keys
-        end
+            methods_list += more_methods
+            methods_list += more_methods_with_type.keys
+          end
 
-        contextual_block&.call(self, method_adder)
+          contextual_block&.call(self, method_adder)
 
-        defined?(super) ? super() + methods_list : methods_list
+          defined?(super) ? super() + methods_list : methods_list
+        end
       end
 
       define_singleton_method(:methods_schema) do |context = nil|

data/lib/curlybars/version.rb

@@ -1,3 +1,3 @@
 module Curlybars
-  VERSION = '1.3.0'.freeze
+  VERSION = '1.3.1'.freeze
 end

data/spec/curlybars/method_whitelist_spec.rb

@@ -16,6 +16,8 @@ describe Curlybars::MethodWhitelist do
 
   let(:validation_context_class) do
     Class.new do
+      attr_accessor :invocation_count
+
       def foo?
         true
       end
@@ -130,10 +132,15 @@ describe Curlybars::MethodWhitelist do
 
         Class.new do
           extend Curlybars::MethodWhitelist
+          attr_accessor :invocation_count
+
           allow_methods :cook, link: LinkPresenter do |context, allow_method|
             if context.foo?
               allow_method.call(:bar)
             end
+
+            context.invocation_count ||= 0
+            context.invocation_count += 1
           end
 
           def foo?
@@ -173,6 +180,14 @@ describe Curlybars::MethodWhitelist do
         expect(post_presenter.new.allowed_methods).to eq([:cook, :link, :bar, :form, :foo_bar, :wave])
       end
 
+      it "only invokes the context block once" do
+        presenter = post_presenter.new
+
+        10.times { presenter.allowed_methods }
+
+        expect(presenter.invocation_count).to eq(1)
+      end
+
       it "returns a dependency_tree with inheritance and composition with context" do
         expect(post_presenter.dependency_tree(validation_context_class.new)).
           to eq(

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: curlybars
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.3.1
 platform: ruby
 authors:
 - Libo Cannici
@@ -13,7 +13,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-12-17 00:00:00.000000000 Z
+date: 2020-01-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack