test: instructor for courses

lms/lms/doctype/lms_certificate/test_lms_certificate.py

@@ -12,7 +12,13 @@ from lms.lms.doctype.lms_course.test_lms_course import new_course
 
 class TestLMSCertificate(unittest.TestCase):
 	def test_certificate_creation(self):
-		course = new_course("Test Certificate", {"enable_certification": 1, "expiry": 2})
+		course = new_course(
+			"Test Certificate",
+			{
+				"enable_certification": 1,
+				"expiry": 2,
+			},
+		)
 		certificate = create_certificate(course.name)
 
 		self.assertEqual(certificate.member, "Administrator")

lms/lms/doctype/lms_course/test_lms_course.py

@@ -68,12 +68,19 @@ def new_course(title, additional_filters=None):
 		return frappe.get_doc("LMS Course", course)
 	else:
 		create_evaluator()
+		user = frappe.db.get_value(
+			"User",
+			{
+				"user_type": "System User",
+			},
+		)
 		filters = {
 			"title": title,
 			"short_introduction": title,
 			"description": title,
 			"video_link": "https://youtu.be/pEbIhUySqbk",
 			"image": "/assets/lms/images/course-home.png",
+			"instructors": [{"instructor": user}],
 		}
 
 		if additional_filters:

lms/lms/test_utils.py

@@ -22,6 +22,8 @@ class TestUtils(unittest.TestCase):
 		)
 
 	def test_evaluation_details(self):
+		user = new_user("Eval", "eval@test.com")
+
 		course = new_course(
 			"Test Evaluation Details",
 			{
@@ -30,9 +32,9 @@ class TestUtils(unittest.TestCase):
 				"evaluator": "evaluator@example.com",
 				"max_attempts": 3,
 				"duration": 2,
+				"instructors": [{"instructor": user.name}],
 			},
 		)
-		user = new_user("Eval", "eval@test.com")
 
 		# Two evaluations failed within max attempts. Check eligibility for a third evaluation
 		create_evaluation(user.name, course.name, getdate("21-03-2022"), 0.4, "Fail")