config_leaf 0.0.1 → 0.0.2

data/README.md

@@ -41,7 +41,7 @@ their object has been instantiated:
         @list = []
 
         # Allow the user access via ConfigLeaf syntax.
-        ConfigLeaf.wrap object, &block if block_given?
+        ConfigLeaf.wrap self, &block if block_given?
       end
 
       def reverse!
@@ -54,21 +54,21 @@ their object has been instantiated:
       end
     end
 
-    # User uses the ConfigLeaf block.
-    object = Cheese.new do
+    # User uses the ConfigLeaf block syntax.
+    object1 = Cheese.new do
       list [1, 2, 3]      # Calls object.list = [1, 2, 3]
       list << 4           # Calls object.list << 4
       value 5             # Calls object.value = 5
       value list.size     # Calls object.value = object.list.size
       reverse!            # Calls object.reverse!
-      
-      # These would be possible if using instance_eval, but not here.
-      #explode!           # Not allowed because it is protected (raises an exception).
-      #@value = 5         # Would set @value on the wrapper, not in the wrapped object, so has no effect.
+
+      ## @value = 99      # Would set @value on the wrapper, not in the wrapped object, so raises an exception.
+
+      ## explode!         # Not allowed because it is protected (raises an exception).
     end
 
     # User chooses to not use ConfigLeaf block syntax by requesting a block parameter.
-    object = Cheese.new do |c|
+    object2 = Cheese.new do |c|
       c.list = [1, 2, 3]
       c.list << 4
       c.value = 5
@@ -76,6 +76,9 @@ their object has been instantiated:
       c.reverse!
     end
 
+    p object1 #=> #<Cheese:0x4340658 @value=4, @list=[4, 3, 2, 1]>
+    p object2 #=> #<Cheese:0x45d2b58 @value=4, @list=[4, 3, 2, 1]>
+
 Contributing
 ------------
 

data/lib/config_leaf.rb

@@ -43,7 +43,7 @@ module ConfigLeaf
     #         @value = 0
     #         @list = []
     #
-    #         ConfigLeaf.wrap object, &block if block_given?
+    #         ConfigLeaf.wrap self, &block if block_given?
     #       end
     #
     #       def invert
@@ -52,7 +52,7 @@ module ConfigLeaf
     #     end
     #
     #     # User chooses to use the ConfigLeaf block syntax.
-    #     object = Cheese.new do
+    #     object1 = Cheese.new do
     #       list [1, 2, 3]      # Calls object.list = [1, 2, 3]
     #       list << 4           # Calls object.list << 4
     #       value 5             # Calls object.value = 5
@@ -61,7 +61,7 @@ module ConfigLeaf
     #     end
     #
     #    # User chooses to not use ConfigLeaf block syntax by requesting a block parameter.
-    #    object = Cheese.new do |c|
+    #    object2 = Cheese.new do |c|
     #       c.list = [1, 2, 3]
     #       c.list << 4
     #       c.value = 5

data/lib/config_leaf/version.rb

@@ -1,3 +1,3 @@
 module ConfigLeaf
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/lib/config_leaf/wrapper.rb

@@ -9,7 +9,7 @@ module ConfigLeaf
       alias_method :wrap, :new
     end
 
-    # If passed a block, the DSLWrapper will #instance_eval it automatically.
+    # If passed a block, the Wrapper will #instance_eval it automatically.
     #
     # @param owner [Object] Object to redirect the public methods of.
     def initialize(owner, &block)
@@ -37,6 +37,11 @@ module ConfigLeaf
       end
 
       instance_eval &block if block_given?
+
+      if instance_variables != [:@_owner]
+        altered = instance_variables - [:@_owner]
+        raise "Instance variable#{altered.one? ? '' : 's'} #{altered.join ", "} set in ConfigLeaf scope"
+      end
     end
 
     private

data/spec/config_leaf/wrapper_spec.rb

@@ -88,6 +88,16 @@ describe ConfigLeaf::Wrapper do
     end
   end
 
+  context "in block" do
+    it "should fail if an ivar were changed during the block, since they would be discarded!" do
+      lambda { described_class.wrap owner do @frog = 12 end }.should raise_error RuntimeError, "Instance variable @frog set in ConfigLeaf scope"
+    end
+
+    it "should fail if any ivars were changed during the block, since they would be discarded!" do
+      lambda { described_class.wrap owner do @frog = 12; @fish = 1 end }.should raise_error RuntimeError, "Instance variables @frog, @fish set in ConfigLeaf scope"
+    end
+  end
+
   describe "" do
     it "should instance_eval when given a block on the constructor (.new)" do
       mock(owner).frog(5)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: config_leaf
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-10-19 00:00:00.000000000 Z
+date: 2012-10-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -130,7 +130,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 196843955
+      hash: 847368189
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -139,10 +139,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 196843955
+      hash: 847368189
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.23
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: Terse configuration syntax for objects