bol 0.0.1.beta

data/lib/bol/version.rb

@@ -0,0 +1,3 @@
+module Bol
+  VERSION = '0.0.1.beta'
+end

data/spec/bol/configuration_spec.rb

@@ -0,0 +1,46 @@
+require 'spec_helper'
+
+describe Bol::Configuration do
+  describe 'as a hash' do
+    it 'should should set known keys' do
+      c = Bol::Configuration.new access_key: 'foo'
+      c[:access_key].should == 'foo'
+    end
+
+    it 'should raise for unknown keys' do
+      expect { Bol::Configuration.new foo: 'bar' }.to raise_error ArgumentError
+    end
+
+    it 'should set a single key' do
+      c = Bol::Configuration.new
+      c[:access_key] = 'bar'
+      c[:access_key].should == 'bar'
+    end
+  end
+
+  describe 'validation' do
+    it 'should raise error when not properly configured' do
+      c = Bol::Configuration.new
+      expect { c.validate }.to raise_error(Bol::ConfigurationError)
+    end
+
+    it 'should not raise error when all keys are configured' do
+      c = Bol::Configuration.new access_key: 'foo', secret: 'bar'
+      expect { c.validate }.to_not raise_error(Bol::ConfigurationError)
+    end
+  end
+
+  describe 'as attributes' do
+    let(:config) { Bol::Configuration.new }
+
+    it 'should should set known keys' do
+      config.access_key = 'bar'
+      config.access_key.should == 'bar'
+    end
+
+    it 'should raise for unknown keys' do
+      expect { config.foo = 'bar' }.to raise_error NoMethodError
+      expect { config.foo }.to raise_error NoMethodError
+    end
+  end
+end

data/spec/bol/parsers/products_spec.rb

@@ -0,0 +1,77 @@
+require 'spec_helper'
+
+describe Bol::Parsers::Products do
+  let(:request) do
+    double('request').tap do |r|
+      r.stub(:fetch).and_return(r)
+      r.stub(:body).and_return(body)
+    end
+  end
+  let(:product) { Bol::Parsers::Products.new(request).objects.first }
+
+  describe 'parsing products' do
+    let(:body) { fixture('products.xml') }
+
+    it 'should set simple attributes' do
+      product.id.should == '1001004006016448'
+      product.title.should == 'Harry Potter Boxed Set (Adult Edition)'
+      product.subtitle.should == 'Volume 1 - 7, paperback'
+      product.publisher.should == 'Bloomsbury Publishing PLC'
+      product.ean.should == '9780747595847'
+      product.attributes[:binding_description].should == 'Paperback'
+      product.language_code.should == 'en'
+      product.language_description.should == 'Engels'
+      product.url.should == 'http://www.bol.com/nl/p/engelse-boeken/harry-potter-boxed-set/1001004006016448/index.html'
+      product.rating.should == '5'
+      product.authors.should include 'J. K. Rowling'
+      product.authors.should include 'J.K. Rowling'
+    end
+
+    it 'should parse release date' do
+      product.release_date.year.should == 2008
+      product.release_date.month.should == 10
+      product.release_date.day.should == 1
+    end
+
+    it 'should parse cover' do
+      product.cover.should == 'http://s-bol.com/imgbase0/imagebase/thumb/FC/8/4/4/6/1001004006016448.jpg'
+      product.cover(:medium).should == 'http://s-bol.com/imgbase0/imagebase/thumb/FC/8/4/4/6/1001004006016448.jpg'
+      product.cover(:extra_small).should == 'http://s-bol.com/imgbase0/imagebase/mini/FC/8/4/4/6/1001004006016448.jpg'
+      product.cover(:small).should == 'http://s-bol.com/imgbase0/imagebase/tout/FC/8/4/4/6/1001004006016448.jpg'
+      product.cover(:large).should == 'http://s-bol.com/imgbase0/imagebase/regular/FC/8/4/4/6/1001004006016448.jpg'
+      product.cover(:extra_large).should == 'http://s-bol.com/imgbase0/imagebase/large/FC/8/4/4/6/1001004006016448.jpg'
+    end
+  end
+
+  describe 'parsing product search results' do
+    let(:body) { fixture('searchproducts-music.xml') }
+    let(:products) { Bol::Parsers::Products.new(request).objects }
+
+    it 'should parse into array of products' do
+      products.size.should == 2
+      products[0].should be_instance_of Bol::Product
+      products[1].should be_instance_of Bol::Product
+    end
+
+    it 'should have parsed xml into products' do
+      products[0].title.should == 'Glass: Violin Concerto, Company etc / Adele Anthony, Takuo Yuasa et al'
+      products[1].title.should == 'Adele'
+    end
+  end
+
+  describe 'parsing product listings' do
+    let(:body) { fixture('productlists.xml') }
+    let(:products) { Bol::Parsers::Products.new(request).objects }
+
+    it 'should parse into array of products' do
+      products.size.should == 2
+      products[0].should be_instance_of Bol::Product
+      products[1].should be_instance_of Bol::Product
+    end
+
+    it 'should have parsed xml into products' do
+      products[0].title.should == 'In mijn dromen'
+      products[1].title.should == 'Kort'
+    end
+  end
+end

