Autoformat all (python) files

Author: krishnans2006

create_debug_users.py

@@ -2,14 +2,24 @@
 import os
 import sys
 import subprocess
+
 if not __file__.endswith("shell.py"):
-    subprocess.call([sys.executable, os.path.join(os.path.dirname(__file__), "manage.py"), "shell", "-c", open(__file__).read()])
+    subprocess.call(
+        [
+            sys.executable,
+            os.path.join(os.path.dirname(__file__), "manage.py"),
+            "shell",
+            "-c",
+            open(__file__).read(),
+        ]
+    )
     exit()
 
 from tin.apps.users.models import User
 
 password = input("Enter password for all users: ")
 
+# fmt: off
 users = [
     #[username, id, full_name, first_name, last_name, email, is_teacher, is_student, is_staff, is_superuser]
     ["2020jbeutner", 33538, "John Beutner", "John", "Beutner", "[email protected]", False, True, False, False],
@@ -18,12 +28,14 @@
     ["pcgabor", None, "Peter Gabor", "Peter", "Gabor", "[email protected]", True, False, False, False],
     ["admin", None, "Admin", "Admin", "Admin", "[email protected]", False, False, True, True],
 ]
+# fmt: on
+
 for user_info in users:
     print("Creating user {}".format(user_info[0]))
     if user_info[1] is None:
-        user = User.objects.get_or_create(username = user_info[0])[0]
+        user = User.objects.get_or_create(username=user_info[0])[0]
     else:
-        user = User.objects.get_or_create(id = user_info[1])[0]
+        user = User.objects.get_or_create(id=user_info[1])[0]
         user.username = user_info[0]
 
     user.full_name = user_info[2]
@@ -36,4 +48,3 @@
     user.is_superuser = user_info[9]
     user.set_password(password)
     user.save()
-

manage.py

@@ -2,8 +2,8 @@
 import os
 import sys
 
-if __name__ == '__main__':
-    os.environ.setdefault('DJANGO_SETTINGS_MODULE', 'tin.settings')
+if __name__ == "__main__":
+    os.environ.setdefault("DJANGO_SETTINGS_MODULE", "tin.settings")
     try:
         from django.core.management import execute_from_command_line
     except ImportError as exc:

tin/apps/assignments/models.py

@@ -31,7 +31,9 @@ def filter_visible(self, user):
         if user.is_superuser:
             return self.all()
         else:
-            return self.filter(Q(course__teacher=user) | Q(course__students=user, hidden=False)).distinct()
+            return self.filter(
+                Q(course__teacher=user) | Q(course__students=user, hidden=False)
+            ).distinct()
 
     def filter_editable(self, user):
         if user.is_superuser:
@@ -58,10 +60,10 @@ class Assignment(models.Model):
     description = models.CharField(max_length=4096)
 
     LANGUAGES = (
-        ('P', 'Python 3'),
-        ('J', 'Java'),
+        ("P", "Python 3"),
+        ("J", "Java"),
     )
-    language = models.CharField(max_length=1, choices=LANGUAGES, default='P')
+    language = models.CharField(max_length=1, choices=LANGUAGES, default="P")
 
     filename = models.CharField(max_length=50, default="main.py")
 
@@ -145,11 +147,7 @@ def save_grader_file(self, grader_text: str) -> None:
             raise FileNotFoundError from e
 
     def save_file(self, file_text: str, file_name: str) -> None:
-        fpath = os.path.join(
-            settings.MEDIA_ROOT,
-            "assignment-{}".format(self.id),
-            file_name
-        )
+        fpath = os.path.join(settings.MEDIA_ROOT, "assignment-{}".format(self.id), file_name)
 
         os.makedirs(os.path.dirname(fpath), exist_ok=True)
 
@@ -178,8 +176,9 @@ def check_rate_limit(self, student) -> None:
 
         if (
             Submission.objects.filter(
-                date_submitted__gte=now - datetime.timedelta(minutes=self.submission_limit_interval),
-                student=student
+                date_submitted__gte=now
+                - datetime.timedelta(minutes=self.submission_limit_interval),
+                student=student,
             ).count()
             > self.submission_limit_count
         ):
@@ -262,7 +261,10 @@ def __str__(self):
         return f"Quiz for {self.assignment}"
 
     def issues_for_student(self, student):
-        return sum(lm.severity for lm in self.log_messages.filter(student=student)) >= settings.QUIZ_ISSUE_THRESHOLD
+        return (
+            sum(lm.severity for lm in self.log_messages.filter(student=student))
+            >= settings.QUIZ_ISSUE_THRESHOLD
+        )
 
     def open_for_student(self, student):
         return not (self.locked_for_student(student) or self.ended_for_student(student))
@@ -275,15 +277,9 @@ def ended_for_student(self, student):
 
 
 class LogMessage(models.Model):
