bluff 0.0.4 → 0.1.0

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## Jan 16 2012 - v0.1.0
+
+* Migrated over to new persistence/adapter terminology
+
 ## Jan 12 2012 - v0.0.4
 
 * Bluff calls are stricter now to give us more control in the bluff definitions. A bluff call accepts a single hash argument and default block values can no longer be defined in the block signatures.

data/README.md

@@ -86,12 +86,14 @@ Feel free to use them or huck em to the curb and just work directly with the att
 
 ```
 User.bluff  # returns an unsaved instance -- could also use Bluff.user
-User.bluff! # returns a saved instance -- could also use Bluff.user!
+User.bluff! # the "bang bluff" -- returns a persisted/saved instance, could also use Bluff.user!
 User.bluff({:name => 'Ryan'}) # supply overrides/defaults in the attributes hash
 ```
 
 ### METHOD BLUFFS (MOCKS / STUBS)
 
+** NOT IMPLEMENTED YET **
+
 All objects and classes are given a `bluff()` method, which is
 used for bluffing both models and methods.  If given a symbol, the 
 `bluff()` method can be used to define method stubs as follows:

data/lib/bluff/builder.rb

@@ -3,8 +3,8 @@ require 'active_support/core_ext/array/wrap'
 require 'active_support/core_ext/array/conversions'
 require 'active_support/core_ext/hash/indifferent_access'
 
-require 'bluff/support/backend'
-require 'bluff/support/backend/active_record'
+require 'bluff/support/persistence/base'
+require 'bluff/support/persistence/active_record'
 
 module Bluff
   module Builder
@@ -97,7 +97,7 @@ module Bluff
       def define_bluff_bang(field)
         define_singleton_method "#{field}!" do |*arguments|
           send(field, *arguments).tap do |record|
-            Bluff::Support::Backend.save!(record, field)
+            Bluff::Support.persist(record, {:as => field})
           end
         end
       end

data/lib/bluff/support/{backend → persistence}/active_record.rb

@@ -3,14 +3,14 @@ module ActiveRecord
   end
 end
 
-module Bluff::Support::ActiveRecord
-  class Backend
+module Bluff::Support::Persistence
+  class ActiveRecordAdapter
     class << self
-      def accepts?(record)
+      def persists?(record)
         record.is_a?(ActiveRecord::Base)
       end
 
-      def save!(record)
+      def persist(record)
         ActiveRecord::Base.transaction do
           record.class.reflect_on_all_associations(:belongs_to).each do |reflection|
             association = record.send(reflection.name)
@@ -22,6 +22,6 @@ module Bluff::Support::ActiveRecord
       end
     end
 
-    Bluff::Support::Backend.register(self)
+    Bluff::Support::Persistence::Base.register(self)
   end
 end

data/lib/bluff/support/persistence/base.rb

@@ -0,0 +1,31 @@
+module Bluff
+  module Support
+    module Persistence
+      class Base
+        @@adapters = []
+    
+        class << self
+          def register(adapter)
+            @@adapters << adapter
+          end
+      
+          def persist(record, options = {})
+            adapters = @@adapters.select {|adapter| adapter.persists?(record)}
+        
+            if adapters.empty?
+              raise "#{options[:as] || record.class} is not persistable (#{record})"
+            else
+              # should first try to persist new associations
+              adapters.first.persist(record)
+            end
+          end
+        end
+      end
+    end
+    
+    # just looks a little cleaner in use
+    def self.persist(record, options = {})
+      Persistence::Base.persist(record, options)
+    end
+  end
+end

data/lib/bluff/support/support.rb

@@ -11,6 +11,6 @@
 
 module Bluff
   module Support
-    # autoload
+    # autoload bluffs for rspec
   end
 end

data/lib/bluff/version.rb

@@ -1,3 +1,3 @@
 module Bluff
-  VERSION = "0.0.4"
+  VERSION = "0.1.0"
 end

data/notes.md

