Fixed bug

CSCI-GA-2820-SP24-003 · Apr 1, 2024 · 38db5dc · 38db5dc
1 parent 0829fc8
commit 38db5dc
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/service/models/shop_cart_item.py b/service/models/shop_cart_item.py
@@ -102,7 +102,7 @@ def find_by_product_id(cls, product_id):
             product_id (string): the product_id of the ShopCart Item you want to match
         """
         logger.info("Processing lookup for product_id: %s", product_id)
-        return cls.query.session.get(cls, product_id)
+        return cls.query.filter(cls.product_id == product_id).first()
 
     # @classmethod
     # def find_by_shopcart_id(cls, shopcart_id):

diff --git a/service/routes.py b/service/routes.py
@@ -254,7 +254,7 @@ def get_shopcart_items(shopcart_id, item_id):
 ######################################################################
 # RETRIEVE AN ITEM FROM SHOPCART BY PRODUCT ID
 ######################################################################
-@app.route("/shopcarts/<int:shopcart_id>/items/<int:product_id>", methods=["GET"])
+@app.route("/shopcarts/<int:shopcart_id>/products/<int:product_id>", methods=["GET"])
 def get_shopcart_items_by_product_id(shopcart_id, product_id):
     """
     Get an Item

diff --git a/tests/test_routes.py b/tests/test_routes.py
@@ -360,7 +360,7 @@ def test_get_shopcart_item_by_product_id(self):
 
         # retrieve it back
         resp = self.client.get(
-            f"{BASE_URL}/{shopcart.id}/items/{product_id}",
+            f"{BASE_URL}/{shopcart.id}/products/{product_id}",
             content_type="application/json",
         )
         self.assertEqual(resp.status_code, status.HTTP_200_OK)