spree_api_v2 0.2.0

data/spec/controllers/spree/api/v2/orders_controller_spec.rb

@@ -0,0 +1,15 @@
+describe Spree::Api::V2::OrdersController do
+  routes { Spree::Core::Engine.routes }
+
+  let(:order) { create :order }
+  let(:user) { order.user }
+
+  before { user.generate_spree_api_key! }
+
+  describe '#show' do
+    it 'will respond with the order' do
+      get :show, token: user.spree_api_key, id: order.id
+      expect(JSON.parse(response.body)['data']['type']).to eql 'spree_orders'
+    end
+  end
+end

data/spec/controllers/spree/api/v2/prices_controller_spec.rb

@@ -0,0 +1,55 @@
+describe Spree::Api::V2::PricesController do
+  routes { Spree::Core::Engine.routes }
+
+  let!(:price) { create :price }
+  let(:new_variant) { create :variant }
+
+  describe '#index' do
+    it 'will list all prices' do
+      get :index
+      price_ids = parse_json(response.body)['data'].map do |price|
+        price['id']
+      end
+      expect(price_ids).to include price.id.to_s
+    end
+
+    context 'by variant id' do
+      it 'will list its prices' do
+        get :index, variant_id: price.variant.id
+        price_ids = parse_json(response.body)['data'].map do |price|
+          price['id']
+        end
+        expect(price_ids).to include price.id.to_s
+      end
+
+      it 'will not list another variants prices' do
+        get :index, variant_id: new_variant.id
+        price_ids = parse_json(response.body)['data'].map do |price|
+          price['id']
+        end
+        expect(price_ids).to_not include price.id.to_s
+      end
+    end
+  end
+
+  describe '#show' do
+    it 'will show the price' do
+      get :show, id: price.id
+      price_id = parse_json(response.body)['data']['id']
+      expect(price_id).to eql price.id.to_s
+    end
+
+    context 'by variant id' do
+      it 'will show its prices' do
+        get :show, id: price.id, variant_id: price.variant.id
+        price_id = parse_json(response.body)['data']['id']
+        expect(price_id).to eql price.id.to_s
+      end
+
+      it 'will not show another variants price' do
+        get :show, id: price.id, variant_id: new_variant.id
+        expect(response).to have_http_status 404
+      end
+    end
+  end
+end

data/spec/controllers/spree/api/v2/products_controller_spec.rb

@@ -0,0 +1,95 @@
+describe Spree::Api::V2::ProductsController do
+  routes { Spree::Core::Engine.routes }
+
+  let!(:product) { create :product }
+  let(:taxon) { create :taxon, products: [product] }
+  let(:new_taxon) { create :taxon }
+
+  describe '#index' do
+    it 'will respond with products' do
+      get :index
+      expect(JSON.parse(response.body)['data'].size).to eql 1
+    end
+
+    context 'by taxon id' do
+      it 'will respond with the taxons products' do
+        get :index, taxon_id: taxon.id
+        product_ids = parse_json(response.body)['data'].map do |products|
+          products['id']
+        end
+        expect(product_ids).to include product.id.to_s
+      end
+
+      it 'will not respond with a product that is not associated to the taxon' do
+        get :index, taxon_id: new_taxon.id
+        product_ids = parse_json(response.body)['data'].map do |products|
+          products['id']
+        end
+        expect(product_ids).to_not include product.id.to_s
+      end
+    end
+  end
+
+  describe '#show' do
+    it 'will render the selected product' do
+      get :show, id: product.id
+      expect(parse_json(response.body)['data']['id']).to eql product.id.to_s
+    end
+
+    context 'by taxon id' do
+      it 'will respond with the taxons products' do
+        get :show, taxon_id: taxon.id, id: product.id
+        product_id = parse_json(response.body)['data']['id']
+        expect(product_id).to eql product.id.to_s
+      end
+
+      it 'will not respond with a product that is not associated to the taxon' do
+        get :show, taxon_id: new_taxon.id, id: product.id
+        expect(response).to have_http_status 404
+      end
+    end
+
+    context 'by option type' do
+      let(:option_type) { create :option_type, products: [product] }
+      let(:new_option_type) { create :option_type }
+
+      it 'will find product' do
+        get :show, option_type_id: option_type.id, id: product.id
+        product_id = parse_json(response.body)['data']['id']
+        expect(product_id).to eql product.id.to_s
+      end
+
+      it 'will not find a product belong to a different option type' do
+        get :show, option_type_id: new_option_type.id, id: product.id
+        expect(response).to have_http_status 404
+      end
+    end
+
+    context 'by variant id' do
+      it 'will show its product' do
+        get :show, variant_id: product.master.id
+        product_id = parse_json(response.body)['data']['id']
+        expect(product_id).to eql product.id.to_s
+      end
+    end
+
+    context 'by price id' do
+      it 'will show its product' do
+        get :show, price_id: product.master.prices.first.id
+        product_id = parse_json(response.body)['data']['id']
+        expect(product_id).to eql product.id.to_s
+      end
+    end
+
+    context 'by image id' do
+      let(:image) { create :image }
+
+      it 'will show its product' do
+        product.images << image
+        get :show, image_id: image.id
+        product_id = parse_json(response.body)['data']['id']
+        expect(product_id).to eql product.id.to_s
+      end
+    end
+  end
+end

