schuetz3/schuetz3-erpnext
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #17290 from alyf-de/company_test

Browse Source 
fix(test): provide a helpful error message
This commit is contained in:
Nabin Hait
2019-04-19 13:10:46 +05:30
committed by GitHub
parent 95fce2395a df16cdcf31
commit 2c607e5562
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
erpnext/setup/doctype/company/test_company.py
View File

@@ -4,6 +4,8 @@ from __future__ import unicode_literals
import frappe import frappe
import unittest import unittest
import json
from frappe import _
from frappe.utils import random_string from frappe.utils import random_string
from erpnext.accounts.doctype.account.chart_of_accounts.chart_of_accounts import get_charts_for_country from erpnext.accounts.doctype.account.chart_of_accounts.chart_of_accounts import get_charts_for_country
@@ -78,7 +80,10 @@ class TestCompany(unittest.TestCase):
if account_type in ["Bank", "Cash"]: if account_type in ["Bank", "Cash"]:
filters["is_group"] = 1 filters["is_group"] = 1
self.assertTrue(frappe.get_all("Account", filters)) has_matching_accounts = frappe.get_all("Account", filters)
error_message = _("No Account matched these filters: {}".format(json.dumps(filters)))
self.assertTrue(has_matching_accounts, msg=error_message)
finally: finally:
self.delete_mode_of_payment(template) self.delete_mode_of_payment(template)
frappe.delete_doc("Company", template) frappe.delete_doc("Company", template)