ripar 0.0.1

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: f44f23505f92820e31ac7f4b67ede9e07e209983
+  data.tar.gz: 6994a3df9208181c7613edeba903268fe1624df3
+SHA512:
+  metadata.gz: c7387c6606f90ee3ea131064e2b9f0123d76bc7c7e8427a0a26f9d3f5ec7e3aea06ae64262f7adc1796b8615562346ca1e980b60a6dbc3b85e682cc16112546b
+  data.tar.gz: 24a3048c49f28abaa233ebeb22b3aba41b085f5711f65faf52e42101832da60fe35d545385058b15b3420f11ee5dfae0379a19067edd3e3ab606159ee0455e67

data/.gitignore

@@ -0,0 +1,17 @@
+*.gem
+*.rbc
+.bundle
+.config
+.yardoc
+Gemfile.lock
+InstalledFiles
+_yardoc
+coverage
+doc/
+lib/bundler/man
+pkg
+rdoc
+spec/reports
+test/tmp
+test/version_tmp
+tmp

data/.rvmrc

@@ -0,0 +1 @@
+rvm 2.1.1@ripar --create

data/.travis.yml

@@ -0,0 +1,9 @@
+language: ruby
+rvm:
+  # needs bundler-1.3.4, and 1.5 is in gemspec
+  # and the project uses 2.0 features
+  #- 1.9.3
+  - 2.0.0
+  # not supported by travis. yet?
+  #- 2.1.1
+script: bundle exec rspec spec

data/Gemfile

@@ -0,0 +1,24 @@
+def from_gemrc
+  # auto-load from ~/.gemrc
+  home_gemrc = Pathname('~/.gemrc').expand_path
+
+  if home_gemrc.exist?
+    require 'yaml'
+    # use all the sources specified in .gemrc
+    YAML.load_file(home_gemrc)[:sources]
+  end
+end
+
+# use the gemrc source if defined or CANON is truthy in ENV
+# otherwise just use the default
+def preferred_sources
+  rv = from_gemrc unless eval(ENV['CANON']||'')
+  rv ||= []
+  rv << 'http://rubygems.org' if rv.empty?
+  rv
+end
+
+preferred_sources.each{|src| source src}
+
+# Specify your gem's dependencies in ripar.gemspec
+gemspec

data/LICENSE.txt

@@ -0,0 +1,22 @@
+Copyright (c) 2014 John Anderson
+
+MIT License
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/README.md

