spree_api 2.4.0 → 2.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2bebe4795d7751605292a86a15274c44a7c09ffc
-  data.tar.gz: b062e7c4ab428d02d25ffdfaf5f18b25d046ddf1
+  metadata.gz: 3736aa7fa433082c6e7771f2e1850ff8f7e14a76
+  data.tar.gz: 51e229c7ed5a6d30f00ad17aa48ec58d8dc8a5b8
 SHA512:
-  metadata.gz: 4f5db8d5263f27997792eb1e719df569caaa66cbb92f9a315df7fe545ffaffad417b1398d9f0051e2697dbe07f32edd1912fd16db16a3ed7e2ee7a992afb032c
-  data.tar.gz: 7570289fbe966a629e81c814e18c0a27205c25176e74df4b54794b2217041118d6cc3162f3d3e37797007ce4a72458f85464a72166666dd9eb1cacdc84d7df3e
+  metadata.gz: e9d188802236b8f7c836f98df8183b028dc4de465080f0382b5119f1017bf11dd1735b43b9048773f10aea96bb95f3474ce86c93f12a736ccbb0068e597d47e2
+  data.tar.gz: a2e7952731ee46c9896bc562d74926abeb370a80adb8f58f446a8ebfdd7d5d4dd9300e60eda4e536c22db35f844126c0921f716a5b83e0d76594c0b97d1ce165

data/app/controllers/spree/api/stores_controller.rb

@@ -0,0 +1,55 @@
+module Spree
+  module Api
+    class StoresController < Spree::Api::BaseController
+
+      before_filter :get_store, except: [:index, :create]
+
+      def index
+        authorize! :read, Store
+        @stores = Store.accessible_by(current_ability, :read).all
+        respond_with(@stores)
+      end
+
+      def create
+        authorize! :create, Store
+        @store = Store.new(store_params)
+        @store.code = params[:store][:code]
+        if @store.save
+          respond_with(@store, status: 201, default_template: :show)
+        else
+          invalid_resource!(@store)
+        end
+      end
+
+      def update
+        authorize! :update, @store
+        if @store.update_attributes(store_params)
+          respond_with(@store, status: 200, default_template: :show)
+        else
+          invalid_resource!(@store)
+        end
+      end
+
+      def show
+        authorize! :read, @store
+        respond_with(@store)
+      end
+
+      def destroy
+        authorize! :destroy, @store
+        @store.destroy
+        respond_with(@store, status: 204)
+      end
+
+      private
+
+      def get_store
+        @store = Store.find(params[:id])
+      end
+
+      def store_params
+        params.require(:store).permit(permitted_store_attributes)
+      end
+    end
+  end
+end

data/app/helpers/spree/api/api_helpers.rb

@@ -28,7 +28,8 @@ module Spree
         :stock_location_attributes,
         :stock_movement_attributes,
         :stock_item_attributes,
-        :promotion_attributes
+        :promotion_attributes,
+        :store_attributes
       ]
 
       mattr_reader *ATTRIBUTES
@@ -150,7 +151,13 @@ module Spree
       ]
 
       @@promotion_attributes = [
-         :id, :name, :description, :expires_at, :starts_at, :type, :usage_limit, :match_policy, :code, :advertise, :path
+        :id, :name, :description, :expires_at, :starts_at, :type, :usage_limit,
+        :match_policy, :code, :advertise, :path
+      ]
+
+      @@store_attributes = [
+        :id, :name, :url, :meta_description, :meta_keywords, :seo_title,
+        :mail_from_address, :default_currency, :code, :default
       ]
 
       def variant_attributes

data/app/views/spree/api/stores/index.v1.rabl

@@ -0,0 +1,4 @@
+object false
+child(@stores => :stores) do
+  attributes *store_attributes
+end

data/app/views/spree/api/stores/show.v1.rabl

@@ -0,0 +1,2 @@
+object @store
+attributes *store_attributes

data/config/routes.rb

@@ -112,6 +112,8 @@ Spree::Core::Engine.add_routes do
       resources :stock_items
     end
 
+    resources :stores
+
     get '/config/money', to: 'config#money'
     get '/config', to: 'config#show'
 

data/spec/controllers/spree/api/stores_controller_spec.rb

