Merge pull request zendesk#125 from zendesk/sdavidovitz/build

add a Collection#build method

Author: steved

lib/zendesk_api/collection.rb

@@ -56,7 +56,28 @@ def initialize(client, resource, options = {})
  opts = args.last.is_a?(Hash) ? args.pop : {}
  opts.merge!(:association => @association)
 
- @resource_class.send(deferrable, @client, @options.merge(opts))
+ @resource_class.send(deferrable, @client, opts)
+ end
+ end
+
+ # Convenience method to build a new resource and
+ # add it to the collection. Fetches the collection as well.
+ # @param [Hash] options Options or attributes to pass
+ def build(opts = {})
+ wrap_resource(opts, true).tap do |res|
+ self << res
+ end
+ end
+
+ # Convenience method to build a new resource and
+ # add it to the collection. Fetches the collection as well.
+ # @param [Hash] options Options or attributes to pass
+ def build!(opts = {})
+ wrap_resource(opts, true).tap do |res|
+ fetch!
+
+ # << does a fetch too
+ self << res
  end
  end
 
@@ -119,6 +140,7 @@ def include(*sideloads)
  # @raise [ArgumentError] if the resource doesn't belong in this collection
  def <<(item)
  fetch
+
  if item.is_a?(Resource)
  if item.is_a?(@resource_class)
  @resources << item

spec/core/collection_spec.rb

@@ -98,6 +98,18 @@
  it "should pass association" do
  subject.last.association.should == association
  end
+
+ it "should #build a resource and add it" do
+ resource = subject.build
+ subject.should include(resource)
+ resource.association.should == subject.association
+ end
+
+ it "should #build! a resource and add it" do
+ resource = subject.build!
+ subject.should include(resource)
+ resource.association.should == subject.association
+ end
  end
  end
  end