bricks 0.4.1 → 0.5.0

data/History.md

@@ -1,6 +1,12 @@
 History
 =======
 
+0.5.0
+-----
+
+* Now includes generalized hook framework, but only supports after(:clone) hook.
+* Ask for an attribute that hasn't been initialized yet, and it will be.
+
 0.4.1
 -----
 

data/README.md

@@ -18,6 +18,7 @@ We'll use the following domain to describe *Brick's* features:
     #  title           :string(255)
     #  author          :string(255)
     #  formatted_title :string(510)
+    #  popularity      :integer(4)
     #  publication_id  :integer(4)
     #
     class Article < ActiveRecord::Base
@@ -244,6 +245,30 @@ Note that if you want to override a *-to-many association inside a trait, you ne
 
 For an executable version of this documentation, please see spec/bricks_spec.rb.
 
+### Hooks
+
+*Bricks includes a simple, general hook framework. It allows you to do something like this:
+
+    builder Article
+      # ...
+
+      trait :on_the_bugle do
+        publication.name "The Daily Bugle"
+        popularity       75
+      end
+
+      trait :on_the_planet do
+        publication.name "The Daily Planet"
+        popularity       85
+      end
+
+      after :clone do
+        send %w(on_the_bugle on_the_planet)[rand(2)]
+      end
+    end
+
+*Bricks* supports a single hook right now: after(:clone). It will be executed whenever you use any of #build, #build!, #create or #create!, right before you start customizing the resulting builder on your test.
+
 Installation
 ------------
 

data/VERSION

@@ -1 +1 @@
-0.4.1
+0.5.0

data/bricks.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{bricks}
-  s.version = "0.4.1"
+  s.version = "0.5.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["David Leal"]
-  s.date = %q{2011-06-23}
+  s.date = %q{2011-06-29}
   s.email = %q{david@mojotech.com}
   s.extra_rdoc_files = [
     "LICENSE.txt",

data/lib/bricks/builder.rb

@@ -23,6 +23,12 @@ module Bricks
       self
     end
 
+    def after(hook, &block)
+      @traits.class_eval do
+        define_method "__after_#{hook}", &block
+      end
+    end
+
     def derive(args = {})
       build_attrs
 
@@ -30,7 +36,9 @@ module Bricks
       save   = args.has_key?(:save) ? args[:save] : @save
       search = args.has_key?(:search) ? args[:search] : @search
 
-      Builder.new(klass, @attrs, @traits, save, search)
+      Builder.new(klass, @attrs, @traits, save, search).tap { |b|
+        b.run_hook :after, :clone if ! args[:class]
+      }
     end
 
     def initialize(
@@ -80,7 +88,8 @@ module Bricks
                elsif settable?(attr)
                  set attr, *args, &block
                else
-                 raise Bricks::NoAttributeOrTrait, "Can't find `#{name}'."
+                 raise Bricks::NoAttributeOrTrait,
+                       "Can't find `#{name}' on builder for #{@class}."
                end
 
       if return_object
@@ -93,6 +102,14 @@ module Bricks
       end
     end
 
+    protected
+
+    def run_hook(position, name)
+      full_name = "__#{position}_#{name}"
+
+      send full_name if respond_to?(full_name)
+    end
+
     private
 
     def subject
@@ -113,23 +130,53 @@ module Bricks
       obj.save!
     end
 
-    def initialize_object(parent)
-      obj = @class.new
+    class Proxy
+      attr_reader :obj
+
+      def initialize(obj, attrs, parent)
+        @obj       = obj
+        @attrs_in  = attrs.dup
+        @attrs_out = {}
+        @parent    = parent
+      end
 
-      @attrs.each { |(k, v)|
-        val = case v
+      def method_missing(name, *args)
+        name_y = name.to_sym
+
+        if @attrs_in.assoc(name_y) && ! @attrs_out.has_key?(name_y)
+          fix_attr name
+        else
+          @obj.send name, *args
+        end
+      end
+
+      def build
+        @attrs_in.each { |(k, _)| send k }
+
+        @obj
+      end
+
+      def fix_attr(name)
+        val = case v = @attrs_in.assoc(name).last
               when Proc
-                v.call *[obj, parent].take([v.arity, 0].max)
+                case r = v.call(*[self, @parent].take([v.arity, 0].max))
+                when Proxy
+                  r.obj
+                else
+                  r
+                end
               when Builder, BuilderSet
-                v.generate(:parent => obj)
+                v.generate(:parent => self)
               else
                 v
               end
 
-        obj.send "#{k}=", val
-      }
+        @attrs_out[name] = @obj.send("#{name}=", val)
+      end
+    end
 
-      obj
+    def initialize_object(parent)
+      Proxy.new(@class.new, @attrs, parent).build
     end
 
     def settable?(name)

data/spec/bricks/builder_spec.rb

@@ -11,7 +11,7 @@ describe Bricks::Builder do
     }.new
 
     class Person
-      attr_accessor :name
+      attr_accessor :name, :first_name, :last_name
     end
   end
 
@@ -38,4 +38,19 @@ describe Bricks::Builder do
 
     b.generate.object_id.should_not == b.generate.object_id
   end
+
+  describe "attribute evaluation ordering" do
+    before :all do
+    end
+
+    it "doesn't care which order the attributes are declared" do
+      b = Bricks::Builder.new Person do
+        name { |obj| obj.first_name + " " + obj.last_name }
+        first_name { "Jack" }
+        last_name { "Black" }
+      end
+
+      b.derive.generate.name.should == "Jack Black"
+    end
+  end
 end

data/spec/bricks_spec.rb

@@ -44,6 +44,10 @@ describe Bricks do
 
           %w(Tom Dick Harry).each { |n| readers.name(n) }
         end
+
+        after :clone do
+          popularity (1..100).to_a[rand(100)]
+        end
       end
 
       builder Newspaper do
@@ -217,5 +221,15 @@ describe Bricks do
       build!(Reader).should be_a(Reader)
     end
   end
+
+  describe "hooks" do
+    it "executes the `generate' hook after a builder is cloned" do
+      build!(Article).popularity.should_not == build!(Article).popularity
+    end
+
+    it "it does not override values set after the builder is cloned" do
+      build(Article).popularity!(50).popularity.should == 50
+    end
+  end
 end
 

data/spec/support/active_record.rb

@@ -17,6 +17,7 @@ ActiveRecord::Schema.define(:version => 20110608204150) do
     t.string   "language"
     t.integer  "newspaper_id"
     t.string   "title"
+    t.integer  "popularity"
   end
 
   create_table "newspapers", :force => true do |t|

metadata

@@ -2,7 +2,7 @@
 name: bricks
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.4.1
+  version: 0.5.0
 platform: ruby
 authors: 
 - David Leal
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-06-23 00:00:00 +01:00
+date: 2011-06-29 00:00:00 +01:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -101,7 +101,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: -949772447
+      hash: -614702947
       segments: 
       - 0
       version: "0"