@@ -0,0 +1,133 @@
+require "spec_helper"
+
+module Spree
+  describe Api::StoresController, type: :controller do
+    render_views
+
+    let!(:store) do
+      create(:store, name: "My Spree Store", url: "spreestore.example.com")
+    end
+
+    before do
+      stub_authentication!
+    end
+
+    context "as an admin" do
+      sign_in_as_admin!
+
+      let!(:non_default_store) do
+        create(:store,
+          name: "Extra Store",
+          url: "spreestore-5.example.com",
+          default: false
+        )
+      end
+
+      it "I can list the available stores" do
+        api_get :index
+        expect(json_response["stores"]).to eq([
+          {
+            "id" => store.id,
+            "name" => "My Spree Store",
+            "url" => "spreestore.example.com",
+            "meta_description" => nil,
+            "meta_keywords" => nil,
+            "seo_title" => nil,
+            "mail_from_address" => "spree@example.org",
+            "default_currency" => nil,
+            "code" => "spree_1",
+            "default" => true
+          },
+          {
+            "id" => non_default_store.id,
+            "name" => "Extra Store",
+            "url" => "spreestore-5.example.com",
+            "meta_description" => nil,
+            "meta_keywords" => nil,
+            "seo_title" => nil,
+            "mail_from_address" => "spree@example.org",
+            "default_currency" => nil,
+            "code" => "spree_2",
+            "default" => false
+          }
+        ])
+      end
+
+      it "I can get the store details" do
+        api_get :show, id: store.id
+        expect(json_response).to eq(
+          "id" => store.id,
+          "name" => "My Spree Store",
+          "url" => "spreestore.example.com",
+          "meta_description" => nil,
+          "meta_keywords" => nil,
+          "seo_title" => nil,
+          "mail_from_address" => "spree@example.org",
+          "default_currency" => nil,
+          "code" => store.code,
+          "default" => true
+        )
+      end
+
+      it "I can create a new store" do
+        store_hash = {
+          code: "spree123",
+          name: "Hack0rz",
+          url: "spree123.example.com",
+          mail_from_address: "me@example.com"
+        }
+        api_post :create, store: store_hash
+        expect(response.status).to eq(201)
+      end
+
+      it "I can update an existing store" do
+        store_hash = {
+          url: "spree123.example.com",
+          mail_from_address: "me@example.com"
+        }
+        api_put :update, id: store.id, store: store_hash
+        expect(response.status).to eq(200)
+        expect(store.reload.url).to eql "spree123.example.com"
+        expect(store.reload.mail_from_address).to eql "me@example.com"
+      end
+
+      context "deleting a store" do
+        it "will fail if it's the default Store" do
+          api_delete :destroy, id: store.id
+          expect(response.status).to eq(422)
+          expect(json_response["errors"]["base"]).to eql(
+          ["Cannot destroy the default Store."]
+          )
+        end
+
+        it "will destroy the store" do
+          api_delete :destroy, id: non_default_store.id
+          expect(response.status).to eq(204)
+        end
+      end
+    end
+
+    context "as an user" do
+
+      it "I cannot list all the stores" do
+        api_get :index
+        expect(response.status).to eq(401)
+      end
+
+      it "I cannot get the store details" do
+        api_get :show, id: store.id
+        expect(response.status).to eq(401)
+      end
+
+      it "I cannot create a new store" do
+        api_post :create, store: {}
+        expect(response.status).to eq(401)
+      end
+
+      it "I cannot update an existing store" do
+        api_put :update, id: store.id, store: {}
+        expect(response.status).to eq(401)
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: spree_api
 version: !ruby/object:Gem::Version
-  version: 2.4.0
+  version: 2.4.1
 platform: ruby
 authors:
 - Ryan Bigg
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-20 00:00:00.000000000 Z
+date: 2014-11-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: spree_core
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.4.0
+        version: 2.4.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.4.0
+        version: 2.4.1
 - !ruby/object:Gem::Dependency
   name: rabl
   requirement: !ruby/object:Gem::Requirement
@@ -88,6 +88,7 @@ files:
 - app/controllers/spree/api/stock_items_controller.rb
 - app/controllers/spree/api/stock_locations_controller.rb
 - app/controllers/spree/api/stock_movements_controller.rb
+- app/controllers/spree/api/stores_controller.rb
 - app/controllers/spree/api/taxonomies_controller.rb
 - app/controllers/spree/api/taxons_controller.rb
 - app/controllers/spree/api/users_controller.rb
@@ -166,6 +167,8 @@ files:
 - app/views/spree/api/stock_locations/show.v1.rabl
 - app/views/spree/api/stock_movements/index.v1.rabl
 - app/views/spree/api/stock_movements/show.v1.rabl
+- app/views/spree/api/stores/index.v1.rabl
+- app/views/spree/api/stores/show.v1.rabl
 - app/views/spree/api/taxonomies/index.v1.rabl
 - app/views/spree/api/taxonomies/jstree.rabl
 - app/views/spree/api/taxonomies/nested.v1.rabl
@@ -228,6 +231,7 @@ files:
 - spec/controllers/spree/api/stock_items_controller_spec.rb
 - spec/controllers/spree/api/stock_locations_controller_spec.rb
 - spec/controllers/spree/api/stock_movements_controller_spec.rb
+- spec/controllers/spree/api/stores_controller_spec.rb
 - spec/controllers/spree/api/taxonomies_controller_spec.rb
 - spec/controllers/spree/api/taxons_controller_spec.rb
 - spec/controllers/spree/api/unauthenticated_products_controller_spec.rb
@@ -292,6 +296,7 @@ test_files:
 - spec/controllers/spree/api/stock_items_controller_spec.rb
 - spec/controllers/spree/api/stock_locations_controller_spec.rb
 - spec/controllers/spree/api/stock_movements_controller_spec.rb
+- spec/controllers/spree/api/stores_controller_spec.rb
 - spec/controllers/spree/api/taxonomies_controller_spec.rb
 - spec/controllers/spree/api/taxons_controller_spec.rb
 - spec/controllers/spree/api/unauthenticated_products_controller_spec.rb