Skip to content

Commit

Permalink
Revert "fix: set empty value for tax template in item details (#37496)"
Browse files Browse the repository at this point in the history 
This reverts commit b0d440c.
  • Loading branch information
@deepeshgarg007
deepeshgarg007 authored Nov 8, 2023
1 parent 4783e4b commit 70d99ee
Show file tree
Hide file tree
Showing 4 changed files with 113 additions and 111 deletions.
105 changes: 55 additions & 50 deletions erpnext/accounts/doctype/pos_invoice/test_pos_invoice.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@

import frappe
from frappe import _
from frappe.utils import add_days, nowdate

from erpnext.accounts.doctype.pos_invoice.pos_invoice import make_sales_return
from erpnext.accounts.doctype.pos_profile.test_pos_profile import make_pos_profile
Expand Down Expand Up @@ -126,64 +125,70 @@ def test_tax_calculation_with_multiple_items(self):
self.assertEqual(inv.grand_total, 5474.0)

def test_tax_calculation_with_item_tax_template(self):
import json
inv = create_pos_invoice(qty=84, rate=4.6, do_not_save=1)
item_row = inv.get("items")[0]

from erpnext.stock.get_item_details import get_item_details
add_items = [
(54, "_Test Account Excise Duty @ 12 - _TC"),
(288, "_Test Account Excise Duty @ 15 - _TC"),
(144, "_Test Account Excise Duty @ 20 - _TC"),
(430, "_Test Item Tax Template 1 - _TC"),
]
for qty, item_tax_template in add_items:
item_row_copy = copy.deepcopy(item_row)
item_row_copy.qty = qty
item_row_copy.item_tax_template = item_tax_template
inv.append("items", item_row_copy)