-    quiz = models.ForeignKey(
-        Quiz,
-        on_delete=models.CASCADE,
-        related_name="log_messages"
-    )
+    quiz = models.ForeignKey(Quiz, on_delete=models.CASCADE, related_name="log_messages")
     student = models.ForeignKey(
-        get_user_model(),
-        on_delete=models.CASCADE,
-        related_name="log_messages"
+        get_user_model(), on_delete=models.CASCADE, related_name="log_messages"
     )
 
     date = models.DateTimeField(auto_now_add=True)

tin/apps/assignments/views.py

@@ -81,7 +81,9 @@ def show_view(request, assignment_id):
             if period == "":
                 if request.user in course.teacher.all():
                     try:
-                        period = course.period_set.filter(teacher=request.user).order_by("name")[0].id
+                        period = (
+                            course.period_set.filter(teacher=request.user).order_by("name")[0].id
+                        )
                     except:
                         period = "all"
                 else:
@@ -91,7 +93,9 @@ def show_view(request, assignment_id):
                 active_period = "all"
                 student_list = course.students.all().order_by("periods", "last_name")
             else:
-                active_period = get_object_or_404(Period.objects.filter(course=course), id=int(period))
+                active_period = get_object_or_404(
+                    Period.objects.filter(course=course), id=int(period)
+                )
                 student_list = active_period.students.all().order_by("last_name")
         else:
             active_period = "all"
@@ -114,7 +118,13 @@ def show_view(request, assignment_id):
                 )
             else:
                 students_and_submissions.append(
-                    (student, period, latest_submission, assignment.quiz.ended_for_student(student), assignment.quiz.locked_for_student(student))
+                    (
+                        student,
+                        period,
+                        latest_submission,
+                        assignment.quiz.ended_for_student(student),
+                        assignment.quiz.locked_for_student(student),
+                    )
                 )
 
         context = {
@@ -158,10 +168,7 @@ def create_view(request, course_id):
 
             quiz_type = assignment_form.cleaned_data["is_quiz"]
             if quiz_type != "-1":
-                Quiz.objects.create(
-                    assignment = assignment,
-                    action = quiz_type
-                )
+                Quiz.objects.create(assignment=assignment, action=quiz_type)
 
             return redirect("assignments:show", assignment.id)
     else:
@@ -191,7 +198,9 @@ def edit_view(request, assignment_id):
         initial_is_quiz = assignment.quiz.action
     except:
         pass
-    assignment_form = AssignmentForm(course, instance=assignment, initial={"is_quiz": initial_is_quiz})
+    assignment_form = AssignmentForm(
+        course, instance=assignment, initial={"is_quiz": initial_is_quiz}
+    )
     if request.method == "POST":
         assignment_form = AssignmentForm(course, data=request.POST, instance=assignment)
         if assignment_form.is_valid():
@@ -209,10 +218,7 @@ def edit_view(request, assignment_id):
                     assignment.save()
                     assignment.quiz.save()
                 except:
-                    Quiz.objects.create(
-                        assignment = assignment,
-                        action = quiz_type
-                    )
+                    Quiz.objects.create(assignment=assignment, action=quiz_type)
 
             return redirect("assignments:show", assignment.id)
 
@@ -351,7 +357,11 @@ def student_submissions_view(request, assignment_id, student_id):
     )
     latest_submission = submissions.first() if submissions else None
 
-    log_messages = assignment.quiz.log_messages.filter(student=request.user).order_by("date") if assignment.is_quiz else None
+    log_messages = (
+        assignment.quiz.log_messages.filter(student=request.user).order_by("date")
+        if assignment.is_quiz
+        else None
+    )
 
     latest_submission_text = None
     if latest_submission:
@@ -507,14 +517,22 @@ def submit_view(request, assignment_id):
 
 @login_required
 def quiz_view(request, assignment_id):
-    assignment = get_object_or_404(Assignment.objects.filter_visible(request.user), id=assignment_id)
+    assignment = get_object_or_404(
+        Assignment.objects.filter_visible(request.user), id=assignment_id
+    )
 
-    if not assignment.is_quiz or assignment.quiz.locked_for_student(request.user) or assignment.quiz.ended_for_student(request.user):
+    if (
+        not assignment.is_quiz
+        or assignment.quiz.locked_for_student(request.user)
+        or assignment.quiz.ended_for_student(request.user)
+    ):
         raise http.Http404
 
     student = request.user
 
-    submissions = Submission.objects.filter(student=student, assignment=assignment).order_by("-date_submitted")
+    submissions = Submission.objects.filter(student=student, assignment=assignment).order_by(
+        "-date_submitted"
+    )
     latest_submission = submissions.first() if submissions else None
 
     latest_submission_text = None
@@ -529,7 +547,10 @@ def quiz_view(request, assignment_id):
         if assignment.grader_file is None:
             return redirect("assignments:show", assignment.id)
 
