From 66cc4f30d232beb6abbe21769fd9984c211382b2 Mon Sep 17 00:00:00 2001
From: Rushabh Mehta <rmehta@gmail.com>
Date: Wed, 4 Jun 2014 17:41:06 +0530
Subject: [PATCH] fix assign test case

---
 frappe/core/doctype/customize_form/test_customize_form.py | 2 +-
 frappe/public/js/frappe/form/assign_to.js                 | 4 ++--
 frappe/tests/test_assign.py                               | 5 +++--
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/frappe/core/doctype/customize_form/test_customize_form.py b/frappe/core/doctype/customize_form/test_customize_form.py
index f6167f6dce..fe05a22935 100644
--- a/frappe/core/doctype/customize_form/test_customize_form.py
+++ b/frappe/core/doctype/customize_form/test_customize_form.py
@@ -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)
 
diff --git a/frappe/public/js/frappe/form/assign_to.js b/frappe/public/js/frappe/form/assign_to.js
index 66f8afb5db..7fc7a9dd36 100644
--- a/frappe/public/js/frappe/form/assign_to.js
+++ b/frappe/public/js/frappe/form/assign_to.js
@@ -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">\
diff --git a/frappe/tests/test_assign.py b/frappe/tests/test_assign.py
index ac96d9daf7..eefdfc2bc9 100644
--- a/frappe/tests/test_assign.py
+++ b/frappe/tests/test_assign.py
@@ -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])