diff --git a/app/controllers/admin/dfc_product_imports_controller.rb b/app/controllers/admin/dfc_product_imports_controller.rb index 0c545f37bf..8b7bb07e10 100644 --- a/app/controllers/admin/dfc_product_imports_controller.rb +++ b/app/controllers/admin/dfc_product_imports_controller.rb @@ -48,9 +48,9 @@ module Admin existing_variant = @enterprise.supplied_variants.linked_to(semantic_id) if existing_variant - SuppliedProductBuilder.update_product(subject, existing_variant) + SuppliedProductImporter.update_product(subject, existing_variant) else - SuppliedProductBuilder.store_product(subject, @enterprise) + SuppliedProductImporter.store_product(subject, @enterprise) end end diff --git a/engines/dfc_provider/app/controllers/dfc_provider/supplied_products_controller.rb b/engines/dfc_provider/app/controllers/dfc_provider/supplied_products_controller.rb index 438c9c86a3..51857b80b4 100644 --- a/engines/dfc_provider/app/controllers/dfc_provider/supplied_products_controller.rb +++ b/engines/dfc_provider/app/controllers/dfc_provider/supplied_products_controller.rb @@ -14,7 +14,7 @@ module DfcProvider return head :bad_request unless supplied_product - variant = SuppliedProductBuilder.store_product( + variant = SuppliedProductImporter.store_product( supplied_product, current_enterprise, ) @@ -33,7 +33,7 @@ module DfcProvider return head :bad_request unless supplied_product - SuppliedProductBuilder.update_product(supplied_product, variant) + SuppliedProductImporter.update_product(supplied_product, variant) end private diff --git a/engines/dfc_provider/app/services/supplied_product_builder.rb b/engines/dfc_provider/app/services/supplied_product_builder.rb index c442302bc8..63398b3f89 100644 --- a/engines/dfc_provider/app/services/supplied_product_builder.rb +++ b/engines/dfc_provider/app/services/supplied_product_builder.rb @@ -31,96 +31,11 @@ class SuppliedProductBuilder < DfcBuilder ) end - def self.store_product(subject, enterprise) - return unless subject.is_a? DataFoodConsortium::Connector::SuppliedProduct - - variant = SuppliedProductBuilder.import_variant(subject, enterprise) - product = variant.product - - product.save! if product.new_record? - variant.save! if variant.new_record? - - variant - end - - def self.update_product(supplied_product, variant) - apply(supplied_product, variant) - - variant.product.save! - variant.save! - - variant - end - - def self.import_variant(supplied_product, supplier) - product = referenced_spree_product(supplied_product, supplier) - - if product - Spree::Variant.new( product:, supplier:, price: 0,).tap do |variant| - apply(supplied_product, variant) - end - else - product = import_product(supplied_product, supplier) - product.variants.first.tap { |variant| apply(supplied_product, variant) } - end.tap do |variant| - link = supplied_product.semanticId - variant.semantic_links.new(semantic_id: link) if link.present? - end - end - - def self.referenced_spree_product(supplied_product, supplier) - uri = supplied_product.spree_product_uri - id = supplied_product.spree_product_id - - if uri.present? - route = Rails.application.routes.recognize_path(uri) - params = Rack::Utils.parse_nested_query(URI.parse(uri).query) - - # Check that the given URI points to us: - return unless uri == urls.enterprise_url(route.merge(params)) - - supplier.supplied_products.find_by(id: params["spree_product_id"]) - elsif id.present? - supplier.supplied_products.find_by(id:) - end - end - - def self.import_product(supplied_product, supplier) - Spree::Product.new( - name: supplied_product.name, - description: supplied_product.description, - price: 0, # will be in DFC Offer - supplier_id: supplier.id, - primary_taxon_id: taxon(supplied_product).id, - image: ImageBuilder.import(supplied_product.image), - ).tap do |product| - QuantitativeValueBuilder.apply(supplied_product.quantity, product) - product.ensure_standard_variant - end - end - - def self.apply(supplied_product, variant) - variant.product.assign_attributes(description: supplied_product.description) - - variant.display_name = supplied_product.name - variant.primary_taxon = taxon(supplied_product) - QuantitativeValueBuilder.apply(supplied_product.quantity, variant) - - catalog_item = supplied_product&.catalogItems&.first - offer = catalog_item&.offers&.first - CatalogItemBuilder.apply_stock(catalog_item, variant) - OfferBuilder.apply(offer, variant) - end - def self.product_type(variant) taxon_dfc_id = variant.primary_taxon&.dfc_id DataFoodConsortium::Connector::SKOSParser.concepts[taxon_dfc_id] end - def self.taxon(supplied_product) - ProductTypeImporter.taxon(supplied_product.productType) - end - - private_class_method :product_type, :taxon + private_class_method :product_type end diff --git a/engines/dfc_provider/app/services/supplied_product_importer.rb b/engines/dfc_provider/app/services/supplied_product_importer.rb new file mode 100644 index 0000000000..4426064913 --- /dev/null +++ b/engines/dfc_provider/app/services/supplied_product_importer.rb @@ -0,0 +1,90 @@ +# frozen_string_literal: true + +class SuppliedProductImporter < DfcBuilder + def self.store_product(subject, enterprise) + return unless subject.is_a? DataFoodConsortium::Connector::SuppliedProduct + + variant = import_variant(subject, enterprise) + product = variant.product + + product.save! if product.new_record? + variant.save! if variant.new_record? + + variant + end + + def self.update_product(supplied_product, variant) + apply(supplied_product, variant) + + variant.product.save! + variant.save! + + variant + end + + def self.import_variant(supplied_product, supplier) + product = referenced_spree_product(supplied_product, supplier) + + if product + Spree::Variant.new( product:, supplier:, price: 0,).tap do |variant| + apply(supplied_product, variant) + end + else + product = import_product(supplied_product, supplier) + product.variants.first.tap { |variant| apply(supplied_product, variant) } + end.tap do |variant| + link = supplied_product.semanticId + variant.semantic_links.new(semantic_id: link) if link.present? + end + end + + def self.referenced_spree_product(supplied_product, supplier) + uri = supplied_product.spree_product_uri + id = supplied_product.spree_product_id + + if uri.present? + route = Rails.application.routes.recognize_path(uri) + params = Rack::Utils.parse_nested_query(URI.parse(uri).query) + + # Check that the given URI points to us: + return unless uri == urls.enterprise_url(route.merge(params)) + + supplier.supplied_products.find_by(id: params["spree_product_id"]) + elsif id.present? + supplier.supplied_products.find_by(id:) + end + end + + def self.import_product(supplied_product, supplier) + Spree::Product.new( + name: supplied_product.name, + description: supplied_product.description, + price: 0, # will be in DFC Offer + supplier_id: supplier.id, + primary_taxon_id: taxon(supplied_product).id, + image: ImageBuilder.import(supplied_product.image), + ).tap do |product| + QuantitativeValueBuilder.apply(supplied_product.quantity, product) + product.ensure_standard_variant + end + end + + def self.apply(supplied_product, variant) + variant.product.assign_attributes(description: supplied_product.description) + + variant.display_name = supplied_product.name + variant.primary_taxon = taxon(supplied_product) + QuantitativeValueBuilder.apply(supplied_product.quantity, variant) + + catalog_item = supplied_product&.catalogItems&.first + offer = catalog_item&.offers&.first + CatalogItemBuilder.apply_stock(catalog_item, variant) + OfferBuilder.apply(offer, variant) + end + + def self.taxon(supplied_product) + ProductTypeImporter.taxon(supplied_product.productType) + end + + private_class_method :taxon +end diff --git a/engines/dfc_provider/spec/services/supplied_product_builder_spec.rb b/engines/dfc_provider/spec/services/supplied_product_builder_spec.rb index bd222af048..e76523ecb4 100644 --- a/engines/dfc_provider/spec/services/supplied_product_builder_spec.rb +++ b/engines/dfc_provider/spec/services/supplied_product_builder_spec.rb @@ -83,272 +83,4 @@ RSpec.describe SuppliedProductBuilder do ) end end - - describe ".store_product" do - let(:subject) { builder.store_product(product, supplier) } - let(:product) { - DfcIo.import(product_json).find do |subject| - subject.is_a? DataFoodConsortium::Connector::SuppliedProduct - end - } - let(:product_json) { ExampleJson.read("product.GET") } - - before do - taxon.save! - end - - it "stores a new Spree Product and Variant" do - expect { subject }.to change { - Spree::Product.count - }.by(1) - - expect(subject).to be_a(Spree::Variant) - expect(subject).to be_valid - expect(subject).to be_persisted - expect(subject.name).to eq("Fillet Steak - 201g x 1 Steak") - expect(subject.variant_unit).to eq("items") - expect(subject.variant_unit_scale).to eq(nil) - expect(subject.variant_unit_with_scale).to eq("items") - expect(subject.unit_value).to eq(1) - end - end - - describe ".update_product" do - let(:subject) { builder.update_product(product, variant) } - let(:product) { - DfcIo.import(product_json).find do |subject| - subject.is_a? DataFoodConsortium::Connector::SuppliedProduct - end - } - let(:product_json) { ExampleJson.read("product.GET") } - - it "updates a variant" do - variant # Create test data first - - expect { subject }.not_to change { - Spree::Variant.count - } - - expect(subject).to eq variant - expect(subject.display_name).to eq "Fillet Steak - 201g x 1 Steak" - expect(subject.variant_unit).to eq "items" - expect(subject.unit_value).to eq 1 - expect(subject.on_demand).to eq false - expect(subject.on_hand).to eq 11 - end - end - - describe ".import_product" do - let(:supplied_product) do - DfcProvider::SuppliedProduct.new( - "https://example.net/tomato", - name: "Tomato", - description: "Awesome tomato", - quantity: DataFoodConsortium::Connector::QuantitativeValue.new( - unit: DfcLoader.connector.MEASURES.KILOGRAM, - value: 2, - ), - productType: product_type, - image: "https://cd.net/tomato.png?v=5", - ) - end - let(:product_type) { DfcLoader.connector.PRODUCT_TYPES.VEGETABLE.NON_LOCAL_VEGETABLE } - let!(:taxon) { - create( - :taxon, - name: "Non local vegetable", - dfc_id: "https://github.com/datafoodconsortium/taxonomies/releases/latest/download/productTypes.rdf#non-local-vegetable" - ) - } - - before do - stub_request(:get, "https://cd.net/tomato.png?v=5").to_return( - status: 200, - body: black_logo_path.read, - ) - end - - it "creates a new Spree::Product" do - product = builder.import_product(supplied_product, supplier) - - expect(product).to be_a(Spree::Product) - expect(product.name).to eq("Tomato") - expect(product.description).to eq("Awesome tomato") - expect(product.variant_unit).to eq("weight") - expect(product.image).to be_present - expect(product.image.attachment).to be_attached - expect(product.image.url(:product)).to match /^http.*tomato\.png/ - end - - describe "taxon" do - it "assigns the taxon matching the DFC product type" do - product = builder.import_product(supplied_product, supplier) - - expect(product.variants.first.primary_taxon).to eq(taxon) - end - end - end - - describe ".import_variant" do - let(:imported_variant) { builder.import_variant(supplied_product, supplier) } - let(:supplied_product) do - DfcProvider::SuppliedProduct.new( - "https://example.net/tomato", - name: "Tomato", - description: "Awesome tomato", - quantity: DataFoodConsortium::Connector::QuantitativeValue.new( - unit: DfcLoader.connector.MEASURES.KILOGRAM, - value: 2, - ), - productType: product_type, - catalogItems: [catalog_item], - ) - end - let(:product_type) { DfcLoader.connector.PRODUCT_TYPES.VEGETABLE.NON_LOCAL_VEGETABLE } - let(:catalog_item) { - DataFoodConsortium::Connector::CatalogItem.new( - nil, - # On-demand is expressed as negative stock. - # And some APIs send strings instead of numbers... - stockLimitation: "-1", - offers: [offer], - ) - } - let(:offer) { - DataFoodConsortium::Connector::Offer.new( - nil, - price: DataFoodConsortium::Connector::Price.new(value: "15.50"), - ) - } - - it "creates a new Spree::Product and variant" do - create(:taxon) - - expect(imported_variant).to be_a(Spree::Variant) - expect(imported_variant).to be_valid - expect(imported_variant.id).to be_nil - expect(imported_variant.semantic_links.size).to eq 1 - - link = imported_variant.semantic_links[0] - expect(link.semantic_id).to eq "https://example.net/tomato" - - imported_product = imported_variant.product - expect(imported_product).to be_a(Spree::Product) - expect(imported_product).to be_valid - expect(imported_product.id).to be_nil - expect(imported_product.name).to eq("Tomato") - expect(imported_product.description).to eq("Awesome tomato") - expect(imported_product.variant_unit).to eq("weight") - - # Stock can only be checked when persisted: - imported_product.save! - expect(imported_variant.price).to eq 15.50 - expect(imported_variant.on_demand).to eq true - expect(imported_variant.on_hand).to eq 0 - end - - context "with spree_product_id supplied" do - let(:imported_variant) { builder.import_variant(supplied_product, supplier) } - - let(:supplied_product) do - DfcProvider::SuppliedProduct.new( - "https://example.net/tomato", - name: "Tomato", - description: "Better Awesome tomato", - quantity: DataFoodConsortium::Connector::QuantitativeValue.new( - unit: DfcLoader.connector.MEASURES.KILOGRAM, - value: 2, - ), - productType: product_type, - spree_product_id: variant.product.id - ) - end - let(:product_type) { DfcLoader.connector.PRODUCT_TYPES.DRINK.SOFT_DRINK.FRUIT_JUICE } - let!(:new_taxon) { - create( - :taxon, - name: "Fruit Juice", - dfc_id: "https://github.com/datafoodconsortium/taxonomies/releases/latest/download/productTypes.rdf#fruit-juice" - ) - } - - it "update an existing Spree::Product" do - imported_product = imported_variant.product - expect(imported_product.id).to eq(spree_product.id) - expect(imported_product.description).to eq("Better Awesome tomato") - expect(imported_variant.primary_taxon).to eq(new_taxon) - end - - context "when spree_product_uri doesn't match the server host" do - let(:supplied_product) do - DfcProvider::SuppliedProduct.new( - "https://example.net/tomato", - name: "Tomato", - description: "Awesome tomato", - quantity: DataFoodConsortium::Connector::QuantitativeValue.new( - unit: DfcLoader.connector.MEASURES.KILOGRAM, - value: 2, - ), - productType: product_type, - spree_product_uri: "http://another.host/api/dfc/enterprises/10/supplied_products/50" - ) - end - - it "creates a new Spree::Product and variant" do - expect(imported_variant).to be_a(Spree::Variant) - expect(imported_variant.id).to be_nil - - imported_product = imported_variant.product - expect(imported_product).to be_a(Spree::Product) - expect(imported_product.id).to be_nil - expect(imported_product.name).to eq("Tomato") - expect(imported_product.description).to eq("Awesome tomato") - expect(imported_product.variant_unit).to eq("weight") - end - end - end - end - - describe ".referenced_spree_product" do - let(:result) { builder.referenced_spree_product(supplied_product, supplier) } - let(:supplied_product) do - DfcProvider::SuppliedProduct.new( - "https://example.net/tomato", - name: "Tomato", - ) - end - - it "returns nil when no reference is given" do - expect(result).to eq nil - end - - it "returns a product referenced by URI" do - variant.save! - supplied_product.spree_product_uri = - "http://test.host/api/dfc/enterprises/7?spree_product_id=6" - expect(result).to eq spree_product - end - - it "doesn't return a product of another enterprise" do - variant.save! - create(:product, id: 8, supplier_id: create(:enterprise).id) - - supplied_product.spree_product_uri = - "http://test.host/api/dfc/enterprises/7?spree_product_id=8" - expect(result).to eq nil - end - - it "doesn't return a foreign product referenced by URI" do - variant.save! - supplied_product.spree_product_uri = - "http://another.host/api/dfc/enterprises/7?spree_product_id=6" - expect(result).to eq nil - end - - it "returns a product referenced by id" do - variant.save! - supplied_product.spree_product_id = "6" - expect(result).to eq spree_product - end - end end diff --git a/engines/dfc_provider/spec/services/supplied_product_importer_spec.rb b/engines/dfc_provider/spec/services/supplied_product_importer_spec.rb new file mode 100644 index 0000000000..845d2cb3d0 --- /dev/null +++ b/engines/dfc_provider/spec/services/supplied_product_importer_spec.rb @@ -0,0 +1,293 @@ +# frozen_string_literal: true + +require_relative "../spec_helper" + +RSpec.describe SuppliedProductImporter do + include FileHelper + + subject(:importer) { described_class } + let(:variant) { + create(:variant, id: 5, product: spree_product, primary_taxon: taxon, supplier:) + } + let(:spree_product) { + create(:product, id: 6) + } + let(:supplier) { + create(:supplier_enterprise, id: 7) + } + let(:taxon) { + build( + :taxon, + name: "Soft Drink", + dfc_id: "https://github.com/datafoodconsortium/taxonomies/releases/latest/download/productTypes.rdf#soft-drink" + ) + } + + describe ".store_product" do + let(:subject) { importer.store_product(product, supplier) } + let(:product) { + DfcIo.import(product_json).find do |subject| + subject.is_a? DataFoodConsortium::Connector::SuppliedProduct + end + } + let(:product_json) { ExampleJson.read("product.GET") } + + before do + taxon.save! + end + + it "stores a new Spree Product and Variant" do + expect { subject }.to change { + Spree::Product.count + }.by(1) + + expect(subject).to be_a(Spree::Variant) + expect(subject).to be_valid + expect(subject).to be_persisted + expect(subject.name).to eq("Fillet Steak - 201g x 1 Steak") + expect(subject.variant_unit).to eq("items") + expect(subject.variant_unit_scale).to eq(nil) + expect(subject.variant_unit_with_scale).to eq("items") + expect(subject.unit_value).to eq(1) + end + end + + describe ".update_product" do + let(:subject) { importer.update_product(product, variant) } + let(:product) { + DfcIo.import(product_json).find do |subject| + subject.is_a? DataFoodConsortium::Connector::SuppliedProduct + end + } + let(:product_json) { ExampleJson.read("product.GET") } + + it "updates a variant" do + variant # Create test data first + + expect { subject }.not_to change { + Spree::Variant.count + } + + expect(subject).to eq variant + expect(subject.display_name).to eq "Fillet Steak - 201g x 1 Steak" + expect(subject.variant_unit).to eq "items" + expect(subject.unit_value).to eq 1 + expect(subject.on_demand).to eq false + expect(subject.on_hand).to eq 11 + end + end + + describe ".import_product" do + let(:supplied_product) do + DfcProvider::SuppliedProduct.new( + "https://example.net/tomato", + name: "Tomato", + description: "Awesome tomato", + quantity: DataFoodConsortium::Connector::QuantitativeValue.new( + unit: DfcLoader.connector.MEASURES.KILOGRAM, + value: 2, + ), + productType: product_type, + image: "https://cd.net/tomato.png?v=5", + ) + end + let(:product_type) { DfcLoader.connector.PRODUCT_TYPES.VEGETABLE.NON_LOCAL_VEGETABLE } + let!(:taxon) { + create( + :taxon, + name: "Non local vegetable", + dfc_id: "https://github.com/datafoodconsortium/taxonomies/releases/latest/download/productTypes.rdf#non-local-vegetable" + ) + } + + before do + stub_request(:get, "https://cd.net/tomato.png?v=5").to_return( + status: 200, + body: black_logo_path.read, + ) + end + + it "creates a new Spree::Product" do + product = importer.import_product(supplied_product, supplier) + + expect(product).to be_a(Spree::Product) + expect(product.name).to eq("Tomato") + expect(product.description).to eq("Awesome tomato") + expect(product.variant_unit).to eq("weight") + expect(product.image).to be_present + expect(product.image.attachment).to be_attached + expect(product.image.url(:product)).to match /^http.*tomato\.png/ + end + + describe "taxon" do + it "assigns the taxon matching the DFC product type" do + product = importer.import_product(supplied_product, supplier) + + expect(product.variants.first.primary_taxon).to eq(taxon) + end + end + end + + describe ".import_variant" do + let(:imported_variant) { importer.import_variant(supplied_product, supplier) } + let(:supplied_product) do + DfcProvider::SuppliedProduct.new( + "https://example.net/tomato", + name: "Tomato", + description: "Awesome tomato", + quantity: DataFoodConsortium::Connector::QuantitativeValue.new( + unit: DfcLoader.connector.MEASURES.KILOGRAM, + value: 2, + ), + productType: product_type, + catalogItems: [catalog_item], + ) + end + let(:product_type) { DfcLoader.connector.PRODUCT_TYPES.VEGETABLE.NON_LOCAL_VEGETABLE } + let(:catalog_item) { + DataFoodConsortium::Connector::CatalogItem.new( + nil, + # On-demand is expressed as negative stock. + # And some APIs send strings instead of numbers... + stockLimitation: "-1", + offers: [offer], + ) + } + let(:offer) { + DataFoodConsortium::Connector::Offer.new( + nil, + price: DataFoodConsortium::Connector::Price.new(value: "15.50"), + ) + } + + it "creates a new Spree::Product and variant" do + create(:taxon) + + expect(imported_variant).to be_a(Spree::Variant) + expect(imported_variant).to be_valid + expect(imported_variant.id).to be_nil + expect(imported_variant.semantic_links.size).to eq 1 + + link = imported_variant.semantic_links[0] + expect(link.semantic_id).to eq "https://example.net/tomato" + + imported_product = imported_variant.product + expect(imported_product).to be_a(Spree::Product) + expect(imported_product).to be_valid + expect(imported_product.id).to be_nil + expect(imported_product.name).to eq("Tomato") + expect(imported_product.description).to eq("Awesome tomato") + expect(imported_product.variant_unit).to eq("weight") + + # Stock can only be checked when persisted: + imported_product.save! + expect(imported_variant.price).to eq 15.50 + expect(imported_variant.on_demand).to eq true + expect(imported_variant.on_hand).to eq 0 + end + + context "with spree_product_id supplied" do + let(:imported_variant) { importer.import_variant(supplied_product, supplier) } + + let(:supplied_product) do + DfcProvider::SuppliedProduct.new( + "https://example.net/tomato", + name: "Tomato", + description: "Better Awesome tomato", + quantity: DataFoodConsortium::Connector::QuantitativeValue.new( + unit: DfcLoader.connector.MEASURES.KILOGRAM, + value: 2, + ), + productType: product_type, + spree_product_id: variant.product.id + ) + end + let(:product_type) { DfcLoader.connector.PRODUCT_TYPES.DRINK.SOFT_DRINK.FRUIT_JUICE } + let!(:new_taxon) { + create( + :taxon, + name: "Fruit Juice", + dfc_id: "https://github.com/datafoodconsortium/taxonomies/releases/latest/download/productTypes.rdf#fruit-juice" + ) + } + + it "update an existing Spree::Product" do + imported_product = imported_variant.product + expect(imported_product.id).to eq(spree_product.id) + expect(imported_product.description).to eq("Better Awesome tomato") + expect(imported_variant.primary_taxon).to eq(new_taxon) + end + + context "when spree_product_uri doesn't match the server host" do + let(:supplied_product) do + DfcProvider::SuppliedProduct.new( + "https://example.net/tomato", + name: "Tomato", + description: "Awesome tomato", + quantity: DataFoodConsortium::Connector::QuantitativeValue.new( + unit: DfcLoader.connector.MEASURES.KILOGRAM, + value: 2, + ), + productType: product_type, + spree_product_uri: "http://another.host/api/dfc/enterprises/10/supplied_products/50" + ) + end + + it "creates a new Spree::Product and variant" do + expect(imported_variant).to be_a(Spree::Variant) + expect(imported_variant.id).to be_nil + + imported_product = imported_variant.product + expect(imported_product).to be_a(Spree::Product) + expect(imported_product.id).to be_nil + expect(imported_product.name).to eq("Tomato") + expect(imported_product.description).to eq("Awesome tomato") + expect(imported_product.variant_unit).to eq("weight") + end + end + end + end + + describe ".referenced_spree_product" do + let(:result) { importer.referenced_spree_product(supplied_product, supplier) } + let(:supplied_product) do + DfcProvider::SuppliedProduct.new( + "https://example.net/tomato", + name: "Tomato", + ) + end + + it "returns nil when no reference is given" do + expect(result).to eq nil + end + + it "returns a product referenced by URI" do + variant.save! + supplied_product.spree_product_uri = + "http://test.host/api/dfc/enterprises/7?spree_product_id=6" + expect(result).to eq spree_product + end + + it "doesn't return a product of another enterprise" do + variant.save! + create(:product, id: 8, supplier_id: create(:enterprise).id) + + supplied_product.spree_product_uri = + "http://test.host/api/dfc/enterprises/7?spree_product_id=8" + expect(result).to eq nil + end + + it "doesn't return a foreign product referenced by URI" do + variant.save! + supplied_product.spree_product_uri = + "http://another.host/api/dfc/enterprises/7?spree_product_id=6" + expect(result).to eq nil + end + + it "returns a product referenced by id" do + variant.save! + supplied_product.spree_product_id = "6" + expect(result).to eq spree_product + end + end +end