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

fix assign test case

Browse source
This commit is contained in:
Rushabh Mehta 2014-06-04 17:41:06 +05:30
parent 9bb0ebf233
commit 66cc4f30d2
3 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
frappe/core/doctype/customize_form/test_customize_form.py
View file

@ -47,7 +47,7 @@ class TestCustomizeForm(unittest.TestCase):
d = self.get_customize_form("User")
self.assertEquals(d.doc_type, "User")
self.assertEquals(len(d.get("customize_form_fields")), 53)
self.assertEquals(len(d.get("customize_form_fields")), 54)
self.assertEquals(d.get("customize_form_fields")[-1].fieldname, "test_custom_field")
self.assertEquals(d.get("customize_form_fields", {"fieldname": "location"})[0].in_list_view, 1)

4
frappe/public/js/frappe/form/assign_to.js
View file

@ -41,9 +41,9 @@ frappe.ui.form.AssignTo = Class.extend({
if(d && d.length) {
for(var i=0; i<d.length; i++) {
var info = frappe.user_info(d[i].name);
var info = frappe.user_info(d[i].owner);
info.owner = d[i];
info.image = frappe.user_info(d[i].name).image;
info.image = frappe.user_info(d[i].owner).image;
info.description = d[i].description || "";
$(repl('<div class="media">\

5
frappe/tests/test_assign.py
View file

@ -16,7 +16,8 @@ class TestAssign(unittest.TestCase):
"name": todo.name,
"description": todo.description,
})
self.assertTrue("test@example.com" in [d.name for d in added])
self.assertTrue("test@example.com" in [d.owner for d in added])
removed = frappe.widgets.form.assign_to.remove(todo.doctype, todo.name, "test@example.com")
self.assertTrue("test@example.com" not in [d.name for d in removed])
self.assertTrue("test@example.com" not in [d.owner for d in removed])