data/spec/controllers/spree/api/v2/states_controller_spec.rb

@@ -0,0 +1,42 @@
+describe Spree::Api::V2::StatesController do
+  routes { Spree::Core::Engine.routes }
+
+  let!(:state) { create :state }
+
+  describe '#index' do
+    it 'can list all states' do
+      get :index
+      expect(JSON.parse(response.body)['data'].length).to eql 1
+    end
+
+    context 'by country id' do
+      it 'will list selected states by country' do
+        get :index, country_id: state.country.id
+        expect(JSON.parse(response.body)['data'].length).to eql 1
+      end
+
+      it 'will not include states belonging to other countries' do
+        new_state = create :state
+        get :index, country_id: state.country.id
+        state_ids = JSON.parse(response.body)['data'].map do |states|
+          states['id']
+        end
+        expect(state_ids).to_not include new_state
+      end
+    end
+  end
+
+  describe '#show' do
+    it 'will find selected state by id' do
+      get :show, id: state.id
+      state_id = parse_json(response.body)['data']['id']
+      expect(state_id).to eql state.id.to_s
+    end
+
+    it 'can find a state by a country' do
+      get :show, country_id: state.country.id, id: state.id
+      state_id = parse_json(response.body)['data']['id']
+      expect(state_id).to eql state.id.to_s
+    end
+  end
+end

data/spec/controllers/spree/api/v2/taxonomies_controller_spec.rb

@@ -0,0 +1,31 @@
+describe Spree::Api::V2::TaxonomiesController do
+  routes { Spree::Core::Engine.routes }
+
+  let!(:taxonomy) { create :taxonomy }
+
+  describe '#index' do
+    it 'will list taxonomies' do
+      get :index
+      taxonomy_ids = parse_json(response.body)['data'].map do |taxonomy|
+        taxonomy['id']
+      end
+      expect(taxonomy_ids).to include taxonomy.id.to_s
+    end
+  end
+
+  describe '#show' do
+    let(:taxon) { taxonomy.root }
+
+    it 'will show taxonomy' do
+      get :show, id: taxonomy.id
+      taxonomy_id = parse_json(response.body)['data']['id']
+      expect(taxonomy_id).to eql taxonomy.id.to_s
+    end
+
+    it 'can show taxonomy by taxon_id' do
+      get :show, taxon_id: taxon.id
+      taxonomy_id = parse_json(response.body)['data']['id']
+      expect(taxonomy_id).to eql taxonomy.id.to_s
+    end
+  end
+end

data/spec/controllers/spree/api/v2/taxons_controller_spec.rb

@@ -0,0 +1,54 @@
+describe Spree::Api::V2::TaxonsController do
+  routes { Spree::Core::Engine.routes }
+
+  let!(:taxon) { create :taxon }
+
+  describe '#index' do
+    it 'will list the taxons' do
+      get :index
+      taxon_ids = parse_json(response.body)['data'].map do |taxon|
+        taxon['id']
+      end
+      expect(taxon_ids).to include taxon.id.to_s
+    end
+
+    it 'can list the taxons filtered by taxonomy_id' do
+      get :index, taxonomy_id: taxon.taxonomy.id
+      taxon_ids = parse_json(response.body)['data'].map do |taxon|
+        taxon['id']
+      end
+      expect(taxon_ids).to include taxon.id.to_s
+    end
+  end
+
+  describe '#show' do
+    it 'will show the selected taxon' do
+      get :show, id: taxon.id
+      taxon_id = parse_json(response.body)['data']['id']
+      expect(taxon_id).to eql taxon.id.to_s
+    end
+
+    context 'by parent' do
+      it 'can show a taxons parent' do
+        child = create :taxon, parent: taxon
+        get :show, taxon_id: child.id
+        taxon_id = parse_json(response.body)['data']['id']
+        expect(taxon_id).to eql taxon.id.to_s
+      end
+    end
+
+    context 'by taxonomy_id' do
+      it 'will fetch the selected taxon' do
+        get :show, taxonomy_id: taxon.taxonomy.id, id: taxon.id
+        taxon_id = parse_json(response.body)['data']['id']
+        expect(taxon_id).to eql taxon.id.to_s
+      end
+
+      it 'when taxon does not belong to taxonomy' do
+        new_taxonomy = create :taxonomy
+        get :show, taxonomy_id: new_taxonomy.id, id: taxon.id
+        expect(response).to have_http_status 404
+      end
+    end
+  end
+end

data/spec/controllers/spree/api/v2/variants_controller_spec.rb

