vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

test: Use new_doc instead of get_doc

Browse source
This commit is contained in:
Gavin D'souza 2022-04-20 11:58:35 +05:30
parent 656e03444e
commit 9839c4d5d6
2 changed files with 2 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
frappe/core/doctype/file/test_file.py
View file

@ -396,9 +396,8 @@ class TestFile(FrappeTestCase):
self.assertRaises(OSError, file1.save)
def test_file_url_validation(self):
test_file: File = frappe.get_doc(
{"doctype": "File", "file_name": "logo", "file_url": "https://frappe.io/files/frappe.png"}
)
test_file: File = frappe.new_doc("File")
test_file.update({"file_name": "logo", "file_url": "https://frappe.io/files/frappe.png"})
self.assertIsNone(test_file.validate())

2
frappe/model/create_new.py
View file

@ -23,8 +23,6 @@ def get_new_doc(doctype, parent_doc=None, parentfield=None, as_dict=False):
doc = copy.deepcopy(frappe.local.new_doc_templates[doctype])
# doc = make_new_doc(doctype)
set_dynamic_default_values(doc, parent_doc, parentfield)
if as_dict: