From 61110336bc5a924346f0654cb5bb812101c3bc7a Mon Sep 17 00:00:00 2001
From: HombreLaser <sebastian-440@live.com>
Date: Thu, 6 Apr 2023 14:53:02 -0600
Subject: Añade specs de cxarts_controller
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/controllers/api/carts_controller.rb        |  2 +-
 app/models/cart.rb                             |  2 +-
 spec/factories/cart.rb                         |  7 +++++++
 spec/factories/product_cart.rb                 |  9 +++++++++
 spec/models/product_cart_spec.rb               | 13 +++++++++++--
 spec/requests/carts_controller/destroy_spec.rb | 21 +++++++++++++++++++++
 6 files changed, 50 insertions(+), 4 deletions(-)
 create mode 100644 spec/factories/cart.rb
 create mode 100644 spec/factories/product_cart.rb
 create mode 100644 spec/requests/carts_controller/destroy_spec.rb

diff --git a/app/controllers/api/carts_controller.rb b/app/controllers/api/carts_controller.rb
index 0e14c17..d7c1529 100644
--- a/app/controllers/api/carts_controller.rb
+++ b/app/controllers/api/carts_controller.rb
@@ -20,7 +20,7 @@ module Api
     end
 
     def destroy
-      if @cart.delete_product(product.id)
+      if @cart.delete_product(params[:id])
         render status: :no_content
       else
         render status: :not_found
diff --git a/app/models/cart.rb b/app/models/cart.rb
index b08fc13..87e91d3 100644
--- a/app/models/cart.rb
+++ b/app/models/cart.rb
@@ -7,7 +7,7 @@ class Cart < ApplicationRecord
   has_many :products, through: :product_carts
 
   def delete_product(product_id)
-    relation = product_carts.find_by(product_id:, cart_id:)
+    relation = ProductCart.find_by(product_id:, cart_id: id)
 
     return false if relation.nil?
 
diff --git a/spec/factories/cart.rb b/spec/factories/cart.rb
new file mode 100644
index 0000000..e184181
--- /dev/null
+++ b/spec/factories/cart.rb
@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+FactoryBot.define do
+  factory :cart, class: 'Cart' do
+    user_account { create(:user_account) }
+  end
+end
diff --git a/spec/factories/product_cart.rb b/spec/factories/product_cart.rb
new file mode 100644
index 0000000..713fd0a
--- /dev/null
+++ b/spec/factories/product_cart.rb
@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+FactoryBot.define do
+  factory :product_cart, class: 'ProductCart' do
+    cart { create(:cart) }
+    product { create(:product) }
+    quantity { rand(1..1000) }
+  end
+end
diff --git a/spec/models/product_cart_spec.rb b/spec/models/product_cart_spec.rb
index 5a300b8..00a8f6c 100644
--- a/spec/models/product_cart_spec.rb
+++ b/spec/models/product_cart_spec.rb
@@ -1,6 +1,15 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 RSpec.describe ProductCart, type: :model do
-  it { should belong_to(:product) }
-  it { should belong_to(:cart) }
+  let(:product_cart) { create(:product_cart) }
+
+  it 'should belong to product' do
+    expect(product_cart).to respond_to(:product)
+  end
+
+  it 'should belong to cart' do
+    expect(product_cart).to respond_to(:cart)
+  end
 end
diff --git a/spec/requests/carts_controller/destroy_spec.rb b/spec/requests/carts_controller/destroy_spec.rb
new file mode 100644
index 0000000..ce083ef
--- /dev/null
+++ b/spec/requests/carts_controller/destroy_spec.rb
@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'DELETE /api/account/cart/product_id', type: :request do
+  let(:user) { create(:user_account, role: 'regular') }
+  let(:product) { create(:product) }
+  let(:token) { jwt(user) }
+  let(:headers) { { 'Authorization' => "Bearer #{token['token']}" } }
+
+  before(:each) do
+    user.cart = Cart.create
+    user.cart.add_product(product.id, 1)
+  end
+
+  it_behaves_like 'a DELETE request' do
+    let(:product_not_in_cart) { create(:product) }
+    let(:resource) { "/api/account/cart/#{product.id}" }
+    let(:nonexistent_resource) { "/api/account/cart/#{product_not_in_cart.id}" }
+  end
+end
-- 
cgit v1.2.3