From 78ee51c6ca6eab82238a74dfcc3ef490ad408d38 Mon Sep 17 00:00:00 2001 From: marination Date: Tue, 10 Aug 2021 12:18:49 +0530 Subject: [PATCH] chore: debug travis --- .../e_commerce/doctype/item_review/item_review.json | 5 +++-- .../e_commerce/doctype/item_review/test_item_review.py | 10 ++++++---- .../doctype/website_item/test_website_item.py | 1 + erpnext/e_commerce/doctype/wishlist/test_wishlist.py | 6 ++---- 4 files changed, 12 insertions(+), 10 deletions(-) diff --git a/erpnext/e_commerce/doctype/item_review/item_review.json b/erpnext/e_commerce/doctype/item_review/item_review.json index cd6e20b10e..57f719fc3c 100644 --- a/erpnext/e_commerce/doctype/item_review/item_review.json +++ b/erpnext/e_commerce/doctype/item_review/item_review.json @@ -23,7 +23,8 @@ "fieldtype": "Link", "label": "Website Item", "options": "Website Item", - "read_only": 1 + "read_only": 1, + "reqd": 1 }, { "fieldname": "user", @@ -86,7 +87,7 @@ ], "index_web_pages_for_search": 1, "links": [], - "modified": "2021-08-09 08:06:02.476254", + "modified": "2021-08-10 12:08:58.119691", "modified_by": "Administrator", "module": "E-commerce", "name": "Item Review", diff --git a/erpnext/e_commerce/doctype/item_review/test_item_review.py b/erpnext/e_commerce/doctype/item_review/test_item_review.py index bcf8b5e5b4..b19f42ada1 100644 --- a/erpnext/e_commerce/doctype/item_review/test_item_review.py +++ b/erpnext/e_commerce/doctype/item_review/test_item_review.py @@ -14,15 +14,17 @@ from erpnext.e_commerce.shopping_cart.cart import get_party from erpnext.e_commerce.doctype.e_commerce_settings.test_e_commerce_settings import setup_e_commerce_settings class TestItemReview(unittest.TestCase): - @classmethod - def setUpClass(cls): + def setUp(self): item = make_item("Test Mobile Phone") if not frappe.db.exists("Website Item", {"item_code": "Test Mobile Phone"}): make_website_item(item, save=True) - @classmethod - def tearDownClass(cls): + setup_e_commerce_settings({"enable_reviews": 1}) + frappe.local.shopping_cart_settings = None + + def tearDown(self): frappe.get_cached_doc("Website Item", {"item_code": "Test Mobile Phone"}).delete() + setup_e_commerce_settings({"enable_reviews": 0}) def test_add_and_get_item_reviews_from_customer(self): "Add / Get Reviews from a User that is a valid customer (has added to cart or purchased in the past)" diff --git a/erpnext/e_commerce/doctype/website_item/test_website_item.py b/erpnext/e_commerce/doctype/website_item/test_website_item.py index d2fb08f64f..9afca251b4 100644 --- a/erpnext/e_commerce/doctype/website_item/test_website_item.py +++ b/erpnext/e_commerce/doctype/website_item/test_website_item.py @@ -300,6 +300,7 @@ class TestWebsiteItem(unittest.TestCase): item_code = "Test Mobile Phone" self.create_regular_web_item() setup_e_commerce_settings({"show_stock_availability": 1}) + frappe.local.shopping_cart_settings = None # set warehouse frappe.db.set_value("Website Item", {"item_code": item_code}, "website_warehouse", "_Test Warehouse - _TC") diff --git a/erpnext/e_commerce/doctype/wishlist/test_wishlist.py b/erpnext/e_commerce/doctype/wishlist/test_wishlist.py index e765030caf..42cf8683d9 100644 --- a/erpnext/e_commerce/doctype/wishlist/test_wishlist.py +++ b/erpnext/e_commerce/doctype/wishlist/test_wishlist.py @@ -11,8 +11,7 @@ from erpnext.e_commerce.doctype.website_item.website_item import make_website_it from erpnext.e_commerce.doctype.wishlist.wishlist import add_to_wishlist, remove_from_wishlist class TestWishlist(unittest.TestCase): - @classmethod - def setUpClass(cls): + def setUp(self): item = make_item("Test Phone Series X") if not frappe.db.exists("Website Item", {"item_code": "Test Phone Series X"}): make_website_item(item, save=True) @@ -21,8 +20,7 @@ class TestWishlist(unittest.TestCase): if not frappe.db.exists("Website Item", {"item_code": "Test Phone Series Y"}): make_website_item(item, save=True) - @classmethod - def tearDownClass(cls): + def tearDown(self): frappe.get_cached_doc("Website Item", {"item_code": "Test Phone Series X"}).delete() frappe.get_cached_doc("Website Item", {"item_code": "Test Phone Series Y"}).delete() frappe.get_cached_doc("Item", "Test Phone Series X").delete()