data/spec/bol/product_spec.rb

@@ -0,0 +1,71 @@
+require 'spec_helper'
+
+describe Bol::Product do
+  describe '#find' do
+    let(:r) { Bol::Product.find(1) }
+
+    before do
+      Bol.configure access_key: 'foo', secret: 'bar'
+      FakeWeb.register_uri(:get, 'https://openapi.bol.com/openapi/services/rest/catalog/v3/products/1?categoryId=0', body: fixture('products.xml'))
+    end
+
+    it 'should return product instance' do
+      r.should be_instance_of Bol::Product
+    end
+
+    it 'should raise error when not found' do
+      FakeWeb.register_uri(:get, %r{https://openapi.bol.com}, status: ['404', 'Not found'])
+      expect { Bol::Product.find(999) }.to raise_error(Bol::NotFound)
+    end
+  end
+
+  describe 'attributes' do
+    it 'should delegate [] to attributes' do
+      product = Bol::Product.new
+      product.attributes[:foo] = 'bar'
+      product[:foo].should == 'bar'
+    end
+
+    it 'should expose attributes as methods' do
+      product = Bol::Product.new
+      product.attributes[:foo] = 'bar'
+      product.foo.should == 'bar'
+    end
+
+    describe 'referral_url' do
+      let(:product) { Bol::Product.new }
+
+      before do
+        product.attributes[:url]   = 'http://foo.bar'
+        product.attributes[:id]    = 'qux'
+        product.attributes[:title] = 'bla'
+      end
+
+      it 'should generate referral URL' do
+        product.referral_url('foo').should == "http://partnerprogramma.bol.com/click/click?p=1&t=url&s=foo&url=http%3A%2F%2Ffoo.bar&f=API&subid=qux&name=bla"
+      end
+    end
+
+    describe '#cover' do
+      let(:product) { Bol::Product.new }
+      before do
+        product.attributes[:cover] = {
+          medium: 'foo',
+          small: 'bar'
+        }
+      end
+
+      it 'should return medium by default' do
+        product.cover.should == 'foo'
+      end
+
+      it 'should take format as argument' do
+        product.cover(:small).should == 'bar'
+      end
+
+      it 'should raise on invalid format' do
+        expect { product.cover(:baz) }.to raise_error KeyError
+      end
+    end
+  end
+end

data/spec/bol/proxy_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+
+describe Bol::Proxy do
+  let(:parser)  { double('parser') }
+  let(:query)   { double('query', category_id: 1) }
+  let(:request) { double('request', query: query) }
+  let(:subject) { described_class.new(request, parser) }
+
+  it 'should delegate methods to query' do
+    subject.category_id.should == 1
+  end
+
+  it 'should delegate query' do
+    subject.query.should == query
+  end
+
+  it 'should delegate query methods' do
+    query.should_receive(:offset).with(5)
+    query.should_receive(:limit).with(10)
+    subject.offset(5)
+    subject.limit(10)
+  end
+
+  it 'should get objects from parser' do
+    parser.should_receive(:objects).and_return([])
+    subject.all
+    subject.all
+  end
+
+  it 'should get objects from parser when looping' do
+    parser.should_receive(:objects).and_return([])
+    subject.each {}
+  end
+end

data/spec/bol/query_spec.rb

@@ -0,0 +1,129 @@
+require 'spec_helper'
+
+describe Bol::Query do
+  let(:subject) { Bol::Query.new(0) }
+
+  describe 'category scope' do
+    it 'should set category scope' do
+      Bol::Query.new(1).category_id.should == 1
+    end
+
+    it 'should raise unless given a number' do
+      expect { Bol::Query.new('foo') }.to raise_error ArgumentError
+    end
+
+    it 'should set category param' do
+      Bol::Query.new(1).params[:categoryId].should == 1
+    end
+
+    it 'should accept multiple category IDs' do
+      expect { Bol::Query.new('1+2+3') }.to_not raise_error
+    end
+  end
+
+  describe '#has_param?' do
+    it { should have_param(:categoryId) }
+    it { should_not have_param(:term) }
+    it { should_not have_param(:foo) }
+  end
+
+  describe '#limit' do
+    context 'when set' do
+      let(:subject) { Bol::Query.new(0).limit('10') }
+      its(:limit) { should == 10 }
+      it { should have_param(:nrProducts) }
+    end
+
+    context "when not set" do
+      it { should_not have_param(:nrProducts) }
+    end
+  end
+
+  describe '#page' do
+    context 'page 2' do
+      let(:subject) { Bol::Query.new(0).page(2) }
+      it { should have_param(:nrProducts) }
+      it { should have_param(:offset) }
+      its(:limit) { should == 10 }
+      its(:offset) { should == 10 }
+    end
+
+    context 'defaults' do
+      let(:subject) { Bol::Query.new(0).page(nil) }
+      it { should have_param(:nrProducts) }
+      it { should have_param(:offset) }
+      its(:limit) { should == 10 }
+      its(:offset) { should == 0 }
+    end
+
+    context '20 per page' do
+      before { Bol.configuration[:per_page] = 20 }
+      after { Bol.configuration[:per_page] = 10 }
+      let(:subject) { Bol::Query.new(0).page(3) }
+      it { should have_param(:nrProducts) }
+      it { should have_param(:offset) }
+      its(:limit) { should == 20 }
+      its(:offset) { should == 40 }
+    end
+  end
+
+  describe '#offset' do
+    context 'when set' do
+      let(:subject) { Bol::Query.new(0).offset('10') }
+      its(:offset) { should == 10 }
+      it { should have_param(:offset) }
+    end
+
+    context 'when not set' do
+      it { should_not have_param(:offset) }
+    end
+  end
+
+  describe '#search' do
+    context 'when set' do
+      let(:subject) { Bol::Query.new(0).search('foo') }
+      it { should have_param(:term) }
+      its(:search) { should == 'foo' }
+    end
+
+    context 'when not set' do
+      it { should_not have_param(:term) }
+    end
+  end
+
+  describe '#order' do
+    it 'should raise unless wrong format' do
+      expect { subject.order('foo') }.to raise_error ArgumentError
+    end
+
+    it 'should add order direction on request' do
+      subject.order('price ASC').params[:sortingAscending].should == 'true'
+      subject.order('price DESC').params[:sortingAscending].should == 'false'
+    end
+
+    it 'should add order key on request' do
+      subject.order('price ASC').params[:sortingMethod].should == 'price'
+    end
+
+    it 'should return query' do
+      subject.order('price ASC').should == subject
+    end
+  end
+
+  describe "query methods" do
+    it 'should include categories' do
+      subject.include :categories
+      subject.should be_categories
+    end
+
+    it 'should include categories' do
+      subject.include :categories
+      subject.should be_categories
+    end
+
+    it 'should include categories' do
+      subject.include :categories
+      subject.should be_categories
+    end
+  end
+end

data/spec/bol/request_spec.rb

@@ -0,0 +1,119 @@
+require 'spec_helper'
+
+describe Bol::Request do
+  let(:query)   { Bol::Query.new(1).limit(3) }
+  let(:request) { Bol::Request.new(query) }
+
+  it 'should require a query' do
+    expect { Bol::Request.new }.to raise_error ArgumentError
+  end
+
+  describe 'query filtering' do
+    it 'should list all params in a hash' do
+      request.params.should == {
+        includeCategories: false,
+        includeProducts: false,
+        includeRefinements: false,
+        categoryId: 1,
+        nrProducts: 3
+      }
+    end
+
+    it 'should not list ignored params' do
+      Bol::Requests::Product.new(1, query).params.should == {
+        categoryId: 1,
+        nrProducts: 3
+      }
+    end
+
+    it 'should convert query to params' do
+      request.query_string.should == "categoryId=1&nrProducts=3&includeCategories=false&includeProducts=false&includeRefinements=false"
+    end
+  end
+
+  describe 'signing and headers' do
+    it 'should raise error when key and secret are not configured' do
+      expect { request.fetch }.should raise_error Bol::ConfigurationError
+    end
+
+    it 'should request application/xml' do
+      Bol.configure access_key: 'foo', secret: 'bar'
+      Net::HTTP.any_instance.should_receive(:request).with do |req|
+        req['Content-Type'] == 'application/xml'
+      end.and_return(double(code: 200))
+      request.fetch
+    end
+
+    it 'should set special date header' do
+      Bol.configure access_key: 'foo', secret: 'bar'
+      Net::HTTP.any_instance.should_receive(:request).with do |req|
+        !req['X-OpenAPI-Date'].nil?
+      end.and_return(double(code: 200))
+      request.fetch
+    end
+
+    it 'should add signature to request by default' do
+      Bol.configure access_key: 'foo', secret: 'bar'
+      Net::HTTP.any_instance.should_receive(:request).with do |req|
+        !req['X-OpenAPI-Authorization'].nil?
+      end.and_return(double(code: 200))
+      request.fetch
+    end
+  end
+
+  it 'should trigger event manually' do
+    Bol.configure access_key: 'foo', secret: 'bar'
+    Net::HTTP.any_instance.should_receive(:request).and_return(double(code: 200))
+    request.fetch
+  end
+
+  describe Bol::Requests::Product do
+    let(:request) { Bol::Requests::Product.new(1, query) }
+
+    it 'should get from correct URL' do
+      request.uri.to_s.should == 'https://openapi.bol.com/openapi/services/rest/catalog/v3/products/1?categoryId=1&nrProducts=3'
+    end
+
+    it 'should require product id as term' do
+      expect { Bol::Requests::Product.new(query) }.to raise_error ArgumentError
+    end
+
+    it 'should URL encode the terms' do
+      Bol::Requests::Product.new('foo bar', query).uri.to_s.should ==
+        'https://openapi.bol.com/openapi/services/rest/catalog/v3/products/foo%20bar?categoryId=1&nrProducts=3'
+    end
+  end
+
+  describe Bol::Requests::Search do
+    let(:query)   { Bol::Query.new(1).limit(3).search('foo') }
+    let(:request) { Bol::Requests::Search.new(query) }
+
+    it 'should get from correct URL' do
+      request.uri.to_s.should ==
+        'https://openapi.bol.com/openapi/services/rest/catalog/v3/searchresults?categoryId=1&nrProducts=3&term=foo'
+    end
+
+    it 'should require search terms' do
+      expect { Bol::Requests::Product.new(Bol::Query.new(0)) }.to raise_error ArgumentError
+    end
+
+    it 'should encode the search terms' do
+      query = Bol::Query.new(1).limit(3).search('foo bar')
+      Bol::Requests::Search.new(query).uri.to_s.should ==
+        'https://openapi.bol.com/openapi/services/rest/catalog/v3/searchresults?categoryId=1&nrProducts=3&term=foo%20bar'
+    end
+  end
+
+  describe Bol::Requests::List do
+    let(:query)   { Bol::Query.new(0) }
+    let(:request) { Bol::Requests::List.new('foo', query) }
+
+    it 'should get from correct URL' do
+      request.uri.to_s.should == 'https://openapi.bol.com/openapi/services/rest/catalog/v3/listresults/foo/0?categoryId=0&includeCategories=false&includeProducts=false&includeRefinements=false'
+    end
+
+    it 'should require param type' do
+      expect { Bol::Requests::Product.new(Bol::Query.new(0)) }.to raise_error ArgumentError
+    end
+  end
+end