-        if Submission.objects.filter(student=request.user, complete=False).count() >= settings.CONCURRENT_USER_SUBMISSION_LIMIT:
+        if (
+            Submission.objects.filter(student=request.user, complete=False).count()
+            >= settings.CONCURRENT_USER_SUBMISSION_LIMIT
+        ):
             text_form = TextSubmissionForm(request.POST)
             text_errors = (
                 "You may only have a maximum of {} submission{} running at the same "
@@ -590,7 +611,9 @@ def quiz_view(request, assignment_id):
 
 @login_required
 def quiz_report_view(request, assignment_id):
-    assignment = get_object_or_404(Assignment.objects.filter_visible(request.user), id=assignment_id)
+    assignment = get_object_or_404(
+        Assignment.objects.filter_visible(request.user), id=assignment_id
+    )
 
     content = request.GET.get("content", "")
     severity = int(request.GET.get("severity", 0))
@@ -599,10 +622,7 @@ def quiz_report_view(request, assignment_id):
         json_data = json.dumps("no action")
     else:
         LogMessage.objects.create(
-            quiz=assignment.quiz,
-            student=request.user,
-            content=content,
-            severity=severity
+            quiz=assignment.quiz, student=request.user, content=content, severity=severity
         )
 
         resp = "no action"
@@ -618,21 +638,22 @@ def quiz_report_view(request, assignment_id):
 
 @login_required
 def quiz_end_view(request, assignment_id):
-    assignment = get_object_or_404(Assignment.objects.filter_visible(request.user), id=assignment_id)
+    assignment = get_object_or_404(
+        Assignment.objects.filter_visible(request.user), id=assignment_id
+    )
 
     LogMessage.objects.create(
-        quiz=assignment.quiz,
-        student=request.user,
-        content="Ended quiz",
-        severity=0
+        quiz=assignment.quiz, student=request.user, content="Ended quiz", severity=0
     )
 
     return redirect("assignments:show", assignment.id)
 
 
 @teacher_or_superuser_required
 def quiz_clear_view(request, assignment_id, user_id):
-    assignment = get_object_or_404(Assignment.objects.filter_editable(request.user), id=assignment_id)
+    assignment = get_object_or_404(
+        Assignment.objects.filter_editable(request.user), id=assignment_id
+    )
     user = get_object_or_404(get_user_model(), id=user_id)
 
     assignment.quiz.log_messages.filter(student=user).delete()

tin/apps/courses/models.py

@@ -47,11 +47,9 @@ def __repr__(self):
     def get_teacher_str(self):
         return ", ".join((t.last_name for t in self.teacher.all()))
 
-
     def is_student_in_course(self, user):
         return user in self.students.all()
 
-
     def is_only_student_in_course(self, user):
         return user in self.students.all() and not (user.is_superuser or user in self.teacher.all())
 

tin/apps/submissions/models.py

@@ -19,17 +19,13 @@ def filter_visible(self, user):
         if user.is_superuser:
             return self.all()
         else:
-            return self.filter(
-                Q(assignment__course__teacher=user) | Q(student=user)
-            ).distinct()
+            return self.filter(Q(assignment__course__teacher=user) | Q(student=user)).distinct()
 
     def filter_editable(self, user):
         if user.is_superuser:
             return self.all()
         else:
-            return self.filter(
-                Q(assignment__course__teacher=user) | Q(student=user)
-            ).distinct()
+            return self.filter(Q(assignment__course__teacher=user) | Q(student=user)).distinct()
 
 
 def upload_submission_file_path(submission, _):  # pylint: disable=unused-argument

tin/apps/submissions/tasks.py

@@ -26,7 +26,7 @@
 
 def truncate_output(text, field_name):
     max_len = Submission._meta.get_field(field_name).max_length
-    return ("..." + text[-max_len + 5:]) if len(text) > max_len else text
+    return ("..." + text[-max_len + 5 :]) if len(text) > max_len else text
 
 
 @shared_task
@@ -72,7 +72,15 @@ def run_submission(submission_id):
         else:
             folder_name = "testing"
 
-        with open(os.path.join(settings.BASE_DIR, "sandboxing", "wrappers", folder_name, f"{submission.assignment.language}.txt")) as wrapper_file:
+        with open(
+            os.path.join(
+                settings.BASE_DIR,
+                "sandboxing",
+                "wrappers",
+                folder_name,
+                f"{submission.assignment.language}.txt",
+            )
+        ) as wrapper_file:
             wrapper_text = wrapper_file.read().format(
                 has_network_access=bool(submission.assignment.has_network_access),
                 venv_path=(

tin/apps/venvs/models.py

@@ -176,9 +176,9 @@ def install_packages(self, pkgs):
                 raise FileNotFoundError from e
 
             try:
-                self.package_installation_output = res.stdout.decode()[-16 * 1024:]
+                self.package_installation_output = res.stdout.decode()[-16 * 1024 :]
             except UnicodeDecodeError:
-                self.package_installation_output = str(res.stdout)[-16 * 1024:]
+                self.package_installation_output = str(res.stdout)[-16 * 1024 :]
         finally:
             self.installing_packages = False
             self.save()