dumb_delegator 0.6.0 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 92d88fb2dc6ea85760c5b7d646349b40bca6dbfe
-  data.tar.gz: e457fa632733529197f767f6fc798e35d57db3fe
+  metadata.gz: cae66a8a74772ba193dd066ab5b030b6cc9ab526
+  data.tar.gz: dfa72a06e4197d605dcd0dbb3d4865c92d9b2d61
 SHA512:
-  metadata.gz: 6ca10a81f035210ac1b7b45b579e765af3baa14c2c9fa349814fac4aea22bc1a008d888145bef02f53048fbe9bdf43da4b8eb5db9d8f9c002e4d5705c3eba976
-  data.tar.gz: a641af0af00a6999fa5d3f5484a742db80d23c47e7b31475673ec66080219f4d0b127158fdbd434f4e16b042f7ff848528358c470e760c93543ef7830ef111a2
+  metadata.gz: 8df3f3d465f33af6d63aa2f9444effa6dbcdbc22e8b1d7b6062030ba3ea5921618602e8bc6d69fcecd7b9fc53d45e0e5b691a95a8342dbb452ab5946fda4826f
+  data.tar.gz: 93dd9e69cd72bd079f84070eba0f077c267f3432e0566b3d6135cb911a0142a0b858b15717edc4578db1d63dad13dc6be6dbaa11d9089fa75aeb3517ccff2a0f

data/.gitignore

@@ -3,6 +3,7 @@
 .bundle
 .config
 .yardoc
+.ruby-version
 Gemfile.lock
 InstalledFiles
 _yardoc

data/.travis.yml

@@ -4,5 +4,6 @@ rvm:
   - 1.9.3
   - 2.0.0
   - 2.1.0
+  - 2.1.1
   - jruby-19mode
   - rbx

data/README.md

@@ -1,7 +1,9 @@
 # DumbDelegator
 
