raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-02-08 22:56:06 +00:00
Code Issues Packages Projects Releases Wiki Activity

Use default enterprise_id to get the entreprise

Browse Source
This commit is contained in:
François Turbelin
2020-05-06 21:45:35 +02:00
parent 99798b010f
commit f6e06b5658
3 changed files with 44 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
engines/dfc_provider/app/controllers/dfc_provider/api/products_controller.rb
View File

@@ -28,7 +28,12 @@ module DfcProvider
private
def check_enterprise
@enterprise = @user.enterprises.first
@enterprise =
if params[:enterprise_id] == 'default'
@user.enterprises.first
else
@user.enterprises.where(id: params[:enterprise_id]).first
end
return if @enterprise.present?

4
engines/dfc_provider/config/routes.rb
View File

@@ -3,8 +3,8 @@
DfcProvider::Engine.routes.draw do
namespace :api do
scope :dfc_provider, as: :dfc_provider, path: '/dfc_provider' do
resource :enterprise, only: :none do
resources :products, only: %i[index]
resources :enterprises, only: :none do
resources :products, only: [:index]
end
end
end

47
engines/dfc_provider/spec/controllers/dfc_provider/api/products_controller_spec.rb
View File

@@ -7,9 +7,7 @@ describe DfcProvider::Api::ProductsController, type: :controller do
let(:user) { create(:user) }
let(:enterprise) { create(:distributor_enterprise, owner: user) }
let(:product) do
create(:simple_product, supplier: enterprise )
end
let(:product) { create(:simple_product, supplier: enterprise ) }
let!(:visible_inventory_item) do
create(:inventory_item,
enterprise: enterprise,
@@ -31,22 +29,49 @@ describe DfcProvider::Api::ProductsController, type: :controller do
end
context 'with an enterprise' do
before { get :index }
context 'given with an id' do
context 'related to the user' do
before { get :index, enterprise_id: enterprise.id }
it 'is successful' do
expect(response.status).to eq 200
it 'is successful' do
expect(response.status).to eq 200
end
it 'renders the related product' do
expect(response.body)
.to include(product.variants.first.name)
end
end
context 'not related to the user' do
let(:enterprise) { create(:enterprise) }
before { get :index, enterprise_id: enterprise.id }
it 'returns not_found head' do
expect(response.status).to eq 404
end
end
end
it 'renders the related product' do
expect(response.body)
.to include("\"DFC:description\":\"#{product.variants.first.name}\"")
context 'as default' do
before { get :index, enterprise_id: 'default' }
it 'is successful' do
expect(response.status).to eq 200
end
it 'renders the related product' do
expect(response.body)
.to include(product.variants.first.name)
end
end
end
context 'without a recorded enterprise' do
let(:enterprise) { create(:enterprise) }
before { get :index }
before { get :index, enterprise_id: 'default' }
it 'returns not_found head' do
expect(response.status).to eq 404
@@ -61,7 +86,7 @@ describe DfcProvider::Api::ProductsController, type: :controller do
.and_return(nil)
end
before { get :index }
before { get :index, enterprise_id: 'default' }
it 'returns unauthorized head' do
expect(response.status).to eq 401