reviewed 0.4.6 → 0.5.0

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 204586e11652106dfafc82e5aea75e5d3438a53d
+  data.tar.gz: 9e7819e0a1081293bd3e6aa3cc0fc69b1130c59e
+SHA512:
+  metadata.gz: 80722e2f6171b5f3b4fef91fe1290a32bb38d2fa70a7b233c23c63bc6d9f49ac483a13d1c29dd4fe35e2ede7b24cdf9537f527fe2f4d13eba3edaf4b4b945576
+  data.tar.gz: 343297a543489c280abbf6a8d61d514432fcbb66614f66eac688fb98c4e2bc72852fd14334becd48e9272b7c17070c97953d9772cdc639ee1caaaf33e3cbdcd0

data/.ruby-version

@@ -1 +1 @@
-1.9.3
+2.0.0-p195

data/lib/reviewed/article.rb

@@ -5,7 +5,6 @@ require 'reviewed/deal'
 
 module Reviewed
   class Article < Base
-
     has_attachments
 
     has_many :pages

data/lib/reviewed/attachable.rb

@@ -1,21 +1,28 @@
 module Reviewed
   module Attachable
 
-    def attachments tag, opts={}
-      (@attachments ||= {})[tag] ||= fetch_attachments tag, opts
+    def attachments tag=nil, opts={}
+      if default_attachments.include?(tag.to_s)
+        return attributes['attachments'].select { |x| x.tags.include?(tag.to_s) }
+      else
+        fetch_attachments(opts.merge!(tags: tag)).to_a
+      end
     end
 
     def gallery tags=nil, num=8, page=1
-      fetch_attachments tags, :gallery => true, :per_page => num, :page => page, :order => 'priority'
+      fetch_attachments tags: tags, :gallery => true, :per_page => num, :page => page, :order => 'priority'
     end
 
     private
 
-    def fetch_attachments tag, opts={}
-      params = opts.merge :tags => tag
-      req = Request.new :resource => Attachment, :scope => self
-      req.where params
+    def default_attachments
+      attributes['attachments'] ||= []
+      attributes['attachments'].map(&:tags).flatten.uniq.compact
     end
 
+    def fetch_attachments opts={}
+      req = Request.new :resource => Attachment, :scope => self
+      req.where opts
+    end
   end
 end

data/lib/reviewed/product.rb

@@ -2,7 +2,6 @@ require 'reviewed/attachment'
 
 module Reviewed
   class Product < Base
-
     has_attachments
     has_many :manufacturer_specs
     has_many :awards

data/lib/reviewed/version.rb

@@ -1,4 +1,4 @@
 module Reviewed
-  VERSION = "0.4.6"
+  VERSION = "0.5.0"
   API_VERSION = 'v1'
 end

data/spec/attachable_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe Reviewed::Article, vcr: true do
+
+  let(:client) do
+    Reviewed::Client.new(api_key: TEST_KEY, base_uri: TEST_URL)
+  end
+
+  before(:each) do
+    @article = client.articles.find('50fb9a81bd0286d55504b952')
+  end
+
+  it 'returns local attachments when available' do
+    Reviewed::Request.should_not_receive(:new)
+    @article.attachments('hero').first.tags.should eql(['hero'])
+  end
+
+  it 'fetches when a tag is not in pre-loaded set' do
+    @article.should_receive(:fetch_attachments).with({tags: 'foobar'})
+    @article.attachments('foobar').should eql([])
+  end
+end