@@ -0,0 +1,108 @@
+describe Spree::Api::V2::VariantsController do
+  routes { Spree::Core::Engine.routes }
+
+  let!(:variant) { create :variant }
+  let(:new_option_value) { create :option_value }
+
+  describe '#index' do
+    it 'will list all variants' do
+      get :index
+      variant_ids = parse_json(response.body)['data'].map do |variant|
+        variant['id']
+      end
+      expect(variant_ids).to include variant.id.to_s
+    end
+
+    context 'by product id' do
+      let(:new_product) { create :product }
+
+      it 'will list its variants' do
+        get :index, product_id: variant.product.id
+        variant_ids = parse_json(response.body)['data'].map do |variant|
+          variant['id']
+        end
+        expect(variant_ids).to include variant.id.to_s
+      end
+
+      it 'will not list another products variants' do
+        get :index, product_id: new_product.id
+        variant_ids = parse_json(response.body)['data'].map do |variant|
+          variant['id']
+        end
+        expect(variant_ids).to_not include variant.id.to_s
+      end
+    end
+
+    context 'by option value' do
+      it 'will list its variants' do
+        get :index, option_value_id: variant.option_values.first.id
+        variant_ids = parse_json(response.body)['data'].map do |variant|
+          variant['id']
+        end
+        expect(variant_ids).to include variant.id.to_s
+      end
+
+      it 'will not list another products variants' do
+        get :index, option_value_id: new_option_value.id
+        variant_ids = parse_json(response.body)['data'].map do |variant|
+          variant['id']
+        end
+        expect(variant_ids).to_not include variant.id.to_s
+      end
+    end
+  end
+
+  describe '#show' do
+    it 'will show the variant' do
+      get :show, id: variant.id
+      variant_id = parse_json(response.body)['data']['id']
+      expect(variant_id).to eql variant.id.to_s
+    end
+
+    context 'by product id' do
+      let(:new_product) { create :product }
+
+      it 'will show its variant' do
+        get :show, id: variant.id, product_id: variant.product.id
+        variant_id = parse_json(response.body)['data']['id']
+        expect(variant_id).to eql variant.id.to_s
+      end
+
+      it 'will not show another products variant' do
+        get :show, id: variant.id, product_id: new_product.id
+        expect(response).to have_http_status 404
+      end
+    end
+
+    context 'by option_value id' do
+      it 'will show its variant' do
+        get :show, id: variant.id, option_value_id: variant.option_values.first.id
+        variant_id = parse_json(response.body)['data']['id']
+        expect(variant_id).to eql variant.id.to_s
+      end
+
+      it 'will not show another option_values variant' do
+        get :show, id: variant.id, option_value_id: new_option_value.id
+        expect(response).to have_http_status 404
+      end
+    end
+
+    context 'by price id' do
+      it 'will show its variant' do
+        get :show, price_id: variant.prices.first.id
+        variant_id = parse_json(response.body)['data']['id']
+        expect(variant_id).to eql variant.id.to_s
+      end
+    end
+
+    context 'by image id' do
+      let(:image) { create :image, viewable: variant }
+
+      it 'will show its variant' do
+        get :show, image_id: image.id
+        variant_id = parse_json(response.body)['data']['id']
+        expect(variant_id).to eql variant.id.to_s
+      end
+    end
+  end
+end

data/spec/models/spree/base_decorator_spec.rb

@@ -0,0 +1,9 @@
+describe Spree::Base do
+  it '.paginate' do
+    # The pagination is concerned onto abstract models. There's not really a way
+    # to be able to ensure that the model is paginated, so instead of ensuring
+    # that the result is paginated (tested in the Spree::Api::BaseController),
+    # this just makes sure it can respond to the +.paginate+ method.
+    expect(described_class).to respond_to(:paginate).with(1).argument
+  end
+end

data/spec/models/spree/price_decorator_spec.rb

@@ -0,0 +1,3 @@
+describe Spree::Price do
+  it { is_expected.to delegate_method(:product).to :variant }
+end

data/spec/serializers/spree/address_serializer_spec.rb

@@ -0,0 +1,35 @@
+describe Spree::AddressSerializer do
+  let(:address) { create :address }
+  subject { ActiveModel::Serializer::Adapter::JsonApi.new(described_class.new address) }
+
+  its(:to_json) do
+    is_expected.to be_json_eql <<-JSON
+      {
+        "data" : {
+          "attributes" : {
+            "address1" : "#{address.address1}",
+            "address2" : "#{address.address2}",
+            "city" : "#{address.city}",
+            "first_name" : "#{address.first_name}",
+            "last_name" : "#{address.last_name}",
+            "phone" : "#{address.phone}",
+            "zipcode" : "#{address.zipcode}"
+          },
+          "relationships" : {
+            "country" : {
+              "data" : {
+                "type" : "spree_countries"
+              }
+            },
+            "state" : {
+              "data" : {
+                "type" : "spree_states"
+              }
+            }
+          },
+          "type" : "spree_addresses"
+        }
+      }
+    JSON
+  end
+end