From 5fa8bdd40c03a14b9aa6a6c90841a058481c15da Mon Sep 17 00:00:00 2001 From: pateljannat Date: Tue, 15 Jun 2021 13:09:48 +0530 Subject: [PATCH] fix: invite request test, removed print statements and unused classes' --- community/lms/doctype/invite_request/test_invite_request.py | 2 +- community/lms/widgets/RenderBatch.html | 2 +- community/www/batch/progress.py | 4 ---- 3 files changed, 2 insertions(+), 6 deletions(-) diff --git a/community/lms/doctype/invite_request/test_invite_request.py b/community/lms/doctype/invite_request/test_invite_request.py index 31bc7afe..20fe531d 100644 --- a/community/lms/doctype/invite_request/test_invite_request.py +++ b/community/lms/doctype/invite_request/test_invite_request.py @@ -18,7 +18,7 @@ class TestInviteRequest(unittest.TestCase): filters={"invite_email": "test_invite@example.com"}, fieldname=["invite_email", "status", "signup_email"], as_dict=True) - self.assertEqual(invite.status, "Pending") + self.assertEqual(invite.status, "Approved") self.assertEqual(invite.signup_email, None) def test_create_invite_request_update(self): diff --git a/community/lms/widgets/RenderBatch.html b/community/lms/widgets/RenderBatch.html index 4a9431bb..cf23b7a4 100644 --- a/community/lms/widgets/RenderBatch.html +++ b/community/lms/widgets/RenderBatch.html @@ -1,6 +1,6 @@
-
Session every {{batch.sessions_on}}
+
Session every {{batch.sessions_on}}
{{frappe.utils.format_time(batch.start_time, "short")}} - {{frappe.utils.format_time(batch.end_time, "short")}}
diff --git a/community/www/batch/progress.py b/community/www/batch/progress.py index 13578ac1..70024864 100644 --- a/community/www/batch/progress.py +++ b/community/www/batch/progress.py @@ -14,13 +14,11 @@ def get_context(context): context.exercise = exercise context.report = BatchReport(context.course, context.batch) - print(context.report) class BatchReport: def __init__(self, course, batch): self.submissions = get_submissions(batch) self.exercises = self.get_exercises(course.name) - print(self.submissions) self.submissions_by_exercise = defaultdict(list) for s in self.submissions: self.submissions_by_exercise[s.exercise].append(s) @@ -35,7 +33,6 @@ def get_submissions(batch): students = batch.get_students() students_map = {s.email: s for s in students} names, values = nparams("s", students_map.keys()) - print(students, names, values) sql = """ select owner, exercise, name, solution, creation, image from ( @@ -46,7 +43,6 @@ def get_submissions(batch): """.format(names) data = frappe.db.sql(sql, values=values, as_dict=True) - print(data) for row in data: row['owner'] = students_map[row['owner']] return data