@@ -0,0 +1,36 @@
+# Notes
+
+These notes are intended simply for my own use.
+
+## Bang bluffs
+
+If a bang bluff is called for a bluff that isn't persistable an error should be thrown.
+
+If the bluff is persistable, all persistable associations within the bluffed object should
+be persisted within a transaction (persisting from the inside out).
+
+
+## Block argument reflection
+
+Can use code from merb to reflect on expected block arguments and default values
+
+```
+require 'rubygems'
+require 'merb'
+
+include GetArgs
+
+def foo(bar, zed=42)
+end
+
+method(:foo).get_args # => [[[:bar], [:zed, 42]], [:zed]]
+```
+
+Sourcify is another option but doesn't seem to support blocks with default arguments
+
+Also have Proc#parameters but it doesn't look like it tells you what the default argument value is reported.
+
+
+If we can find a way to do this we can loosen up the single-argument bluff restriction. Only bluffs without
+predefined arguments and bluffs with a hash as the first argument will be given special treatment.
+

data/spec/models/api_spec.rb

@@ -86,7 +86,7 @@ describe Bluff do
         end
         
         Bluff.for(:transient_class) { TransientClass.new }
-        lambda{ Bluff.transient_class! }.should raise_error(RuntimeError, /cannot be bang bluffed/)
+        lambda{ Bluff.transient_class! }.should raise_error(RuntimeError, /transient_class is not persistable/)
       end
     end
     

data/spec/support/bluffable_class.rb

@@ -12,18 +12,18 @@ class BluffableClass
   end
 end
 
-module Bluff::Support::BluffableClass
-  class Backend
+module Bluff::Support::Persistence
+  class BluffableClassAdapter
     class << self
-      def accepts?(record)
+      def persists?(record)
         record.is_a?(BluffableClass)
       end
 
-      def save!(record)
+      def persist(record)
         record.save
       end
     end
-
-    Bluff::Support::Backend.register(self)
+  
+    Bluff::Support::Persistence::Base.register(self)
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bluff
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.1.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-01-12 00:00:00.000000000Z
+date: 2012-01-16 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &2166846800 !ruby/object:Gem::Requirement
+  requirement: &2156611420 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: '3.0'
   type: :development
   prerelease: false
-  version_requirements: *2166846800
+  version_requirements: *2156611420
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &2166845760 !ruby/object:Gem::Requirement
+  requirement: &2156610620 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +32,10 @@ dependencies:
         version: '2.8'
   type: :development
   prerelease: false
-  version_requirements: *2166845760
+  version_requirements: *2156610620
 - !ruby/object:Gem::Dependency
   name: faker
-  requirement: &2166844480 !ruby/object:Gem::Requirement
+  requirement: &2156609620 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,7 +43,7 @@ dependencies:
         version: '1.0'
   type: :development
   prerelease: false
-  version_requirements: *2166844480
+  version_requirements: *2156609620
 description: ''
 email:
 - ryan.mohr@gmail.com
@@ -61,10 +61,11 @@ files:
 - lib/bluff/bluffs/data_bluffs.rb
 - lib/bluff/builder.rb
 - lib/bluff/configuration.rb
-- lib/bluff/support/backend.rb
-- lib/bluff/support/backend/active_record.rb
+- lib/bluff/support/persistence/active_record.rb
+- lib/bluff/support/persistence/base.rb
 - lib/bluff/support/support.rb
 - lib/bluff/version.rb
+- notes.md
 - spec/models/api_spec.rb
 - spec/models/data_bluffs_spec.rb
 - spec/models/extensions_spec.rb

data/lib/bluff/support/backend.rb

@@ -1,21 +0,0 @@
-module Bluff::Support
-  class Backend
-    @@handlers = []
-    
-    class << self
-      def register(handler)
-        @@handlers << handler
-      end
-      
-      def save!(record, name)
-        handlers = @@handlers.select {|handler| handler.accepts?(record)}
-        
-        if handlers.empty?
-          raise "#{name} cannot be bang bluffed (#{record})"
-        else
-          handlers.first.save!(record)
-        end
-      end
-    end
-  end
-end