# set tax template in item
item = frappe.get_cached_doc("Item", "_Test Item")
item.set(
inv.append(
"taxes",
[
{
"item_tax_template": "_Test Account Excise Duty @ 15 - _TC",
"valid_from": add_days(nowdate(), -5),
}
],
)
item.save()

# create POS invoice with item
pos_inv = create_pos_invoice(qty=84, rate=4.6, do_not_save=True)
item_details = get_item_details(
doc=pos_inv,
args={
"item_code": item.item_code,
"company": pos_inv.company,
"doctype": "POS Invoice",
"conversion_rate": 1.0,
{
"account_head": "_Test Account Excise Duty - _TC",
"charge_type": "On Net Total",
"cost_center": "_Test Cost Center - _TC",
"description": "Excise Duty",
"doctype": "Sales Taxes and Charges",
"rate": 11,
},
)
tax_map = json.loads(item_details.item_tax_rate)
for tax in tax_map:
pos_inv.append(
"taxes",
{
"charge_type": "On Net Total",
"account_head": tax,
"rate": tax_map[tax],
"description": "Test",
"cost_center": "_Test Cost Center - _TC",
},
)
pos_inv.submit()
pos_inv.load_from_db()

# check if correct tax values are applied from tax template
self.assertEqual(pos_inv.net_total, 386.4)

expected_taxes = [
inv.append(
"taxes",
{
"tax_amount": 57.96,
"total": 444.36,
"account_head": "_Test Account Education Cess - _TC",
"charge_type": "On Net Total",
"cost_center": "_Test Cost Center - _TC",
"description": "Education Cess",
"doctype": "Sales Taxes and Charges",
"rate": 0,
},
]
)
inv.append(
"taxes",
{
"account_head": "_Test Account S&H Education Cess - _TC",
"charge_type": "On Net Total",
"cost_center": "_Test Cost Center - _TC",
"description": "S&H Education Cess",
"doctype": "Sales Taxes and Charges",
"rate": 3,
},
)
inv.insert()

self.assertEqual(inv.net_total, 4600)

self.assertEqual(inv.get("taxes")[0].tax_amount, 502.41)
self.assertEqual(inv.get("taxes")[0].total, 5102.41)

self.assertEqual(inv.get("taxes")[1].tax_amount, 197.80)
self.assertEqual(inv.get("taxes")[1].total, 5300.21)

for i in range(len(expected_taxes)):
for key in expected_taxes[i]:
self.assertEqual(expected_taxes[i][key], pos_inv.get("taxes")[i].get(key))
self.assertEqual(inv.get("taxes")[2].tax_amount, 375.36)
self.assertEqual(inv.get("taxes")[2].total, 5675.57)

self.assertEqual(pos_inv.get("base_total_taxes_and_charges"), 57.96)
self.assertEqual(inv.grand_total, 5675.57)
self.assertEqual(inv.rounding_adjustment, 0.43)
self.assertEqual(inv.rounded_total, 5676.0)

def test_tax_calculation_with_multiple_items_and_discount(self):
inv = create_pos_invoice(qty=1, rate=75, do_not_save=True)
Expand Down
108 changes: 53 additions & 55 deletions erpnext/accounts/doctype/sales_invoice/test_sales_invoice.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -516,72 +516,70 @@ def test_tax_calculation_with_multiple_items(self):
self.assertEqual(si.grand_total, 5474.0)

def test_tax_calculation_with_item_tax_template(self):
import json

from erpnext.stock.get_item_details import get_item_details

# set tax template in item
item = frappe.get_cached_doc("Item", "_Test Item")
item.set(
"taxes",
[
{
"item_tax_template": "_Test Item Tax Template 1 - _TC",
"valid_from": add_days(nowdate(), -5),
}
],
)
item.save()

# create sales invoice with item
si = create_sales_invoice(qty=84, rate=4.6, do_not_save=True)
item_details = get_item_details(
doc=si,
args={
"item_code": item.item_code,
"company": si.company,
"doctype": "Sales Invoice",
"conversion_rate": 1.0,
},
)
tax_map = json.loads(item_details.item_tax_rate)
for tax in tax_map:
si.append(
"taxes",
{
"charge_type": "On Net Total",
"account_head": tax,
"rate": tax_map[tax],
"description": "Test",
"cost_center": "_Test Cost Center - _TC",
},
)
si.submit()
si.load_from_db()
item_row = si.get("items")[0]

# check if correct tax values are applied from tax template
self.assertEqual(si.net_total, 386.4)
add_items = [
(54, "_Test Account Excise Duty @ 12 - _TC"),
(288, "_Test Account Excise Duty @ 15 - _TC"),
(144, "_Test Account Excise Duty @ 20 - _TC"),
(430, "_Test Item Tax Template 1 - _TC"),
]
for qty, item_tax_template in add_items:
item_row_copy = copy.deepcopy(item_row)
item_row_copy.qty = qty
item_row_copy.item_tax_template = item_tax_template
si.append("items", item_row_copy)

expected_taxes = [
si.append(
"taxes",
{
"tax_amount": 19.32,
"total": 405.72,
"account_head": "_Test Account Excise Duty - _TC",
"charge_type": "On Net Total",
"cost_center": "_Test Cost Center - _TC",
"description": "Excise Duty",
"doctype": "Sales Taxes and Charges",
"rate": 11,
},
)
si.append(
"taxes",
{
"tax_amount": 38.64,
"total": 444.36,
"account_head": "_Test Account Education Cess - _TC",
"charge_type": "On Net Total",
"cost_center": "_Test Cost Center - _TC",
"description": "Education Cess",
"doctype": "Sales Taxes and Charges",
"rate": 0,
},
)
si.append(
"taxes",
{
"tax_amount": 57.96,
"total": 502.32,
"account_head": "_Test Account S&H Education Cess - _TC",
"charge_type": "On Net Total",
"cost_center": "_Test Cost Center - _TC",
"description": "S&H Education Cess",
"doctype": "Sales Taxes and Charges",
"rate": 3,
},
]
)
si.insert()

self.assertEqual(si.net_total, 4600)

self.assertEqual(si.get("taxes")[0].tax_amount, 502.41)
self.assertEqual(si.get("taxes")[0].total, 5102.41)

self.assertEqual(si.get("taxes")[1].tax_amount, 197.80)
self.assertEqual(si.get("taxes")[1].total, 5300.21)

for i in range(len(expected_taxes)):
for key in expected_taxes[i]:
self.assertEqual(expected_taxes[i][key], si.get("taxes")[i].get(key))
self.assertEqual(si.get("taxes")[2].tax_amount, 375.36)
self.assertEqual(si.get("taxes")[2].total, 5675.57)

self.assertEqual(si.get("base_total_taxes_and_charges"), 115.92)
self.assertEqual(si.grand_total, 5675.57)
self.assertEqual(si.rounding_adjustment, 0.43)
self.assertEqual(si.rounded_total, 5676.0)

def test_tax_calculation_with_multiple_items_and_discount(self):
si = create_sales_invoice(qty=1, rate=75, do_not_save=True)
Expand Down
10 changes: 5 additions & 5 deletions erpnext/stock/doctype/item/test_item.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ def test_item_tax_template(self):
{
"item_code": "_Test Item With Item Tax Template",
"tax_category": "_Test Tax Category 2",
"item_tax_template": "",
"item_tax_template": None,
},
{
"item_code": "_Test Item Inherit Group Item Tax Template 1",
Expand All @@ -178,7 +178,7 @@ def test_item_tax_template(self):
{
"item_code": "_Test Item Inherit Group Item Tax Template 1",
"tax_category": "_Test Tax Category 2",
"item_tax_template": "",
"item_tax_template": None,
},
{
"item_code": "_Test Item Inherit Group Item Tax Template 2",
Expand All @@ -193,7 +193,7 @@ def test_item_tax_template(self):
{
"item_code": "_Test Item Inherit Group Item Tax Template 2",
"tax_category": "_Test Tax Category 2",
"item_tax_template": "",
"item_tax_template": None,
},
{
"item_code": "_Test Item Override Group Item Tax Template",
Expand All @@ -208,12 +208,12 @@ def test_item_tax_template(self):
{
"item_code": "_Test Item Override Group Item Tax Template",
"tax_category": "_Test Tax Category 2",
"item_tax_template": "",
"item_tax_template": None,
},
]

expected_item_tax_map = {
"": {},
None: {},
"_Test Account Excise Duty @ 10 - _TC": {"_Test Account Excise Duty - _TC": 10},
"_Test Account Excise Duty @ 12 - _TC": {"_Test Account Excise Duty - _TC": 12},
"_Test Account Excise Duty @ 15 - _TC": {"_Test Account Excise Duty - _TC": 15},
Expand Down
1 change: 0 additions & 1 deletion erpnext/stock/get_item_details.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -610,7 +610,6 @@ def _get_item_tax_template(args, taxes, out=None, for_validate=False):

# all templates have validity and no template is valid
if not taxes_with_validity and (not taxes_with_no_validity):
out["item_tax_template"] = ""
return None

# do not change if already a valid template
Expand Down

0 comments on commit 70d99ee

Please sign in to comment.