fix(reportview): test
This commit is contained in:
parent
82399ddfba
commit
337bdc976c
2 changed files with 8 additions and 4 deletions
|
|
@ -59,9 +59,7 @@ def validate_args(data):
|
|||
return data
|
||||
|
||||
def validate_fields(data):
|
||||
if update_wildcard_field_param(data):
|
||||
# no need to validate wildcard fields
|
||||
return
|
||||
wildcard = update_wildcard_field_param(data)
|
||||
|
||||
for field in data.fields or []:
|
||||
fieldname = extract_fieldname(field)
|
||||
|
|
@ -71,7 +69,10 @@ def validate_fields(data):
|
|||
meta, df = get_meta_and_docfield(fieldname, data)
|
||||
|
||||
if not df:
|
||||
raise_invalid_field(fieldname)
|
||||
if wildcard:
|
||||
continue
|
||||
else:
|
||||
raise_invalid_field(fieldname)
|
||||
|
||||
# remove the field from the query if the report hide flag is set and current view is Report
|
||||
if df.report_hide and data.view == 'Report':
|
||||
|
|
|
|||
|
|
@ -17,6 +17,9 @@ from frappe.utils.testutils import add_custom_field, clear_custom_fields
|
|||
test_dependencies = ['User', 'Blog Post', 'Blog Category', 'Blogger']
|
||||
|
||||
class TestReportview(unittest.TestCase):
|
||||
def setUp(self):
|
||||
frappe.set_user("Administrator")
|
||||
|
||||
def test_basic(self):
|
||||
self.assertTrue({"name":"DocType"} in DatabaseQuery("DocType").execute(limit_page_length=None))
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue