diff --git a/erpnext/projects/doctype/project/test_project.py b/erpnext/projects/doctype/project/test_project.py index 4f284860e3e..1defd5f4733 100644 --- a/erpnext/projects/doctype/project/test_project.py +++ b/erpnext/projects/doctype/project/test_project.py @@ -278,6 +278,164 @@ class TestProject(ERPNextTestSuite): project.save() self.assertEqual(project.status, "Completed") + def _project_with_tasks(self, method, count): + name = f"_Test PercentComplete {frappe.generate_hash(length=8)}" + project = frappe.get_doc( + { + "doctype": "Project", + "project_name": name, + "status": "Open", + "percent_complete_method": method, + "company": "_Test Company", + "expected_start_date": nowdate(), + } + ).insert() + task_names = [] + for i in range(count): + task = frappe.get_doc( + { + "doctype": "Task", + "subject": f"{name} Task {i}", + "project": project.name, + "status": "Open", + "exp_start_date": nowdate(), + "exp_end_date": nowdate(), + } + ).insert() + task_names.append(task.name) + return project, task_names + + def test_percent_complete_by_task_completion(self): + project, tasks = self._project_with_tasks("Task Completion", 4) + + frappe.db.set_value("Task", tasks[0], "status", "Completed") + project.update_percent_complete() + self.assertEqual(project.percent_complete, 25) # 1 of 4 + + for task in tasks: + frappe.db.set_value("Task", task, "status", "Completed") + project.update_percent_complete() + self.assertEqual(project.percent_complete, 100) + self.assertEqual(project.status, "Completed") # 100% flips status to Completed + + # reopening a task drops below 100% and flips status back to Open + frappe.db.set_value("Task", tasks[0], "status", "Open") + project.update_percent_complete() + self.assertEqual(project.percent_complete, 75) + self.assertEqual(project.status, "Open") + + # a Cancelled project keeps its status regardless of completion + project.status = "Cancelled" + for task in tasks: + frappe.db.set_value("Task", task, "status", "Completed") + project.update_percent_complete() + self.assertEqual(project.percent_complete, 100) + self.assertEqual(project.status, "Cancelled") + + def test_percent_complete_by_task_progress(self): + project, tasks = self._project_with_tasks("Task Progress", 2) + + frappe.db.set_value("Task", tasks[0], "progress", 50) + frappe.db.set_value("Task", tasks[1], "progress", 100) + project.update_percent_complete() + self.assertEqual(project.percent_complete, 75) # (50 + 100) / 2 + + def test_percent_complete_by_task_weight(self): + project, tasks = self._project_with_tasks("Task Weight", 2) + + frappe.db.set_value("Task", tasks[0], {"progress": 100, "task_weight": 3}) + frappe.db.set_value("Task", tasks[1], {"progress": 0, "task_weight": 1}) + project.update_percent_complete() + self.assertEqual(project.percent_complete, 75) # 100 * 3/4 + 0 * 1/4 + + def test_create_duplicate_project_copies_tasks(self): + from erpnext.projects.doctype.project.project import create_duplicate_project + + source, tasks = self._project_with_tasks("Task Completion", 2) + new_name = f"{source.project_name} Copy" + + create_duplicate_project(frappe.as_json(source.as_dict()), new_name) + + # Project is named by series, so look the copy up by its project_name + new_project = frappe.db.get_value("Project", {"project_name": new_name}) + self.assertTrue(new_project) + copied_tasks = frappe.get_all("Task", filters={"project": new_project}) + self.assertEqual(len(copied_tasks), len(tasks)) + + def test_create_duplicate_project_rejects_same_name(self): + from erpnext.projects.doctype.project.project import create_duplicate_project + + source, _ = self._project_with_tasks("Task Completion", 1) + self.assertRaises( + frappe.ValidationError, + create_duplicate_project, + frappe.as_json(source.as_dict()), + source.name, + ) + + def test_set_project_status_updates_project_and_tasks(self): + from erpnext.projects.doctype.project.project import set_project_status + + project, tasks = self._project_with_tasks("Task Completion", 2) + + set_project_status(project.name, "Cancelled") + + self.assertEqual(frappe.db.get_value("Project", project.name, "status"), "Cancelled") + for task in tasks: + self.assertEqual(frappe.db.get_value("Task", task, "status"), "Cancelled") + + def test_set_project_status_rejects_invalid_status(self): + from erpnext.projects.doctype.project.project import set_project_status + + project, _ = self._project_with_tasks("Task Completion", 1) + self.assertRaises(frappe.ValidationError, set_project_status, project.name, "Open") + + def test_costing_rollup_from_sales_documents(self): + from erpnext.accounts.doctype.sales_invoice.test_sales_invoice import create_sales_invoice + from erpnext.projects.doctype.project.project import update_costing_and_billing + from erpnext.selling.doctype.sales_order.test_sales_order import make_sales_order + + project = make_project({"project_name": f"_Test Costing Rollup {frappe.generate_hash(length=6)}"}) + + sales_order = make_sales_order(do_not_save=True) + sales_order.project = project.name + sales_order.insert() + sales_order.submit() + + sales_invoice = create_sales_invoice(do_not_submit=True) + sales_invoice.project = project.name + sales_invoice.submit() + + update_costing_and_billing(project.name) + project.reload() + + self.assertEqual(project.total_sales_amount, sales_order.base_net_total) + self.assertEqual(project.total_billed_amount, sales_invoice.base_net_total) + # with no costing/purchase/material expense, gross margin is the billed amount in full + self.assertEqual(project.gross_margin, sales_invoice.base_net_total) + self.assertEqual(project.per_gross_margin, 100) + + def test_consumed_material_cost_from_stock_entry(self): + from erpnext.stock.doctype.stock_entry.stock_entry_utils import make_stock_entry + + project = make_project({"project_name": f"_Test Consumed Cost {frappe.generate_hash(length=6)}"}) + + # receive stock, then issue it against the project so it counts as consumed + make_stock_entry(item_code="_Test Item", qty=10, to_warehouse="_Test Warehouse - _TC", rate=100) + issue = make_stock_entry( + item_code="_Test Item", qty=4, from_warehouse="_Test Warehouse - _TC", do_not_save=True + ) + issue.project = project.name + for row in issue.items: + row.project = project.name + issue.insert() + issue.submit() + issue.reload() + + project.set_consumed_material_cost() + self.assertEqual(project.total_consumed_material_cost, sum(row.amount for row in issue.items)) + self.assertGreater(project.total_consumed_material_cost, 0) + def get_project(name, template): project = frappe.get_doc( diff --git a/erpnext/projects/doctype/project_template/test_project_template.py b/erpnext/projects/doctype/project_template/test_project_template.py index 7677abd1890..d13b12400fe 100644 --- a/erpnext/projects/doctype/project_template/test_project_template.py +++ b/erpnext/projects/doctype/project_template/test_project_template.py @@ -8,7 +8,19 @@ from erpnext.tests.utils import ERPNextTestSuite class TestProjectTemplate(ERPNextTestSuite): - pass + def test_dependency_task_must_be_in_template(self): + dependency = create_task("_Test PT Dependency", is_template=1) + dependent = create_task("_Test PT Dependent", is_template=1, depends_on=dependency.name) + + template = frappe.get_doc(doctype="Project Template", name="_Test PT Missing Dependency") + template.append("tasks", {"task": dependent.name}) + # the dependency task is not in the template's task list + self.assertRaises(frappe.ValidationError, template.insert) + + # adding the dependency task makes the template valid + template.append("tasks", {"task": dependency.name}) + template.insert() + self.assertTrue(frappe.db.exists("Project Template", template.name)) def make_project_template(project_template_name, project_tasks=None):