raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-01-24 20:36:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

Update tests and use ransack to search with the new scope: with_properties

Browse Source
This commit is contained in:
Jean-Baptiste Bellet
2022-02-02 09:57:23 +01:00
parent 572a65b324
commit dfe8ed26d9
2 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/controllers/api/v0/order_cycles_controller.rb
View File

@@ -81,7 +81,7 @@ module Api
def permitted_ransack_params
[:name_or_meta_keywords_or_variants_display_as_or_variants_display_name_or_supplier_name_cont,
:properties_id_or_supplier_properties_id_in_any,
:properties_id_or_supplier_properties_id_in_any, :with_properties,
:primary_taxon_id_in_any]
end

6
spec/controllers/api/v0/order_cycles_controller_spec.rb
View File

@@ -65,8 +65,9 @@ module Api
context "with property filters" do
it "filters by product property" do
api_get :products, id: order_cycle.id, distributor: distributor.id,
q: { properties_id_or_supplier_properties_id_in_any: [property1.id, property2.id] }
q: { with_properties: [property1.id, property2.id] }
expect(response.status).to eq 200
expect(product_ids).to include product1.id, product2.id
expect(product_ids).to_not include product3.id
end
@@ -83,8 +84,9 @@ module Api
it "filter out the product that don't inherits from supplier properties" do
api_get :products, id: order_cycle.id, distributor: distributor.id,
q: { properties_id_or_supplier_properties_id_in_any: [supplier_property.id] }
q: { with_properties: [supplier_property.id] }
expect(response.status).to eq 200
expect(product_ids).to include product1.id, product2.id
expect(product_ids).to_not include product3.id
end