smoke_monster 0.3.0 → 0.3.2

data/Gemfile

@@ -3,6 +3,9 @@ source 'https://rubygems.org'
 # Specify your gem's dependencies in smoke_monster.gemspec
 gemspec
 
+gem 'blankslate'
+gem 'mocha'
+
 
 gem 'minitest'
 group :development do

data/lib/smoke_monster/array_to_object.rb

@@ -3,7 +3,7 @@ class Array
   def to_objects(&blk)
     records = blk.call
     return [] if records.empty?
-    records.map{ |record| create_object_for_this_record(record) }
+    records.map { |record| create_object_for_this_record(record) }
   end
 
   private

data/lib/smoke_monster/cover.rb

@@ -1,5 +1,7 @@
+require 'blankslate'
+
 module SmokeMonster
-  class Cover
+  module CoverMethods
     def initialize(subject)
       @subject = subject
     end
@@ -7,11 +9,21 @@ module SmokeMonster
     def method_missing(meth, *args, &blk)
       @subject.send(meth, *args, &blk)
     rescue
-      SmokeMonster::Cover.new nil
+      SmokeMonster::Cover.new Object.new
     end
 
     def the_original_subject
       @subject
     end
   end
+
+  module StrictCoverMethods
+    def method_missing(meth, *args, &blk)
+      @subject.send(meth, *args, &blk)
+    end
+  end
+
+  class Cover < ::BlankSlate
+    include SmokeMonster::CoverMethods
+  end
 end

data/lib/smoke_monster/lazy_cover.rb

@@ -1,27 +1,27 @@
 module SmokeMonster
   class LazyCover < Cover
+    include SmokeMonster::StrictCoverMethods
+
     def initialize(block)
       @block = block
-      self.public_methods.select{ |x| x.to_s.include?("the_original_subject") == false }.each do |method|
-        remove_method method
-      end
-      @can_allow_method_missing_to_work_now = true
     end
 
-    alias :old_the_original_subject :the_original_subject
+    alias :the_original_subject_from_the_base_class :the_original_subject
     def the_original_subject
-      @subject = @block.call
-      old_the_original_subject
+      setup_the_subject
+      the_original_subject_from_the_base_class
     end
 
     private
 
     alias :old_method_missing :method_missing
     def method_missing(meth, *args, &blk)
-      return nil unless @can_allow_method_missing_to_work_now
-      @subject = @block.call
+      setup_the_subject
       old_method_missing meth, *args, &blk
     end
 
+    def setup_the_subject
+      @subject = @block.call if @subject.nil?
+    end
   end
 end

data/lib/smoke_monster/version.rb

@@ -1,3 +1,3 @@
 module SmokeMonster
-  VERSION = "0.3.0"
+  VERSION = "0.3.2"
 end

data/smoke_monster.gemspec

@@ -14,4 +14,8 @@ Gem::Specification.new do |gem|
   gem.name          = "smoke_monster"
   gem.require_paths = ["lib"]
   gem.version       = SmokeMonster::VERSION
+
+  # specify any dependencies here; for example:
+  gem.add_runtime_dependency 'blankslate'
+  gem.add_development_dependency 'mocha'
 end

data/spec/smoke_monster/cover_spec.rb

@@ -14,9 +14,9 @@ describe SmokeMonster::Cover do
       @view_model.or_this_either
     end
 
-    it "should return a view model if the method does not exist" do
-      @view_model.should_be_a_view_model.must_be_instance_of SmokeMonster::Cover
-      @view_model.another_test.must_be_instance_of SmokeMonster::Cover
+    it "should return an object if the method does not exist" do
+      @view_model.should_be_a_view_model.wont_be_nil
+      @view_model.another_test.wont_be_nil
     end
 
     it "should return new view models each time on methods that do not exist" do

data/spec/smoke_monster/lambda_to_object_spec.rb

@@ -6,9 +6,6 @@ describe "lambda to object" do
     before do
       @value = -> { nil }.to_object
     end
-    it "should return a LazyCover object" do
-      @value.class.must_equal SmokeMonster::LazyCover
-    end
 
     it "should remember the original value is nil" do
       @value.the_original_subject.must_equal nil 