-[![Build Status](https://secure.travis-ci.org/stevenharman/dumb_delegator.png)](http://travis-ci.org/stevenharman/dumb_delegator)
-[![Dependency Status](https://gemnasium.com/stevenharman/dumb_delegator.png)](https://gemnasium.com/stevenharman/dumb_delegator)
+[![Gem Version](http://img.shields.io/gem/v/dumb_delegator.svg)](https://rubygems.org/gems/dumb_delegator)
+[![Build Status](https://img.shields.io/travis/stevenharman/dumb_delegator.svg)](https://travis-ci.org/stevenharman/dumb_delegator)
+[![Code Climate](http://img.shields.io/codeclimate/github/stevenharman/dumb_delegator.svg)](https://codeclimate.com/github/stevenharman/dumb_delegator)
+[![Dependency Status](https://img.shields.io/gemnasium/stevenharman/dumb_delegator.svg)](https://gemnasium.com/stevenharman/dumb_delegator)
 
 Ruby provides the `delegate` standard library. However, we found that it is not
 appropriate for many use cases that require nearly every call to be proxied.
@@ -62,7 +64,7 @@ class Coffee
   end
 
   def origin
-    "Colombia"
+    'Colombia'
   end
 end
 

data/Rakefile

@@ -1,9 +1,9 @@
 #!/usr/bin/env rake
-require "bundler/gem_tasks"
+require 'bundler/gem_tasks'
 
 require 'rspec/core/rake_task'
 RSpec::Core::RakeTask.new(:spec) do |t|
-  t.rspec_opts = "--tag ~objectspace" if RUBY_PLATFORM == "java"
+  t.rspec_opts = '--tag ~objectspace' if RUBY_PLATFORM == 'java'
 end
 
 task :default => :spec

data/lib/dumb_delegator.rb

@@ -1,5 +1,5 @@
-require "set"
-require "dumb_delegator/version"
+require 'set'
+require 'dumb_delegator/version'
 
 class DumbDelegator < ::BasicObject
   (::BasicObject.instance_methods - [:equal?, :__id__, :__send__, :method_missing]).each do |method|
@@ -33,6 +33,7 @@ class DumbDelegator < ::BasicObject
   end
 
   def __setobj__(obj)
+    raise ::ArgumentError, 'Delegation to self is not allowed.' if obj.__id__ == __id__
     @__dumb_target__ = obj
   end
 

data/lib/dumb_delegator/version.rb

@@ -1,3 +1,3 @@
 class DumbDelegator < ::BasicObject
-  VERSION = "0.6.0"
+  VERSION = '0.7.0'
 end

data/spec/dumb_delegator_spec.rb

@@ -1,28 +1,28 @@
-require "spec_helper"
+require 'spec_helper'
 
 describe DumbDelegator do
   subject(:dummy) { described_class.new(target) }
   let(:target) { double }
 
-  it "delegates to the target object" do
+  it 'delegates to the target object' do
     expect(target).to receive(:foo)
     dummy.foo
   end
 
-  it "delegates to the target object with arguments" do
+  it 'delegates to the target object with arguments' do
     expect(target).to receive(:foo).with(:bar)
 
     dummy.foo(:bar)
   end
 
-  it "delegates to the target object with a block" do
-    bar_block = proc { "bar" }
+  it 'delegates to the target object with a block' do
+    bar_block = proc { 'bar' }
     expect(target).to receive(:foo) { |&block| expect(block).to eq(bar_block) }
 
     dummy.foo(&bar_block)
   end
 
-  it "does not delegate if the target does not respond_to? the message" do
+  it 'does not delegate if the target does not respond_to? the message' do
     allow(target).to receive(:foo)
     allow(target).to receive(:respond_to?).with(:foo).and_return(false)
 
@@ -31,80 +31,80 @@ describe DumbDelegator do
     }.to raise_error(NoMethodError)
   end
 
-  it "responds to methods defined by child classes that add behavior" do
+  it 'responds to methods defined by child classes that add behavior' do
     expect(target).to receive(:foo).never
     def dummy.foo
-      "bar"
+      'bar'
     end
 
     dummy.foo
   end
 
-  it "delegates methods defined on Object" do
+  it 'delegates methods defined on Object' do
     expect(target).to receive(:class)
     dummy.class
   end
 
-  it "delegates is_a?" do
+  it 'delegates is_a?' do
     expect(target).to receive(:is_a?)
     dummy.is_a?
   end
 
-  it "delegates methods defined on Kernel" do
+  it 'delegates methods defined on Kernel' do
     expect(target).to receive(:print)
     dummy.print
   end
 
-  it "delegates !" do
+  it 'delegates !' do
     expect(target).to receive(:!)
     !dummy
   end
 
-  it "delegates !=" do
+  it 'delegates !=' do
     expect(target).to receive(:!=)
     dummy != 1
   end
 
-  it "delegates ==" do
+  it 'delegates ==' do
     expect(target).to receive(:==)
     dummy == 1
   end
 
-  it "delegates ==" do
+  it 'delegates ==' do
     expect(target).to receive(:==)
     dummy == 1
   end
 
-  it "delegates instance_eval" do
+  it 'delegates instance_eval' do
     expect(target).to receive(:instance_eval)
     dummy.instance_eval { true }
   end
 
-  it "delegates instance_exec" do
+  it 'delegates instance_exec' do
     expect(target).to receive(:instance_exec)
     dummy.instance_exec { true }
   end
 
-  describe "#dup" do
-    it "returns a shallow of itself, the delegator (not the underlying object)", :objectspace => true do
+  describe '#dup' do
+    it 'returns a shallow of itself, the delegator (not the underlying object)', :objectspace => true do
       dupped = dummy.dup
 
       expect(ObjectSpace.each_object(DumbDelegator).map(&:__id__)).to include dupped.__id__
     end
   end
 
-  describe "#clone" do
-    it "returns a shallow of itself, the delegator (not the underlying object)", :objectspace => true do
+  describe '#clone' do
+    it 'returns a shallow of itself, the delegator (not the underlying object)', :objectspace => true do
       cloned = dummy.clone
 
       expect(ObjectSpace.each_object(DumbDelegator).map(&:__id__)).to include cloned.__id__
     end
   end
 
-  describe "marshaling" do
+  describe 'marshaling' do
     let(:target) { Object.new }
 
-    it "marshals and unmarshals itself, the delegator (not the underlying object)", :objectspace => true do
+    it 'marshals and unmarshals itself, the delegator (not the underlying object)', :objectspace => true do
       marshaled = Marshal.dump(dummy)
       unmarshaled = Marshal.load(marshaled)
 
@@ -112,30 +112,30 @@ describe DumbDelegator do
     end
   end
 
-  describe "#respond_to?" do
+  describe '#respond_to?' do
     [:equal?, :__id__, :__send__, :dup, :clone, :__getobj__, :__setobj__, :marshal_dump, :marshal_load, :respond_to?].each do |method|
       it "responds to #{method}" do
         expect(dummy.respond_to?(method)).to be_true
       end
     end
 
-    context "subclasses of DumbDelegator" do
+    context 'subclasses of DumbDelegator' do
       subject(:dummy) { Class.new(DumbDelegator) { def foobar; end }.new([]) }
 
-      it "respond to methods defined on the subclass" do
+      it 'respond to methods defined on the subclass' do
         expect(dummy.respond_to?(:foobar)).to be_true
       end
     end
   end
 
-  describe "#__getobj__" do
-    it "returns the target object" do
+  describe '#__getobj__' do
+    it 'returns the target object' do
       expect(dummy.__getobj__).to equal target
     end
   end
 
-  describe "#__setobj__" do
-    it "resets the target object to a different object" do
+  describe '#__setobj__' do
+    it 'resets the target object to a different object' do
       expect(target).to receive(:foo).never
 
       new_target = double
@@ -144,5 +144,12 @@ describe DumbDelegator do
       dummy.__setobj__(new_target)
       dummy.foo
     end
+
+    it 'cannot delegate to itself' do
+      expect {
+        dummy.__setobj__(dummy)
+        dummy.foo
+      }.to raise_error(ArgumentError, 'Delegation to self is not allowed.')
+    end
   end
 end

data/spec/spec_helper.rb

@@ -1,6 +1,6 @@
 # This file was generated by the `rspec --init` command. Conventionally, all
 # specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
-# Require this file using `require "spec_helper.rb"` to ensure that it is only
+# Require this file using `require 'spec_helper.rb'` to ensure that it is only
 # loaded once.
 #
 # See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dumb_delegator
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.7.0
 platform: ruby
 authors:
 - Andy Lindeman
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-03-10 00:00:00.000000000 Z
+date: 2014-03-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -101,3 +101,4 @@ summary: Delegator and SimpleDelegator in Ruby's stdlib are somewhat useful, but
 test_files:
 - spec/dumb_delegator_spec.rb
 - spec/spec_helper.rb
+has_rdoc: