From 9a2717b662c7dd3fca54e8b8f2e526454ad72d83 Mon Sep 17 00:00:00 2001
From: CruiseDevice
Date: Fri, 19 Feb 2021 15:05:42 +0530
Subject: Fix upload assignment issue for multiple attempts

---
 yaksh/models.py | 9 +++++----
 yaksh/views.py  | 6 +++---
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/yaksh/models.py b/yaksh/models.py
index 11ddf8a..6a745d1 100644
--- a/yaksh/models.py
+++ b/yaksh/models.py
@@ -122,7 +122,7 @@ MOD_GROUP_NAME = 'moderator'
 
 def get_assignment_dir(instance, filename):
     folder_name = instance.course.name.replace(" ", "_")
-    sub_folder_name = instance.question_paper.quiz.description.replace(
+    sub_folder_name = instance.answer_paper.question_paper.quiz.description.replace(
         " ", "_")
     return os.sep.join((folder_name, sub_folder_name, instance.user.username,
                         str(instance.assignmentQuestion.id),
@@ -2661,9 +2661,10 @@ class AssignmentUploadManager(models.Manager):
 class AssignmentUpload(models.Model):
     user = models.ForeignKey(User, on_delete=models.CASCADE)
     assignmentQuestion = models.ForeignKey(Question, on_delete=models.CASCADE)
-    assignmentFile = models.FileField(upload_to=get_assignment_dir, max_length=255)
-    question_paper = models.ForeignKey(QuestionPaper, blank=True, null=True,
-                                       on_delete=models.CASCADE)
+    assignmentFile = models.FileField(upload_to=get_assignment_dir,
+                                      max_length=255)
+    answer_paper = models.ForeignKey(AnswerPaper, blank=True, null=True,
+                                     on_delete=models.CASCADE)
     course = models.ForeignKey(Course, null=True, blank=True,
                                on_delete=models.CASCADE)
     objects = AssignmentUploadManager()
diff --git a/yaksh/views.py b/yaksh/views.py
index 1965191..ceebda6 100644
--- a/yaksh/views.py
+++ b/yaksh/views.py
@@ -722,7 +722,7 @@ def show_question(request, question, paper, error_message=None,
                             assignmentQuestion_id=question.id,
                             course_id=course_id,
                             user=request.user,
-                            question_paper_id=paper.question_paper_id
+                            answer_paper=paper
                         )
     files = FileUpload.objects.filter(question_id=question.id, hide=False)
     course = Course.objects.get(id=course_id)
@@ -861,7 +861,7 @@ def check(request, q_id, attempt_num=None, questionpaper_id=None,
                 assignment_files = AssignmentUpload.objects.filter(
                     assignmentQuestion=current_question, course_id=course_id,
                     assignmentFile__icontains=fname, user=user,
-                    question_paper=questionpaper_id)
+                    answer_paper=paper.id)
                 if assignment_files.exists():
                     assign_file = assignment_files.first()
                     if os.path.exists(assign_file.assignmentFile.path):
@@ -870,7 +870,7 @@ def check(request, q_id, attempt_num=None, questionpaper_id=None,
                 AssignmentUpload.objects.create(
                     user=user, assignmentQuestion=current_question,
                     course_id=course_id,
-                    assignmentFile=fname, question_paper_id=questionpaper_id
+                    assignmentFile=fname, answer_paper_id=paper.id
                 )
             user_answer = 'ASSIGNMENT UPLOADED'
             if not current_question.grade_assignment_upload:
-- 
cgit 


From 3e23a631e9c8e25fd13c7f1f8b0fed54d0c34b8f Mon Sep 17 00:00:00 2001
From: CruiseDevice
Date: Fri, 26 Feb 2021 08:37:27 +0530
Subject: Fix assignment download on grade user interface

---
 yaksh/admin.py          |  3 ++-
 yaksh/base_evaluator.py |  2 +-
 yaksh/models.py         | 26 ++++++++++++++------------
 yaksh/views.py          | 36 ++++++++++++++++--------------------
 4 files changed, 33 insertions(+), 34 deletions(-)

diff --git a/yaksh/admin.py b/yaksh/admin.py
index e98c7c5..011e24f 100644
--- a/yaksh/admin.py
+++ b/yaksh/admin.py
@@ -2,7 +2,7 @@ from yaksh.models import Question, Quiz, QuestionPaper, Profile
 from yaksh.models import (TestCase, StandardTestCase, StdIOBasedTestCase,
                           Course, AnswerPaper, CourseStatus, LearningModule,
                           Lesson, Post, Comment, Topic, TableOfContents,
-                          LessonQuizAnswer, Answer
+                          LessonQuizAnswer, Answer, AssignmentUpload
                           )
 from django.contrib import admin
 
@@ -64,3 +64,4 @@ admin.site.register(Topic)
 admin.site.register(TableOfContents)
 admin.site.register(LessonQuizAnswer)
 admin.site.register(Answer)
+admin.site.register(AssignmentUpload)
\ No newline at end of file
diff --git a/yaksh/base_evaluator.py b/yaksh/base_evaluator.py
index 567d5ad..c7b5c54 100644
--- a/yaksh/base_evaluator.py
+++ b/yaksh/base_evaluator.py
@@ -44,7 +44,7 @@ class BaseEvaluator(object):
         """Returns a string without any null and substitute characters"""
         stripped = ""
         for c in string:
-            if ord(c) is not 26 and ord(c) is not 0:
+            if ord(c) != 26 and ord(c) != 0:
                 stripped = stripped + c
         return ''.join(stripped)
 
diff --git a/yaksh/models.py b/yaksh/models.py
index 6a745d1..b80f98c 100644
--- a/yaksh/models.py
+++ b/yaksh/models.py
@@ -121,10 +121,10 @@ MOD_GROUP_NAME = 'moderator'
 
 
 def get_assignment_dir(instance, filename):
-    folder_name = instance.course.name.replace(" ", "_")
-    sub_folder_name = instance.answer_paper.question_paper.quiz.description.replace(
-        " ", "_")
-    return os.sep.join((folder_name, sub_folder_name, instance.user.username,
+    folder = str(instance.answer_paper.course.id)
+    sub_folder = str(instance.answer_paper.question_paper.quiz.id)
+    user = instance.answer_paper.user.username
+    return os.sep.join((folder, sub_folder, user,
                         str(instance.assignmentQuestion.id),
                         filename
                         ))
@@ -1439,7 +1439,7 @@ class Question(models.Model):
                                       for file in files]
         if self.type == "upload":
             assignment_files = AssignmentUpload.objects.filter(
-                assignmentQuestion=self, user=user
+                assignmentQuestion=self
                 )
             if assignment_files:
                 metadata['assign_files'] = [(file.assignmentFile.path, False)
@@ -2642,16 +2642,16 @@ class AssignmentUploadManager(models.Manager):
     def get_assignments(self, qp, que_id=None, user_id=None, course_id=None):
         if que_id and user_id:
             assignment_files = AssignmentUpload.objects.filter(
-                        assignmentQuestion_id=que_id, user_id=user_id,
-                        question_paper=qp, course_id=course_id
+                        assignmentQuestion_id=que_id, answer_paper__user_id=user_id,
+                        answer_paper__question_paper=qp, answer_paper__course_id=course_id
                         )
             file_name = User.objects.get(id=user_id).get_full_name()
         else:
             assignment_files = AssignmentUpload.objects.filter(
-                        question_paper=qp, course_id=course_id
+                        answer_paper__question_paper=qp, answer_paper__course_id=course_id
                         )
             file_name = "{0}_Assignment_files".format(
-                            assignment_files[0].course.name
+                            assignment_files[0].answer_paper.course.name
                             )
 
         return assignment_files, file_name
@@ -2659,16 +2659,18 @@ class AssignmentUploadManager(models.Manager):
 
 ##############################################################################
 class AssignmentUpload(models.Model):
-    user = models.ForeignKey(User, on_delete=models.CASCADE)
     assignmentQuestion = models.ForeignKey(Question, on_delete=models.CASCADE)
     assignmentFile = models.FileField(upload_to=get_assignment_dir,
                                       max_length=255)
     answer_paper = models.ForeignKey(AnswerPaper, blank=True, null=True,
                                      on_delete=models.CASCADE)
-    course = models.ForeignKey(Course, null=True, blank=True,
-                               on_delete=models.CASCADE)
+    upload_date = models.DateTimeField(auto_now=True)
+
     objects = AssignmentUploadManager()
 
+    def __str__(self):
+        return f'Assignment File of the user {self.answer_paper.user}'
+
 
 ##############################################################################
 class TestCase(models.Model):
diff --git a/yaksh/views.py b/yaksh/views.py
index ceebda6..7d4f788 100644
--- a/yaksh/views.py
+++ b/yaksh/views.py
@@ -720,8 +720,6 @@ def show_question(request, question, paper, error_message=None,
     if question.type == 'upload':
         assignment_files = AssignmentUpload.objects.filter(
                             assignmentQuestion_id=question.id,
-                            course_id=course_id,
-                            user=request.user,
                             answer_paper=paper
                         )
     files = FileUpload.objects.filter(question_id=question.id, hide=False)
@@ -856,22 +854,15 @@ def check(request, q_id, attempt_num=None, questionpaper_id=None,
                     course_id=course_id, module_id=module_id,
                     previous_question=current_question
                 )
+            uploaded_files = []
             for fname in assignment_filename:
                 fname._name = fname._name.replace(" ", "_")
-                assignment_files = AssignmentUpload.objects.filter(
-                    assignmentQuestion=current_question, course_id=course_id,
-                    assignmentFile__icontains=fname, user=user,
-                    answer_paper=paper.id)
-                if assignment_files.exists():
-                    assign_file = assignment_files.first()
-                    if os.path.exists(assign_file.assignmentFile.path):
-                        os.remove(assign_file.assignmentFile.path)
-                    assign_file.delete()
-                AssignmentUpload.objects.create(
-                    user=user, assignmentQuestion=current_question,
-                    course_id=course_id,
-                    assignmentFile=fname, answer_paper_id=paper.id
-                )
+                uploaded_files.append(AssignmentUpload(
+                                    assignmentQuestion=current_question,
+                                    assignmentFile=fname,
+                                    answer_paper_id=paper.id
+                                ))
+            AssignmentUpload.objects.bulk_create(uploaded_files)
             user_answer = 'ASSIGNMENT UPLOADED'
             if not current_question.grade_assignment_upload:
                 new_answer = Answer(
@@ -1874,6 +1865,8 @@ def grade_user(request, quiz_id=None, user_id=None, attempt_number=None,
     and update all their marks and also give comments for each paper.
     """
     current_user = request.user
+    papers = AnswerPaper.objects.filter(user=current_user)
+    print('Paper: ', papers)
     if not is_moderator(current_user):
         raise Http404('You are not allowed to view this page!')
     if not course_id:
@@ -1898,7 +1891,8 @@ def grade_user(request, quiz_id=None, user_id=None, attempt_number=None,
                 course.is_teacher(current_user):
             raise Http404('This course does not belong to you')
         has_quiz_assignments = AssignmentUpload.objects.filter(
-            course_id=course_id, question_paper_id__in=questionpaper_id
+                answer_paper__course_id=course_id,
+                answer_paper__question_paper_id__in=questionpaper_id
             ).exists()
         context = {
             "users": user_details,
@@ -1917,9 +1911,11 @@ def grade_user(request, quiz_id=None, user_id=None, attempt_number=None,
                     attempt_number = attempts[0].attempt_number
             except IndexError:
                 raise Http404('No attempts for paper')
+
             has_user_assignments = AssignmentUpload.objects.filter(
-                course_id=course_id, question_paper_id__in=questionpaper_id,
-                user_id=user_id
+                answer_paper__course_id=course_id,
+                answer_paper__question_paper_id__in=questionpaper_id,
+                answer_paper__user_id=user_id
                 ).exists()
             user = User.objects.get(id=user_id)
             data = AnswerPaper.objects.get_user_data(
@@ -2383,7 +2379,7 @@ def download_assignment_file(request, quiz_id, course_id,
     zipfile_name = string_io()
     zip_file = zipfile.ZipFile(zipfile_name, "w")
     for f_name in assignment_files:
-        folder = f_name.user.get_full_name().replace(" ", "_")
+        folder = f_name.answer_paper.user.get_full_name().replace(" ", "_")
         sub_folder = f_name.assignmentQuestion.summary.replace(" ", "_")
         folder_name = os.sep.join((folder, sub_folder, os.path.basename(
                         f_name.assignmentFile.name))
-- 
cgit 


From 2d46e6799672cc133829ab72a88c645e54087d98 Mon Sep 17 00:00:00 2001
From: CruiseDevice
Date: Fri, 26 Feb 2021 12:54:37 +0530
Subject: Resolve comments

---
 yaksh/models.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/yaksh/models.py b/yaksh/models.py
index b80f98c..ca0f2d8 100644
--- a/yaksh/models.py
+++ b/yaksh/models.py
@@ -121,8 +121,8 @@ MOD_GROUP_NAME = 'moderator'
 
 
 def get_assignment_dir(instance, filename):
-    folder = str(instance.answer_paper.course.id)
-    sub_folder = str(instance.answer_paper.question_paper.quiz.id)
+    folder = str(instance.answer_paper.course_id)
+    sub_folder = str(instance.answer_paper.question_paper.quiz_id)
     user = instance.answer_paper.user.username
     return os.sep.join((folder, sub_folder, user,
                         str(instance.assignmentQuestion.id),
-- 
cgit 


From 0cc290e6207006339b7d553127b432542284eea8 Mon Sep 17 00:00:00 2001
From: CruiseDevice
Date: Fri, 26 Feb 2021 15:05:02 +0530
Subject: Fix Tests

---
 yaksh/test_models.py | 60 ++++++++++++++++++++++++++++++++++++----------------
 1 file changed, 42 insertions(+), 18 deletions(-)

diff --git a/yaksh/test_models.py b/yaksh/test_models.py
index a424b36..86890c0 100644
--- a/yaksh/test_models.py
+++ b/yaksh/test_models.py
@@ -2201,17 +2201,26 @@ class TestCaseTestCases(unittest.TestCase):
 
 
 class AssignmentUploadTestCases(unittest.TestCase):
+
     def setUp(self):
-        self.user1 = User.objects.get(username="creator")
-        self.user1.first_name = "demo"
-        self.user1.last_name = "user"
-        self.user1.save()
-        self.user2 = User.objects.get(username="demo_user3")
-        self.user2.first_name = "demo"
-        self.user2.last_name = "user3"
-        self.user2.save()
-        self.quiz = Quiz.objects.get(description="demo quiz 1")
-        self.course = Course.objects.get(name="Python Course")
+        self.user1 = User.objects.create_user(username='creator1',
+                                    password='demo',
+                                    email='demo@test1.com')
+        self.user2 = User.objects.create_user(username='creator2',
+                                    password='demo',
+                                    email='demo@test2.com')
+        self.quiz = Quiz.objects.create(
+        start_date_time=datetime(2015, 10, 9, 10, 8, 15, 0, tzinfo=pytz.utc),
+        end_date_time=datetime(2199, 10, 9, 10, 8, 15, 0, tzinfo=pytz.utc),
+        duration=30, active=True,
+        attempts_allowed=1, time_between_attempts=0,
+        description='demo quiz 1', pass_criteria=0,
+        instructions="Demo Instructions")
+
+
+        self.course = Course.objects.create(name="Python Course",
+                                   enrollment="Enroll Request", creator=self.user1)
+
         self.questionpaper = QuestionPaper.objects.create(
             quiz=self.quiz, total_marks=0.0, shuffle_questions=True
         )
@@ -2223,26 +2232,32 @@ class AssignmentUploadTestCases(unittest.TestCase):
         self.questionpaper.fixed_question_order = "{0}".format(
             self.question.id)
         self.questionpaper.fixed_questions.add(self.question)
+
+        attempt = 1
+        ip = '127.0.0.1'
+        self.answerpaper1 = self.questionpaper.make_answerpaper(
+                self.user1, ip, attempt, self.course.id
+            )
+
         file_path1 = os.path.join(tempfile.gettempdir(), "upload1.txt")
         file_path2 = os.path.join(tempfile.gettempdir(), "upload2.txt")
         self.assignment1 = AssignmentUpload.objects.create(
-            user=self.user1, assignmentQuestion=self.question,
-            assignmentFile=file_path1, question_paper=self.questionpaper,
-            course=self.course
+            assignmentQuestion=self.question,
+            assignmentFile=file_path1, answer_paper=self.answerpaper1,
             )
         self.assignment2 = AssignmentUpload.objects.create(
-            user=self.user2, assignmentQuestion=self.question,
-            assignmentFile=file_path2, question_paper=self.questionpaper,
-            course=self.course
+            assignmentQuestion=self.question,
+            assignmentFile=file_path2, answer_paper=self.answerpaper1,
             )
 
+
     def test_get_assignments_for_user_files(self):
         assignment_files, file_name = AssignmentUpload.objects.get_assignments(
                                     self.questionpaper, self.question.id,
                                     self.user1.id, self.course.id
                                     )
         self.assertIn("upload1.txt", assignment_files[0].assignmentFile.name)
-        self.assertEqual(assignment_files[0].user, self.user1)
+        self.assertEqual(assignment_files[0].answer_paper.user, self.user1)
         actual_file_name = self.user1.get_full_name().replace(" ", "_")
         file_name = file_name.replace(" ", "_")
         self.assertEqual(file_name, actual_file_name)
@@ -2253,7 +2268,8 @@ class AssignmentUploadTestCases(unittest.TestCase):
             )
         files = [os.path.basename(file.assignmentFile.name)
                  for file in assignment_files]
-        question_papers = [file.question_paper for file in assignment_files]
+        question_papers = [
+            file.answer_paper.question_paper for file in assignment_files]
         self.assertIn("upload1.txt", files)
         self.assertIn("upload2.txt", files)
         self.assertEqual(question_papers[0].quiz, self.questionpaper.quiz)
@@ -2261,6 +2277,14 @@ class AssignmentUploadTestCases(unittest.TestCase):
         file_name = file_name.replace(" ", "_")
         self.assertIn(actual_file_name, file_name)
 
+    def tearDown(self):
+        self.questionpaper.delete()
+        self.question.delete()
+        self.course.delete()
+        self.quiz.delete()
+        self.user2.delete()
+        self.user1.delete()
+
 
 class CourseStatusTestCases(unittest.TestCase):
     def setUp(self):
-- 
cgit 


From 3523fea7d001d3e9ecdd9d87b94121b8c8e75ef0 Mon Sep 17 00:00:00 2001
From: CruiseDevice
Date: Mon, 1 Mar 2021 15:24:33 +0530
Subject: Fix failing testcases in Travis

---
 yaksh/test_views.py | 19 ++++++++++++-------
 yaksh/views.py      |  4 ++--
 2 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/yaksh/test_views.py b/yaksh/test_views.py
index 58b7506..822a6bc 100644
--- a/yaksh/test_views.py
+++ b/yaksh/test_views.py
@@ -938,18 +938,22 @@ class TestDownloadAssignment(TestCase):
             )
         self.question_paper.fixed_questions.add(self.question)
 
+        attempt = 1
+        ip = '127.0.0.1'
+        self.answerpaper1 = self.question_paper.make_answerpaper(
+                self.student1, ip, attempt, self.course.id
+            )
+
         # create assignment file
         assignment_file1 = SimpleUploadedFile("file1.txt", b"Test")
         assignment_file2 = SimpleUploadedFile("file2.txt", b"Test")
         self.assignment1 = AssignmentUpload.objects.create(
-            user=self.student1, assignmentQuestion=self.question,
-            course=self.course,
-            assignmentFile=assignment_file1, question_paper=self.question_paper
+            assignmentQuestion=self.question,
+            assignmentFile=assignment_file1, answer_paper=self.answerpaper1
             )
         self.assignment2 = AssignmentUpload.objects.create(
-            user=self.student2, assignmentQuestion=self.question,
-            course=self.course,
-            assignmentFile=assignment_file2, question_paper=self.question_paper
+            assignmentQuestion=self.question,
+            assignmentFile=assignment_file2, answer_paper=self.answerpaper1
             )
 
     def tearDown(self):
@@ -1037,7 +1041,8 @@ class TestDownloadAssignment(TestCase):
         zip_file = string_io(response.content)
         zipped_file = zipfile.ZipFile(zip_file, 'r')
         self.assertIsNone(zipped_file.testzip())
-        self.assertIn('file2.txt', zipped_file.namelist()[0])
+        self.assertIn('file1.txt', zipped_file.namelist()[0])
+
         zip_file.close()
         zipped_file.close()
 
diff --git a/yaksh/views.py b/yaksh/views.py
index 7d4f788..ea18541 100644
--- a/yaksh/views.py
+++ b/yaksh/views.py
@@ -2182,8 +2182,8 @@ def view_answerpaper(request, questionpaper_id, course_id):
         data = AnswerPaper.objects.get_user_data(user, questionpaper_id,
                                                  course_id)
         has_user_assignments = AssignmentUpload.objects.filter(
-            user=user, course_id=course.id,
-            question_paper_id=questionpaper_id
+            answer_paper__user=user, answer_paper__course_id=course.id,
+            answer_paper__question_paper_id=questionpaper_id
         ).exists()
         context = {'data': data, 'quiz': quiz, 'course_id': course.id,
                    "has_user_assignments": has_user_assignments}
-- 
cgit 


From 085b1fae43a16cff87e1ab0cf3730e84d4943442 Mon Sep 17 00:00:00 2001
From: CruiseDevice
Date: Wed, 3 Mar 2021 23:55:24 +0530
Subject: Fix failing test

---
 yaksh/models.py     |  7 ++++---
 yaksh/test_views.py | 48 +++++++++++++++++++++++++++++++-----------------
 yaksh/views.py      |  1 -
 3 files changed, 35 insertions(+), 21 deletions(-)

diff --git a/yaksh/models.py b/yaksh/models.py
index ca0f2d8..30c69c3 100644
--- a/yaksh/models.py
+++ b/yaksh/models.py
@@ -121,15 +121,16 @@ MOD_GROUP_NAME = 'moderator'
 
 
 def get_assignment_dir(instance, filename):
-    folder = str(instance.answer_paper.course_id)
-    sub_folder = str(instance.answer_paper.question_paper.quiz_id)
+    course = instance.answer_paper.course
+    quiz = instance.answer_paper.question_paper.quiz
+    folder = f'{course.name.replace(" ", "_")}_{course.id}'
+    sub_folder = f'{quiz.description.replace(" ","_")}_{quiz.id}'
     user = instance.answer_paper.user.username
     return os.sep.join((folder, sub_folder, user,
                         str(instance.assignmentQuestion.id),
                         filename
                         ))
 
-
 def get_model_class(model):
     ctype = ContentType.objects.get(app_label="yaksh", model=model)
     model_class = ctype.model_class()
diff --git a/yaksh/test_views.py b/yaksh/test_views.py
index 822a6bc..9d14490 100644
--- a/yaksh/test_views.py
+++ b/yaksh/test_views.py
@@ -944,9 +944,14 @@ class TestDownloadAssignment(TestCase):
                 self.student1, ip, attempt, self.course.id
             )
 
+        self.answerpaper2 = self.question_paper.make_answerpaper(
+                self.student2, ip, attempt, self.course.id
+            )
+
         # create assignment file
         assignment_file1 = SimpleUploadedFile("file1.txt", b"Test")
         assignment_file2 = SimpleUploadedFile("file2.txt", b"Test")
+
         self.assignment1 = AssignmentUpload.objects.create(
             assignmentQuestion=self.question,
             assignmentFile=assignment_file1, answer_paper=self.answerpaper1
@@ -956,22 +961,15 @@ class TestDownloadAssignment(TestCase):
             assignmentFile=assignment_file2, answer_paper=self.answerpaper1
             )
 
-    def tearDown(self):
-        self.client.logout()
-        self.user.delete()
-        self.student1.delete()
-        self.student2.delete()
-        self.assignment1.delete()
-        self.assignment2.delete()
-        self.quiz.delete()
-        self.course.delete()
-        self.learning_module.delete()
-        self.learning_unit.delete()
-        self.mod_group.delete()
-        dir_name = self.course.name.replace(" ", "_")
-        file_path = os.sep.join((settings.MEDIA_ROOT, dir_name))
-        if os.path.exists(file_path):
-            shutil.rmtree(file_path)
+        self.assignment1 = AssignmentUpload.objects.create(
+            assignmentQuestion=self.question,
+            assignmentFile=assignment_file1, answer_paper=self.answerpaper2
+            )
+        self.assignment2 = AssignmentUpload.objects.create(
+            assignmentQuestion=self.question,
+            assignmentFile=assignment_file2, answer_paper=self.answerpaper2
+            )
+
 
     def test_download_assignment_denies_student(self):
         """
@@ -1042,10 +1040,26 @@ class TestDownloadAssignment(TestCase):
         zipped_file = zipfile.ZipFile(zip_file, 'r')
         self.assertIsNone(zipped_file.testzip())
         self.assertIn('file1.txt', zipped_file.namelist()[0])
-
         zip_file.close()
         zipped_file.close()
 
+    def tearDown(self):
+        self.client.logout()
+        self.user.delete()
+        self.student1.delete()
+        self.student2.delete()
+        self.assignment1.delete()
+        self.assignment2.delete()
+        self.quiz.delete()
+        self.learning_module.delete()
+        self.learning_unit.delete()
+        self.mod_group.delete()
+        dir_name = f'{self.course.name.replace(" ", "_")}_{self.course.id}'
+        file_path = os.sep.join((settings.MEDIA_ROOT, dir_name))
+        if os.path.exists(file_path):
+            shutil.rmtree(file_path)
+        self.course.delete()
+
 
 class TestAddQuiz(TestCase):
     def setUp(self):
diff --git a/yaksh/views.py b/yaksh/views.py
index ea18541..3cff7f5 100644
--- a/yaksh/views.py
+++ b/yaksh/views.py
@@ -1866,7 +1866,6 @@ def grade_user(request, quiz_id=None, user_id=None, attempt_number=None,
     """
     current_user = request.user
     papers = AnswerPaper.objects.filter(user=current_user)
-    print('Paper: ', papers)
     if not is_moderator(current_user):
         raise Http404('You are not allowed to view this page!')
     if not course_id:
-- 
cgit 


From ccc578beff8363ea19826d6c621dafabe210531f Mon Sep 17 00:00:00 2001
From: CruiseDevice
Date: Thu, 4 Mar 2021 16:49:55 +0530
Subject: Minor Change in get_assignment_dir

---
 yaksh/models.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/yaksh/models.py b/yaksh/models.py
index 30c69c3..e88a9e4 100644
--- a/yaksh/models.py
+++ b/yaksh/models.py
@@ -123,8 +123,8 @@ MOD_GROUP_NAME = 'moderator'
 def get_assignment_dir(instance, filename):
     course = instance.answer_paper.course
     quiz = instance.answer_paper.question_paper.quiz
-    folder = f'{course.name.replace(" ", "_")}_{course.id}'
-    sub_folder = f'{quiz.description.replace(" ","_")}_{quiz.id}'
+    folder = f'Course_{course.id}'
+    sub_folder = f'Quiz_{quiz.id}'
     user = instance.answer_paper.user.username
     return os.sep.join((folder, sub_folder, user,
                         str(instance.assignmentQuestion.id),
-- 
cgit 


From b457c973b491d373ed9c5087b61bf29ce128d766 Mon Sep 17 00:00:00 2001
From: CruiseDevice
Date: Mon, 15 Mar 2021 13:40:12 +0530
Subject: Replacing .id with _id in get_assignment_dir()

 - .id fetches entire model which is unnecessary. _id only fetches
   the primary key.
---
 yaksh/models.py | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/yaksh/models.py b/yaksh/models.py
index e88a9e4..4661ea7 100644
--- a/yaksh/models.py
+++ b/yaksh/models.py
@@ -121,13 +121,13 @@ MOD_GROUP_NAME = 'moderator'
 
 
 def get_assignment_dir(instance, filename):
-    course = instance.answer_paper.course
-    quiz = instance.answer_paper.question_paper.quiz
-    folder = f'Course_{course.id}'
-    sub_folder = f'Quiz_{quiz.id}'
+    course_id = instance.answer_paper.course_id
+    quiz_id = instance.answer_paper.question_paper.quiz_id
+    folder = f'Course_{course_id}'
+    sub_folder = f'Quiz_{quiz_id}'
     user = instance.answer_paper.user.username
     return os.sep.join((folder, sub_folder, user,
-                        str(instance.assignmentQuestion.id),
+                        str(instance.assignmentQuestion_id),
                         filename
                         ))
 
-- 
cgit 


From 1793f445ec0cbc67df4b4cd3151859e01ed4b2a0 Mon Sep 17 00:00:00 2001
From: CruiseDevice
Date: Mon, 15 Mar 2021 14:59:55 +0530
Subject: PEP8 fix

---
 yaksh/admin.py       |  2 +-
 yaksh/models.py      | 54 ++++++++++++++++++++++++++++++++++------------------
 yaksh/test_models.py | 43 +++++++++++++++++++++++++----------------
 yaksh/test_views.py  |  2 +-
 4 files changed, 65 insertions(+), 36 deletions(-)

diff --git a/yaksh/admin.py b/yaksh/admin.py
index 011e24f..1010536 100644
--- a/yaksh/admin.py
+++ b/yaksh/admin.py
@@ -64,4 +64,4 @@ admin.site.register(Topic)
 admin.site.register(TableOfContents)
 admin.site.register(LessonQuizAnswer)
 admin.site.register(Answer)
-admin.site.register(AssignmentUpload)
\ No newline at end of file
+admin.site.register(AssignmentUpload)
diff --git a/yaksh/models.py b/yaksh/models.py
index 4661ea7..610e984 100644
--- a/yaksh/models.py
+++ b/yaksh/models.py
@@ -131,6 +131,7 @@ def get_assignment_dir(instance, filename):
                         filename
                         ))
 
+
 def get_model_class(model):
     ctype = ContentType.objects.get(app_label="yaksh", model=model)
     model_class = ctype.model_class()
@@ -1381,7 +1382,9 @@ class Question(models.Model):
     # Solution for the question.
     solution = models.TextField(blank=True)
 
-    content = GenericRelation("TableOfContents", related_query_name='questions')
+    content = GenericRelation(
+        "TableOfContents", related_query_name='questions'
+    )
 
     tc_code_types = {
         "python": [
@@ -1838,7 +1841,8 @@ class QuestionPaper(models.Model):
             all_questions = questions
         return all_questions
 
-    def make_answerpaper(self, user, ip, attempt_num, course_id, special=False):
+    def make_answerpaper(self,
+                         user, ip, attempt_num, course_id, special=False):
         """Creates an answer paper for the user to attempt the quiz"""
         try:
             ans_paper = AnswerPaper.objects.get(user=user,
@@ -2035,6 +2039,7 @@ class AnswerPaperManager(models.Manager):
         ).order_by("id").values(
             "answerpaper__id", "question_id", "correct", "answer"
         )
+
         def _get_per_tc_data(answers, q_type):
             tc = []
             for answer in answers["answer"]:
@@ -2047,11 +2052,11 @@ class AnswerPaperManager(models.Manager):
         df = pd.DataFrame(answers)
         if not df.empty:
             for question in all_questions:
-                que = df[df["question_id"]==question.id].groupby(
+                que = df[df["question_id"] == question.id].groupby(
                         "answerpaper__id").tail(1)
                 if not que.empty:
                     total_attempts = que.shape[0]
-                    correct_attempts = que[que["correct"]==True].shape[0]
+                    correct_attempts = que[que["correct"] == True].shape[0]
                     per_tc_ans = {}
                     if question.type in ["mcq", "mcc"]:
                         per_tc_ans = _get_per_tc_data(que, question.type)
@@ -2643,13 +2648,16 @@ class AssignmentUploadManager(models.Manager):
     def get_assignments(self, qp, que_id=None, user_id=None, course_id=None):
         if que_id and user_id:
             assignment_files = AssignmentUpload.objects.filter(
-                        assignmentQuestion_id=que_id, answer_paper__user_id=user_id,
-                        answer_paper__question_paper=qp, answer_paper__course_id=course_id
+                        assignmentQuestion_id=que_id,
+                        answer_paper__user_id=user_id,
+                        answer_paper__question_paper=qp,
+                        answer_paper__course_id=course_id
                         )
             file_name = User.objects.get(id=user_id).get_full_name()
         else:
             assignment_files = AssignmentUpload.objects.filter(
-                        answer_paper__question_paper=qp, answer_paper__course_id=course_id
+                        answer_paper__question_paper=qp,
+                        answer_paper__course_id=course_id
                         )
             file_name = "{0}_Assignment_files".format(
                             assignment_files[0].answer_paper.course.name
@@ -2895,7 +2903,6 @@ class TOCManager(models.Manager):
             toc.get_toc_as_yaml(file_path)
         return file_path
 
-
     def get_question_stats(self, toc_id):
         answers = LessonQuizAnswer.objects.get_queryset().filter(
             toc_id=toc_id).order_by('id')
@@ -2929,7 +2936,7 @@ class TOCManager(models.Manager):
                     if j not in mydata:
                         mydata[j] = 1
                     else:
-                        mydata[j] +=1
+                        mydata[j] += 1
                 data = mydata.copy()
             if is_percent:
                 for key, value in data.items():
@@ -2975,17 +2982,20 @@ class TOCManager(models.Manager):
                 if not is_valid_time_format(time):
                     messages.append(
                         (False,
-                        f"Invalid time format in {name}. "
+                         f"Invalid time format in {name}. "
                          "Format should be 00:00:00")
-                    )
+                        )
                 else:
                     if content_type == 1:
                         topic = Topic.objects.create(**content)
                         toc.append(TableOfContents(
-                            course_id=course_id, lesson_id=lesson_id, time=time,
-                            content_object=topic, content=content_type 
+                            course_id=course_id,
+                            lesson_id=lesson_id, time=time,
+                            content_object=topic, content=content_type
                         ))
-                        messages.append((True, f"{topic.name} added successfully"))
+                        messages.append(
+                            (True, f"{topic.name} added successfully")
+                        )
                     else:
                         content['user'] = user
                         test_cases = content.pop("testcase")
@@ -3002,10 +3012,13 @@ class TOCManager(models.Manager):
                         else:
                             que = Question.objects.create(**content)
                             for test_case in test_cases:
-                                test_case_type = test_case.pop('test_case_type')
+                                test_case_type = test_case.pop(
+                                    'test_case_type'
+                                )
                                 model_class = get_model_class(test_case_type)
                                 model_class.objects.get_or_create(
-                                    question=que, **test_case, type=test_case_type
+                                    question=que,
+                                    **test_case, type=test_case_type
                                 )
                             toc.append(TableOfContents(
                                 course_id=course_id, lesson_id=lesson_id,
@@ -3021,7 +3034,12 @@ class TOCManager(models.Manager):
 
 
 class TableOfContents(models.Model):
-    toc_types = ((1, "Topic"), (2, "Graded Quiz"), (3, "Exercise"), (4, "Poll"))
+    toc_types = (
+        (1, "Topic"),
+        (2, "Graded Quiz"),
+        (3, "Exercise"),
+        (4, "Poll")
+    )
     course = models.ForeignKey(Course, on_delete=models.CASCADE,
                                related_name='course')
     lesson = models.ForeignKey(Lesson, on_delete=models.CASCADE,
@@ -3039,7 +3057,7 @@ class TableOfContents(models.Model):
 
     def get_toc_text(self):
         if self.content == 1:
-            content_name =  self.content_object.name
+            content_name = self.content_object.name
         else:
             content_name = self.content_object.summary
         return content_name
diff --git a/yaksh/test_models.py b/yaksh/test_models.py
index 86890c0..ae1ff22 100644
--- a/yaksh/test_models.py
+++ b/yaksh/test_models.py
@@ -107,6 +107,7 @@ def setUpModule():
     MicroManager.objects.create(manager=user, course=course, quiz=quiz,
                                 student=course_user)
 
+
 def tearDownModule():
     User.objects.all().delete()
     Question.objects.all().delete()
@@ -2203,23 +2204,34 @@ class TestCaseTestCases(unittest.TestCase):
 class AssignmentUploadTestCases(unittest.TestCase):
 
     def setUp(self):
-        self.user1 = User.objects.create_user(username='creator1',
-                                    password='demo',
-                                    email='demo@test1.com')
-        self.user2 = User.objects.create_user(username='creator2',
-                                    password='demo',
-                                    email='demo@test2.com')
+        self.user1 = User.objects.create_user(
+            username='creator1',
+            password='demo',
+            email='demo@test1.com'
+        )
+        self.user2 = User.objects.create_user(
+            username='creator2',
+            password='demo',
+            email='demo@test2.com'
+        )
         self.quiz = Quiz.objects.create(
-        start_date_time=datetime(2015, 10, 9, 10, 8, 15, 0, tzinfo=pytz.utc),
-        end_date_time=datetime(2199, 10, 9, 10, 8, 15, 0, tzinfo=pytz.utc),
-        duration=30, active=True,
-        attempts_allowed=1, time_between_attempts=0,
-        description='demo quiz 1', pass_criteria=0,
-        instructions="Demo Instructions")
-
+            start_date_time=datetime(
+                2015, 10, 9, 10, 8, 15, 0, tzinfo=pytz.utc
+            ),
+            end_date_time=datetime(
+                2199, 10, 9, 10, 8, 15, 0, tzinfo=pytz.utc
+            ),
+            duration=30, active=True,
+            attempts_allowed=1, time_between_attempts=0,
+            description='demo quiz 1', pass_criteria=0,
+            instructions="Demo Instructions"
+        )
 
-        self.course = Course.objects.create(name="Python Course",
-                                   enrollment="Enroll Request", creator=self.user1)
+        self.course = Course.objects.create(
+            name="Python Course",
+            enrollment="Enroll Request",
+            creator=self.user1
+        )
 
         self.questionpaper = QuestionPaper.objects.create(
             quiz=self.quiz, total_marks=0.0, shuffle_questions=True
@@ -2250,7 +2262,6 @@ class AssignmentUploadTestCases(unittest.TestCase):
             assignmentFile=file_path2, answer_paper=self.answerpaper1,
             )
 
-
     def test_get_assignments_for_user_files(self):
         assignment_files, file_name = AssignmentUpload.objects.get_assignments(
                                     self.questionpaper, self.question.id,
diff --git a/yaksh/test_views.py b/yaksh/test_views.py
index 9d14490..e5ba8dd 100644
--- a/yaksh/test_views.py
+++ b/yaksh/test_views.py
@@ -1054,7 +1054,7 @@ class TestDownloadAssignment(TestCase):
         self.learning_module.delete()
         self.learning_unit.delete()
         self.mod_group.delete()
-        dir_name = f'{self.course.name.replace(" ", "_")}_{self.course.id}'
+        dir_name = f'Course_{self.course.id}'
         file_path = os.sep.join((settings.MEDIA_ROOT, dir_name))
         if os.path.exists(file_path):
             shutil.rmtree(file_path)
-- 
cgit