From b1288b75ba1c418ffb1c49380d6aa6f1dab8e6d9 Mon Sep 17 00:00:00 2001
From: Saif Ur Rehman <saifi102@gmail.com>
Date: Sun, 6 Jan 2019 04:41:13 +0500
Subject: [PATCH] fix: Test Records

---
 .../item_tax_template/test_records.json       | 14 +++++++
 erpnext/stock/doctype/item/test_item.py       |  2 +-
 erpnext/stock/doctype/item/test_records.json  |  4 +-
 .../purchase_receipt/purchase_receipt.json    | 39 ++-----------------
 4 files changed, 20 insertions(+), 39 deletions(-)
 create mode 100644 erpnext/accounts/doctype/item_tax_template/test_records.json

diff --git a/erpnext/accounts/doctype/item_tax_template/test_records.json b/erpnext/accounts/doctype/item_tax_template/test_records.json
new file mode 100644
index 0000000000..3411f8c0aa
--- /dev/null
+++ b/erpnext/accounts/doctype/item_tax_template/test_records.json
@@ -0,0 +1,14 @@
+[
+ {
+  "doctype": "Item Tax Template",
+  "title": "_Test Account Excise Duty @ 10",
+  "taxes": [
+   {
+    "doctype": "Item Tax Template Detail",
+    "parentfield": "taxes",
+    "tax_rate": 10,
+    "tax_type": "_Test Account Excise Duty - _TC"
+   }
+  ]
+ }
+]
\ No newline at end of file
diff --git a/erpnext/stock/doctype/item/test_item.py b/erpnext/stock/doctype/item/test_item.py
index 24292f7b4f..b9d53a4705 100644
--- a/erpnext/stock/doctype/item/test_item.py
+++ b/erpnext/stock/doctype/item/test_item.py
@@ -17,7 +17,7 @@ from erpnext.stock.get_item_details import get_item_details
 from six import iteritems
 
 test_ignore = ["BOM"]
-test_dependencies = ["Warehouse", "Item Group"]
+test_dependencies = ["Warehouse", "Item Group", "Item Tax Template"]
 
 def make_item(item_code, properties=None):
 	if frappe.db.exists("Item", item_code):
diff --git a/erpnext/stock/doctype/item/test_records.json b/erpnext/stock/doctype/item/test_records.json
index 18e2910126..4cba5790ba 100644
--- a/erpnext/stock/doctype/item/test_records.json
+++ b/erpnext/stock/doctype/item/test_records.json
@@ -92,8 +92,8 @@
    {
     "doctype": "Item Tax",
     "parentfield": "taxes",
-    "tax_rate": 10,
-    "tax_type": "_Test Account Excise Duty - _TC"
+    "item_tax_template": "_Test Account Excise Duty @ 10",
+    "tax_category": ""
    }
   ],
   "stock_uom": "_Test UOM 1"
diff --git a/erpnext/stock/doctype/purchase_receipt/purchase_receipt.json b/erpnext/stock/doctype/purchase_receipt/purchase_receipt.json
index 8faf17ee9d..61767d9bed 100755
--- a/erpnext/stock/doctype/purchase_receipt/purchase_receipt.json
+++ b/erpnext/stock/doctype/purchase_receipt/purchase_receipt.json
@@ -85,7 +85,7 @@
    "bold": 0, 
    "collapsible": 0, 
    "columns": 0, 
-   "default": "", 
+   "default": "{supplier_name}",
    "fieldname": "title", 
    "fieldtype": "Data", 
    "hidden": 1, 
@@ -248,40 +248,7 @@
    "set_only_once": 0, 
    "translatable": 0, 
    "unique": 0
-  }, 
-  {
-   "allow_bulk_edit": 0, 
-   "allow_in_quick_entry": 0, 
-   "allow_on_submit": 0, 
-   "bold": 0, 
-   "collapsible": 0, 
-   "columns": 0, 
-   "fieldname": "letter_of_credit", 
-   "fieldtype": "Link", 
-   "hidden": 0, 
-   "ignore_user_permissions": 0, 
-   "ignore_xss_filter": 0, 
-   "in_filter": 1, 
-   "in_global_search": 0, 
-   "in_list_view": 0, 
-   "in_standard_filter": 1, 
-   "label": "Letter of Credit", 
-   "length": 0, 
-   "no_copy": 0, 
-   "options": "Letter of Credit", 
-   "permlevel": 0, 
-   "precision": "", 
-   "print_hide": 0, 
-   "print_hide_if_no_value": 0, 
-   "read_only": 0, 
-   "remember_last_selected_value": 0, 
-   "report_hide": 0, 
-   "reqd": 0, 
-   "search_index": 1, 
-   "set_only_once": 0, 
-   "translatable": 0, 
-   "unique": 0
-  }, 
+  },
   {
    "allow_bulk_edit": 0, 
    "allow_in_quick_entry": 0, 
@@ -3880,7 +3847,7 @@
  "quick_entry": 0, 
  "read_only": 0, 
  "read_only_onload": 1, 
- "search_fields": "status, posting_date, supplier, letter_of_credit", 
+ "search_fields": "status, posting_date, supplier",
  "show_name_in_global_search": 1, 
  "sort_field": "modified", 
  "sort_order": "DESC",