@@ -19,9 +16,6 @@ describe "lambda to object" do
     before do
       @value = -> { 1 }.to_object
     end
-    it "should return a LazyCover object" do
-      @value.class.must_equal SmokeMonster::LazyCover
-    end
 
     it "should remember the original value is 1" do
       @value.the_original_subject.must_equal 1

data/spec/smoke_monster/lazy_cover_spec.rb

@@ -0,0 +1,65 @@
+require 'mocha'
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+class LazyCoverTest
+  class << self
+    attr_accessor :was_called
+  end
+end
+describe SmokeMonster::LazyCover do
+
+  it "should not call the block passed to it" do
+    lazy = SmokeMonster::LazyCover.new(-> { raise 'was called' })
+  end
+
+  it "should call the block passed to it when things are references" do
+    LazyCoverTest.was_called = false
+    lazy = SmokeMonster::LazyCover.new(-> { LazyCoverTest.was_called = true })
+    LazyCoverTest.was_called.must_equal false
+    lazy.to_s
+    LazyCoverTest.was_called.must_equal true
+  end
+
+  it "should call the block passed to it only once" do
+    LazyCoverTest.was_called = 0
+    lazy = SmokeMonster::LazyCover.new(-> { LazyCoverTest.was_called = LazyCoverTest.was_called + 1 })
+    LazyCoverTest.was_called.must_equal 0
+    lazy.to_s
+    LazyCoverTest.was_called.must_equal 1
+    lazy.to_s
+    LazyCoverTest.was_called.must_equal 1
+  end
+
+  it "should return the same exceptions passed from its base class methods" do
+
+    object = Object.new
+    object.stubs(:defined_method).returns("yes")
+
+    lazy = SmokeMonster::LazyCover.new(-> { object })
+    lazy.defined_method.must_equal "yes"
+    was_called = false
+    begin
+      lazy.not_a_defined_method
+    rescue NoMethodError
+      was_called = true
+    end
+    was_called.must_equal true
+  end
+
+  describe "the original subject" do
+    it "should call the block passed to it" do
+      SmokeMonster::LazyCover.new(-> { 0 }).the_original_subject.must_equal 0
+      SmokeMonster::LazyCover.new(-> { "x" }).the_original_subject.must_equal "x"
+    end
+
+    it "should only call the block passed to it once" do
+      LazyCoverTest.was_called = 0
+      lazy = SmokeMonster::LazyCover.new(-> { LazyCoverTest.was_called += 1 })
+      lazy.the_original_subject
+      LazyCoverTest.was_called.must_equal 1
+      lazy.the_original_subject
+      LazyCoverTest.was_called.must_equal 1
+    end
+  end
+
+end

data/spec/spec_helper.rb

@@ -1,3 +1,4 @@
+require 'mocha'
 require 'minitest/spec'
 require 'minitest/autorun'
 require './lib/smoke_monster.rb'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: smoke_monster
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,8 +9,30 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-07-06 00:00:00.000000000 Z
-dependencies: []
+date: 2012-07-07 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: blankslate
+  requirement: &70290441103460 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: *70290441103460
+- !ruby/object:Gem::Dependency
+  name: mocha
+  requirement: &70290441102820 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *70290441102820
 description: Unexplained, dark magic.
 email:
 - darren@cauthon.com
@@ -35,6 +57,7 @@ files:
 - spec/smoke_monster/array_to_object_spec.rb
 - spec/smoke_monster/cover_spec.rb
 - spec/smoke_monster/lambda_to_object_spec.rb
+- spec/smoke_monster/lazy_cover_spec.rb
 - spec/smoke_monster/safety_proc_spec.rb
 - spec/spec_helper.rb
 homepage: http://www.github.com/darrencauthon/smoke_monster
@@ -65,5 +88,6 @@ test_files:
 - spec/smoke_monster/array_to_object_spec.rb
 - spec/smoke_monster/cover_spec.rb
 - spec/smoke_monster/lambda_to_object_spec.rb
+- spec/smoke_monster/lazy_cover_spec.rb
 - spec/smoke_monster/safety_proc_spec.rb
 - spec/spec_helper.rb