From 208a06f5f5c4bb0f5a1f10493c8f996fec59da9b Mon Sep 17 00:00:00 2001 From: Rushabh Mehta Date: Fri, 20 Mar 2015 15:57:25 +0530 Subject: [PATCH] [test] newsletter list --- erpnext/crm/doctype/newsletter_list/newsletter_list.py | 4 ++-- erpnext/crm/doctype/newsletter_list/test_newsletter_list.py | 4 ---- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/erpnext/crm/doctype/newsletter_list/newsletter_list.py b/erpnext/crm/doctype/newsletter_list/newsletter_list.py index a1e0cc83a4c..973955d75b5 100644 --- a/erpnext/crm/doctype/newsletter_list/newsletter_list.py +++ b/erpnext/crm/doctype/newsletter_list/newsletter_list.py @@ -48,8 +48,8 @@ class NewsletterList(Document): return self.total_subscribers def on_trash(self): - for d in frappe.get_all("Newsletter List Subscriber", {"newsletter_list": self.name}): - frappe.delete_doc(d.doctype, d.name) + for d in frappe.get_all("Newsletter List Subscriber", "name", {"newsletter_list": self.name}): + frappe.delete_doc("Newsletter List Subscriber", d.name) @frappe.whitelist() def import_from(name, doctype): diff --git a/erpnext/crm/doctype/newsletter_list/test_newsletter_list.py b/erpnext/crm/doctype/newsletter_list/test_newsletter_list.py index d5d0d19c141..c715d8557c0 100644 --- a/erpnext/crm/doctype/newsletter_list/test_newsletter_list.py +++ b/erpnext/crm/doctype/newsletter_list/test_newsletter_list.py @@ -10,8 +10,6 @@ import unittest class TestNewletterList(unittest.TestCase): def test_import(self): - frappe.delete_doc("Newsletter List", "_Test Newsletter List 1") - new_list = frappe.get_doc({ "doctype": "Newsletter List", "title": "_Test Newsletter List 1" @@ -21,8 +19,6 @@ class TestNewletterList(unittest.TestCase): added = new_list.import_from("Lead") - print added - self.assertEquals(added, n_leads) frappe.delete_doc("Newsletter List", new_list.name)