@@ -0,0 +1,128 @@
+[![Gem Version](https://badge.fury.io/rb/ripar.png)](http://badge.fury.io/rb/ripar)
+
+# Ripar
+
+Think riparian. Think old man river, he jus' keep on rollin'. Think
+[rive](http://etymonline.com/index.php?search=rive). Also river, reaver,
+repair, reaper.
+
+Tear chained method calls apart, put them in a block, and return the block value. eg
+
+Before:
+``` ruby
+  result = values.select{|x| x.name =~ /Jo/}.map{|x| x.count}.inject(0){|s,x| s + x}
+```
+
+After:
+``` ruby
+  result = values.rive do
+    select{|x| x.name =~ /Jo/}
+    map{|x| x.count}
+    inject(0){|s,x| s + x}
+  end
+```
+
+This is also a little different to instance_eval, because the following will work:
+
+``` ruby
+  outside_block_regex = /Wahoody-hey/
+
+  result = values.rive do
+    select{|x| x.name =~ outside_block_regex}
+    map{|x| x.count}
+    inject(0){|s,x| s + x}
+  end
+```
+
+**Warning** this can have some rare but weird side effects:
+  - will probably break on classes that have defined method_missing, but not respond_to_missing
+  - an outside variable with the same name as an inside method
+    taking in-place hash argument will cause a syntax error.
+
+But you can obviate all of that by just using the safe syntax:
+
+``` ruby
+  outside_block_regex = /Wahoody-hey/
+
+  result = values.rive do |vs|
+    vs.select{|x| x.name =~ outside_block_regex}
+    vs.map{|x| x.count}
+    vs.inject(0){|s,x| s + x}
+  end
+```
+
+Or using the magic disambiguaters:
+
+``` ruby
+  select = /Wahoody-hey/
+
+  result = values.rive do
+    __inside__.select{|x| x.name =~ __outside__.select}
+    map{|x| x.count}
+    inject(0){|s,x| s + x}
+  end
+```
+
+## Installation
+
+Add this line to your application's Gemfile:
+
+    gem 'ripar'
+
+And then execute:
+
+    $ bundle
+
+Or install it yourself as:
+
+    $ gem install ripar
+
+## Usage
+
+In your class
+``` ruby
+class YourChainableThing
+  include Ripar
+end
+
+yct = YourChainableThing.new.rive do
+  # operations
+end
+```
+
+In a singleton
+``` ruby
+o = Object.new
+class << o
+  include Ripar
+end
+```
+
+Monkey-patch
+``` ruby
+class Object
+  include Ripar
+end
+```
+
+## Contributing
+
+The standard github pull request dance:
+
+  1. Fork it ( http://github.com/<my-github-username>/ripar/fork )
+  1. Create your feature branch (`git checkout -b my-new-feature`)
+  1. Commit your changes (`git commit -am 'Add some feature'`)
+  1. Push to the branch (`git push origin my-new-feature`)
+  1. Create new Pull Request
+
+## PS
+```ruby
+Thing = Struct.new :name, :count
+values = [
+  Thing.new('John', 20),
+  Thing.new('Joe', 7),
+  Thing.new('Paul', 3),
+  Thing.new('James', 3),
+  Thing.new('Wahoody-heydi-dude', 3.141527),
+]
+```

data/Rakefile

@@ -0,0 +1 @@
+require "bundler/gem_tasks"

data/lib/ripar.rb

@@ -0,0 +1,18 @@
+require "ripar/version"
+require "ripar/roller"
+
+# include this in a class/instance and put the calls that
+# would have been chained in the block.
+module Ripar
+  # return the final value
+  # short, unique, unlikely to clash with other methods in Object
+  # if you want to monkey-patch.
+  def rive( &block )
+    Roller.rive self, &block
+  end
+
+  # return the roller object containing the final value
+  def roller( &block )
+    Roller.new self, &block
+  end
+end

data/lib/ripar/combinder.rb

@@ -0,0 +1,107 @@
+# This is the part that lets an instance_eval-style block access
+# variables defined outside of it. Arguably that's a horrible idea, really.
+#
+# Implements method_missing to figure out which of binding.self and obj can handle the missing method.
+# Can have somewhat weird side effect of making variables assigned to lambdas (or anything
+# implementing call actually) directly callable.
+#
+# TODO @binding.eval('self') might define method_missing, so we may actually have to
+# attempt to call the method to see if it's missing.
+class Ripar::Combinder < BasicObject
+  # This should be a refinement
+  module BindingNiceness
+    def self
+      eval('self')
+    end
+
+    def local_variables
+      eval('local_variables')
+    end
+  end
+
+  def initialize( obj, saved_binding )
+    @obj, @binding = obj, saved_binding
+    @binding.extend BindingNiceness
+  end
+
+  class AmbiguousMethod < ::RuntimeError; end
+
+  # long method, but we want to keep BasicObject really empty.
+  # TODO could split into private __methods
+  def method_missing( meth, *args, &blk )
+    if @obj.respond_to?( meth ) && (@binding.self.methods - ::Object.instance_methods).include?( meth )
+      begin
+        return @obj.__ambiguous_method__( @binding.self, meth, *args, &blk )
+      rescue ::NoMethodError => ex
+        ::Kernel.raise AmbiguousMethod, "method :#{meth} exists on both #{@binding.self.inspect} (outside) and #{@obj.inspect} (inside) #{ex.message}"
+      end
+    end
+
+    if @binding.local_variables.include?( meth )
+    # This branch is only necessary to do lambda calls with (),
+    # because variables in the binding are already, well, part of the binding.
+    # So they are picked up before the code ever calls method_missing
+      bound_value = @binding.eval meth.to_s
+
+      if bound_value.respond_to?( :call )
+        # It's a local variable, but it's been forced to come here by (). So call it.
+        bound_value.call(*args)
+      else
+        # assume that the user really wants to call the object's method
+        # rather than access the outside variable.
+        @obj.send meth, *args, &blk
+      end
+    elsif @binding.self.respond_to?( meth )
+      @binding.self.send meth, *args, &blk
+    else
+      @obj.send meth, *args, &blk
+    end
+  end
+
+  def respond_to?( meth, include_all = false )
+    # ::Kernel.puts "Combinder#respond_to #{meth}"
+    # ::Kernel.puts "Combinder local variables #{@binding.local_variables}"
+    return @binding.local_variables.include?( meth ) || @binding.self.respond_to?( meth, include_all ) || @obj.respond_to?( meth, include_all )
+  end
+
+  # for disambiguating outside variables vs method calls, otherwise
+  # Ruby interpreter will find the outside variable name, and
+  # use that instead of the method call. You could also
+  # force the method call with (), but that is sometimes ugly.
+  def __inside__
+    @obj
+  end
+
+  # Forward method calls to bound variables
+  # Could probably just use Combinder here.
+  # TODO this should be accessing bound_self, right?
+  # Variables are accessed anyway by Ruby interpreter because
+  # They're in the binding. Not sure.
+  class BindingWrapper
+    def initialize( wrapped_binding )
+      @binding = wrapped_binding
+      @binding.extend BindingNiceness
+    end
+
+    def method_missing(meth, *args, &blk)
+      ::Kernel.raise "outside variables can't take arguments" if args.size != 0
+
+      if @binding.local_variables.include?( meth )
+        @binding.eval meth.to_s
+      elsif @binding.self.respond_to? meth
+        @binding.self.send meth, *args, &blk
+      else
+        ::Kernel.raise NoMethodError, "No such outside variable #{meth}"
+      end
+    end
+
+    def respond_to_missing?( meth, include_all = false )
+      @binding.local_variables.include?(meth) || @binding.self.respond_to?(meth, include_all)
+    end
+  end
+
+  # access the outside of the block, ie its binding.
+  def __outside__
+    BindingWrapper.new @binding
+  end
+end

data/lib/ripar/roller.rb

@@ -0,0 +1,112 @@
+require "ripar/version"
+require 'ripar/combinder.rb'
+
+# Pass in the original chainable object
+# method calls in block will be applied
+# resulting value will be in riven
+class Ripar::Roller < BasicObject
+  def initialize( original, &block )
+    @original = original
+    # clone is for protection or original - not strictly necessary?
+    @riven = original.clone
+    roll_block( &block ) if block
+  end
+
+  # Callback from Combinder.
+  #
+  # This happens when the outside self is_a?(original.class) already,
+  # and inside is the roller, which will both respond to the same set
+  # of methods. So we want the method calls to go to the roller inside the
+  # roller block, otherwise the current value of riven in the roller
+  # will end up being constantly reset to original. Which
+  # is exactly not the point.
+  def __ambiguous_method__( binding_self, meth, *args, &blk )
+    # TODO maybe this should just always default to the roller? I don't see much point
+    #   in being fancier than that.
+    # ::Kernel.puts "__ambiguous_method__ #{meth} for #{binding_self.inspect} and #{@obj.inspect}"
+    if binding_self == @original
+      # send it to the roller
+      send meth, *args, &blk
+    else
+      # don't know what to do with it, so let Combinder raise an AmbiguousMethod
+      # TODO too much coupling because this class knows about Combinder internals.
+      raise ::NoMethodError, meth
+    end
+  end
+
+  attr_accessor :riven, :original
+
+  # this is so that we a roller is returned from a
+  # call, you can call roller on it again, and keep
+  # things rolling along.
+  def roller( &block )
+    if block
+      roll_block &block
+    else
+      self
+    end
+  end
+
+  # instantiate the roller, pass it the block and
+  # immediately return the modified copy
+  def self.rive( original, &block )
+    new( original, &block ).riven
+  end
+
+  # Forward to riven, if the method exists.
+  def method_missing(meth, *args, &block)
+    if @riven.respond_to? meth
+      interim = @riven.send( meth, *args, &block )
+      # ::Kernel.puts "sending #{meth}(#{args.inspect}) to #{@riven.inspect}: #{interim}"
+      @riven = interim
+    else
+      super
+    end
+  end
+
+  # used by combinder, so must be defined, otherwise it perturbs method_missing
+  def send( meth, *args, &block )
+    method_missing( meth, *args, &block )
+  end
+
+  # used by combinder, so must be defined, otherwise it perturbs method_missing
+  # no point in using respond_to_missing?, because that's part of Object#respond_to,
+  # not BasicObject
+  def respond_to?( meth, include_all = false )
+    @riven.respond_to?(meth, include_all) || __methods__.include?(meth)
+  end
+
+  # make sure this BasicObject plays nicely in pry
+  def inspect
+    %Q{#<#{self.__class__} original: #{@original.inspect}, riven: #{@riven.inspect}>}
+  end
+
+  def pretty_inspect
+    inspect
+  end
+
+  def to_s; inspect; end
+
+  # include useful methods from Kernel, but rename
+  define_method :__class__, ::Kernel.instance_method(:class)
+  define_method :__object_id__, ::Kernel.instance_method(:object_id)
+
+protected
+
+  def roll_block( &block )
+    case block.arity
+    when 0
+      ::Ripar::Combinder.new( self, block.binding ).instance_eval &block
+    when 1
+      yield self
+    else
+      ::Kernel.raise "Don't know how to handle arity #{block.arity}"
+    end
+  end
+
+private
+
+  def __methods__
+    self.__class__.instance_methods
+  end
+end

data/lib/ripar/version.rb

@@ -0,0 +1,3 @@
+module Ripar
+  VERSION = "0.0.1"
+end

data/ripar.gemspec

@@ -0,0 +1,28 @@
+# coding: utf-8
+lib = File.expand_path('../lib', __FILE__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+require 'ripar/version'
+
+Gem::Specification.new do |spec|
+  spec.name          = "ripar"
+  spec.version       = Ripar::VERSION
+  spec.authors       = ["John Anderson"]
+  spec.email         = ["panic@semiosix.com"]
+  spec.summary       = %q{Chained methods cam be clunky. Use a block instead.}
+  spec.description   = %q{Convert series of chained methods from . syntax to block syntax. Like instance_eval but with access to external scope.}
+  spec.homepage      = "http://github.com/djellemah/ripar"
+  spec.license       = "MIT"
+
+  spec.files         = `git ls-files -z`.split("\x0")
+  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
+  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.require_paths = ["lib"]
+
+  spec.add_development_dependency "bundler", "~> 1.5"
+  spec.add_development_dependency "rake"
+  spec.add_development_dependency "rspec"
+  spec.add_development_dependency "pry"
+  spec.add_development_dependency "pry-debundle"
+  spec.add_development_dependency "pry-debugger"
+  spec.add_development_dependency "simplecov"
+end

data/spec/binding_wrapper_spec.rb

@@ -0,0 +1,40 @@
+require 'rspec'
+
+require Pathname(__dir__) + '../lib/ripar.rb'
+
+describe Ripar::Combinder::BindingWrapper do
+  describe '#method_missing' do
+    def flying_dutchman
+      'BigBoat'
+    end
+
+    subject do
+      unknown_soldier = 'SomeGuy'
+      Ripar::Combinder::BindingWrapper.new(binding)
+    end
+
+    it 'fails for args' do
+      ->{subject.unknown_soldier('hello')}.should raise_error(/can't take arguments/)
+    end
+
+    it 'finds local variables' do
+      subject.respond_to?(:unknown_soldier).should be_true
+      subject.unknown_soldier.should == 'SomeGuy'
+      meth = subject.method(:unknown_soldier)
+      meth.call.should == 'SomeGuy'
+    end
+
+    it 'finds self methods' do
+      subject.respond_to?(:flying_dutchman).should be_true
+      subject.flying_dutchman.should == 'BigBoat'
+      meth = subject.method(:flying_dutchman)
+      meth.call.should == 'BigBoat'
+    end
+
+    it 'error for not found method/variable' do
+      subject.respond_to?(:marie_celeste).should be_false
+      ->{subject.marie_celeste}.should raise_error(NoMethodError, /outside/)
+      ->{subject.method(:marie_celeste)}.should raise_error(NameError)
+    end
+  end
+end

data/spec/combinder_spec.rb

@@ -0,0 +1,163 @@
+require 'rspec'
+
+require Pathname(__dir__) + '../lib/ripar.rb'
+
+describe Ripar::Combinder do
+  def an_object
+    @an_object ||= Object.new.tap do |obj|
+      class << obj
+        def flying_dutchman
+          'BigBoat'
+        end
+
+        def oops
+          'dead mudokons'
+        end
+      end
+    end
+  end
+
+  # otherwise rspec tries to create a Combinder and fails
+  subject{}
+
+  # Yes, you really have to set up combinder in example,
+  # so you can get the binding
+
+  describe 'ambiguous method' do
+    def oops
+      'more dead mudokons'
+    end
+
+    it 'raise on ambiguous method' do
+      combinder = Ripar::Combinder.new(an_object, binding)
+      ->{combinder.instance_eval{ oops }}.should raise_error(Ripar::Combinder::AmbiguousMethod, /exists on both/)
+    end
+
+    it 'callback on ambiguous method' do
+      an_object.stub :__ambiguous_method__ do |binding_self, meth, *args|
+        binding_self.object_id.should == self.object_id
+        meth.should == :oops
+        args.should == []
+      end
+
+      Ripar::Combinder.new(an_object, binding).instance_eval{oops}
+    end
+  end
+
+  describe 'local variables' do
+    it 'gets value' do
+      unknown_soldier = 'SomeGuy'
+      combinder = Ripar::Combinder.new(an_object, binding)
+      combinder.instance_eval{ unknown_soldier }.should == 'SomeGuy'
+    end
+
+    it 'respond_to as methods' do
+      unknown_soldier = 'SomeGuy'
+      combinder = Ripar::Combinder.new(an_object, binding)
+      combinder.instance_eval{ respond_to? :unknown_soldier }.should be_true
+    end
+
+    it 'respond_to local variables as methods' do
+      unknown_soldier = 'SomeGuy'
+      combinder = Ripar::Combinder.new(an_object, binding)
+
+      combinder.instance_eval do
+        respond_to?(:unknown_soldier)
+      end.should be_true
+    end
+
+    it 'use respond_to_missing via method()' do
+      unknown_soldier = 'SomeGuy'
+      combinder = Ripar::Combinder.new(an_object, binding)
+
+      combinder.instance_eval do
+        meth = __outside__.method(:unknown_soldier)
+        meth.call
+      end.should == 'SomeGuy'
+    end
+
+    it 'force to inside method call with ()' do
+      flying_dutchman ='LovesCheese'
+
+      combinder = Ripar::Combinder.new(an_object, binding)
+
+      # this is the local variable, above
+      combinder.instance_eval{ flying_dutchman }.should == 'LovesCheese'
+
+      # this is the method call on an_object
+      combinder.instance_eval{ flying_dutchman() }.should == 'BigBoat'
+    end
+  end
+
+  describe 'self methods' do
+    it 'finds them' do
+      combinder = Ripar::Combinder.new(an_object, binding)
+      combinder.instance_eval{ respond_to?(:flying_dutchman) }.should be_true
+      combinder.instance_eval{ flying_dutchman }.should == 'BigBoat'
+    end
+
+    it 'method() works' do
+      pending "this is hard to support. not sure if it's even necessary"
+      combinder.instance_eval{ method(:flying_dutchman) }.call.should == 'BigBoat'
+    end
+  end
+
+  describe 'binding self methods' do
+    def hero
+      'missing, presumed dead'
+    end
+
+    it 'finds them' do
+      combinder = Ripar::Combinder.new(an_object, binding)
+      combinder.instance_eval{ respond_to?(:hero) }.should be_true
+      combinder.instance_eval{ hero }.should == 'missing, presumed dead'
+    end
+  end
+
+  describe 'non-existent variables/methods' do
+    it 'error for not found method/variable' do
+      combinder = Ripar::Combinder.new(an_object, binding)
+      combinder.instance_eval{subject.respond_to?(:marie_celeste)}.should be_false
+
+      lambda do
+        combinder.instance_eval{marie_celeste}
+      end.should raise_error(NoMethodError)
+    end
+
+    it 'method for not found method/variable' do
+      pending 'ditto'
+      combinder.instance_eval do
+        ->{method(:marie_celeste)}.should raise_error(NameError)
+      end
+    end
+  end
+
+  describe '() for .call' do
+    it 'calls lambda' do
+      fn = ->(val){"#{val}, where are you?"}
+
+      combinder = Ripar::Combinder.new(an_object, binding)
+      combinder.instance_eval{fn(oops)}.should == 'dead mudokons, where are you?'
+    end
+
+    it 'calls #call' do
+      obj = Object.new
+      class << obj
+        def call( finagle )
+          finagle * 2
+        end
+      end
+
+      combinder = Ripar::Combinder.new(an_object, binding)
+      combinder.instance_eval{obj(oops)}.should == 'dead mudokonsdead mudokons'
+    end
+  end
+
+  describe '#__inside__' do
+    it 'allows access to inside method' do
+      combinder = Ripar::Combinder.new(an_object, binding)
+      combinder.instance_eval{__inside__}.should == an_object
+    end
+  end
+end
+

data/spec/cov.rb

@@ -0,0 +1,6 @@
+# must be before coverage'd code is loaded
+require 'simplecov'
+
+SimpleCov.start do
+  add_filter '/spec/'
+end

data/spec/roller_spec.rb

@@ -0,0 +1,259 @@
+require 'rspec'
+
+require Pathname(__dir__) + '../lib/ripar.rb'
+
+describe Ripar::Roller do
+  def collection
+    @collection ||= [1,2,3,4,5,6,7,8].tap do |ry|
+      class << ry
+        def multiply( factor: 2 )
+          map{|x| x * factor}
+        end
+
+        include Ripar
+      end
+    end
+  end
+
+  # this deliberately doesn't have a ?
+  def even( arg )
+    arg % 2 == 0
+  end
+
+  def seven
+    7
+  end
+
+  it 'returns an instance immediately' do
+    rvs = collection.rive do
+      reverse
+    end
+
+    rvs.should_not be_respond_to(:__class__)
+    rvs.should be_a(Array)
+    collection.should == rvs.reverse
+  end
+
+  it 'riven returns an instance' do
+    rlr = collection.roller
+    rlr.riven.should_not be_respond_to(:__class__)
+    rlr.riven.should be_a(collection.class)
+  end
+
+  it 'pretends to be the original class' do
+    rlr = collection.roller
+    rlr.should be_a(Array)
+  end
+
+  it 'takes a 1 arg block' do
+    rlr = collection.roller do |rlr|
+      rlr.original.should be_a(Array)
+    end
+  end
+
+  it 'applies a 1 arg block' do
+    evens = collection.rive do |vs|
+      vs.select{|x| even(x)}
+    end
+
+    evens.should be_a(Array)
+    evens.should == [2,4,6,8]
+  end
+
+  # And this is the really important one
+  it 'takes a 0 arg block' do
+    method(:even).should be_a(Method)
+
+    rlr = collection.roller do
+      select{|x| even(x)}
+    end
+
+    rlr.riven.should be_a(Array)
+    rlr.riven.should == [2,4,6,8]
+  end
+
+  it 'fails for -1 arity' do
+    ->{ collection.roller {|*args|} }.should raise_error(/arity -1/)
+  end
+
+  it 'fails for 2 arity' do
+    ->{ collection.roller {|arg1,arg2|} }.should raise_error(/arity 2/)
+  end
+
+  it 'disambiguates outside variable and inside method call' do
+    multiply = 'go forth and'
+    rlr = collection.roller do
+      __inside__.multiply factor: 3
+    end
+    rlr.riven.should == collection.multiply( factor: 3 )
+  end
+
+  it 'access outside variable when name clashes' do
+    reverse = 2
+    rlr = collection.roller do
+      select{|x| x < reverse }
+    end
+    rlr.riven.should == [1]
+  end
+
+  it '(...) will force inside method call and ignore outside local variable' do
+    delete = 'doggone'
+    rlr = collection.roller do
+      delete(2)
+    end
+    rlr.riven.should == 2
+  end
+
+  it '() will force inside method call' do
+    reverse = 'sdrawkcab'
+
+    rlr = collection.roller do
+      reverse()
+    end
+    rlr.riven.should == collection.reverse
+
+    rlr = collection.roller do
+      reverse
+    end
+
+    rlr.riven.should_not == collection.reverse
+  end
+
+  it 'syntax error for outside variable and inside method call in-place hash syntax' do
+    multiply = 'go forth and'
+    rlr = collection.roller do
+      pending "Not possible for in-place hash syntax"
+      # This will cause a syntax error because the interpreter
+      # finds the outside variable, and we're trying to call a method.
+      # but only if we're using the in-place hash syntax
+      # multiply factor: 3
+    end
+    rlr.riven.should == [3,6,9,12,15,18,21,24]
+  end
+
+  it 'in-place hash syntax with name clash ok with (...)' do
+    multiply = 'go forth and'
+    rlr = collection.roller do
+      multiply( factor: 3 )
+    end
+    rlr.riven.should == [3,6,9,12,15,18,21,24]
+  end
+
+  it 'allows explicit access to outside variables' do
+    selection = 8
+    rlr = collection.roller do
+      select{|x| x == __outside__.selection}
+    end
+    rlr.riven.should == [8]
+  end
+
+  it 'allows explicit access to outside methods' do
+    rlr = collection.roller do
+      select{|x| x == __outside__.seven}
+    end
+    rlr.riven.should == [7]
+  end
+
+  it 'allows variable assignment inside' do
+    filter = 'hello'
+    rlr = collection.roller do
+      irl = __inside__
+      irl.map{|x| x-1}
+    end
+    rlr.riven.should == [0,1,2,3,4,5,6,7]
+  end
+
+  it 'can directly call lambdas' do
+    fn = ->(x){x + 4}
+    rlr = collection.roller do
+      select{|x| x < fn(1)}
+    end
+    rlr.riven.should == [1,2,3,4]
+  end
+
+  it 'can directly call outside callables' do
+    fn = Object.new.tap do |obj|
+      class << obj
+        def call(ignored)
+          3
+        end
+      end
+    end
+
+    rlr = collection.roller do
+      select{|x| x < fn(1)}
+    end
+    rlr.riven.should == [1,2]
+  end
+
+  it 'can access outside lambdas' do
+    fn = ->(x){x.odd?}
+    rlr = collection.roller do
+      select &fn
+    end
+    rlr.riven.should == [1,3,5,7]
+  end
+
+  describe 'ambiguous method' do
+    def to_be_duplicated
+    end
+
+    it 'handles non-nested ambiguity' do
+      class << collection
+        def to_be_duplicated
+          require 'pry'; binding.pry
+          flat_map{|x| [x,x]}
+        end
+      end
+
+      ->{collection.roller{ to_be_duplicated }}.should raise_error(Ripar::Combinder::AmbiguousMethod)
+    end
+
+    it 'handles nested rollers' do
+      class << collection
+        def square_evens
+          roller do
+            select &:even?
+            map{|x| x ** 2}
+          end
+        end
+      end
+
+      collection.square_evens.riven.should == [4,16,36,64]
+
+      squares_doubled = collection.roller do
+        square_evens
+        self * 2
+      end
+
+      squares_doubled.inspect.should =~ /Roller/
+      squares_doubled.riven.should == [4,16,36,64] * 2
+      squares_doubled.roller{count}.should == 8
+
+      # TODO this should really be elsewhere
+      squares_doubled.riven.should == 8
+    end
+  end
+
+  describe '#roller' do
+    it 'returns self with no block' do
+      rlr = collection.roller
+      rlr.__class__.should == Ripar::Roller
+      rlr.__object_id__.should == rlr.roller.__object_id__
+    end
+  end
+
+  describe '#inspect' do
+    it 'has class name' do
+      collection.roller.inspect.should =~ /Ripar::Roller/
+    end
+
+    it 'has original' do
+      collection.roller.original.should == collection
+    end
+
+    it 'has riven' do
+      collection.roller.riven.object_id.should_not == collection.object_id
+    end
+  end
+end

metadata

@@ -0,0 +1,163 @@
+--- !ruby/object:Gem::Specification
+name: ripar
+version: !ruby/object:Gem::Version
+  version: 0.0.1
+platform: ruby
+authors:
+- John Anderson
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2014-03-10 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.5'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.5'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: pry-debundle
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: pry-debugger
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+description: Convert series of chained methods from . syntax to block syntax. Like
+  instance_eval but with access to external scope.
+email:
+- panic@semiosix.com
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- ".gitignore"
+- ".rvmrc"
+- ".travis.yml"
+- Gemfile
+- LICENSE.txt
+- README.md
+- Rakefile
+- lib/ripar.rb
+- lib/ripar/combinder.rb
+- lib/ripar/roller.rb
+- lib/ripar/version.rb
+- ripar.gemspec
+- spec/binding_wrapper_spec.rb
+- spec/combinder_spec.rb
+- spec/cov.rb
+- spec/roller_spec.rb
+homepage: http://github.com/djellemah/ripar
+licenses:
+- MIT
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubyforge_project: 
+rubygems_version: 2.2.2
+signing_key: 
+specification_version: 4
+summary: Chained methods cam be clunky. Use a block instead.
+test_files:
+- spec/binding_wrapper_spec.rb
+- spec/combinder_spec.rb
+- spec/cov.rb
+- spec/roller_spec.rb