17 backend testing ☣️[infected, will close]

backend/pigeonhole/apps/courses/migrations/0001_initial.py

@@ -1,9 +1,10 @@
-# Generated by Django 5.0.2 on 2024-03-02 21:03
+# Generated by Django 5.0.3 on 2024-03-10 16:00
 
 from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
+
     initial = True
 
     dependencies = [

backend/pigeonhole/apps/courses/permissions.py

@@ -0,0 +1,30 @@
+from rest_framework import permissions
+from backend.pigeonhole.apps.users.models import Teacher, Student
+
+
+class CourseUserPermissions(permissions.BasePermission):
+    def has_permission(self, request, view):
+        if request.user.is_superuser:
+            return True
+        if Teacher.objects.filter(id=request.user.id).exists():
+            return True
+        elif Student.objects.filter(id=request.user.id).exists():
+            return view.action in ['list', 'retrieve']
+        return False
+
+    def has_object_permission(self, request, view, obj):
+        if request.user.is_superuser:
+            return True
+        if Teacher.objects.filter(id=request.user.id).exists():
+            if request.user.is_admin:
+                return True
+            # Check if the teacher is assigned to the course
+            elif request.user.course.filter(id=obj.id).exists():
+                return True
+            return view.action in ['list', 'retrieve']
+        elif isinstance(request.user, Student):
+            return view.action in ['list', 'retrieve']
+
+        return False
+
+

backend/pigeonhole/apps/courses/views.py

@@ -0,0 +1,44 @@
+from rest_framework import viewsets, status
+from rest_framework.permissions import IsAuthenticated
+from rest_framework.response import Response
+
+from .models import Course, CourseSerializer
+from .permissions import CourseUserPermissions
+
+
+class CourseViewSet(viewsets.ModelViewSet):
+    queryset = Course.objects.all()
+    serializer_class = CourseSerializer
+    permission_classes = [IsAuthenticated, CourseUserPermissions]
+
+    def create(self, request, *args, **kwargs):
+        serializer = CourseSerializer(data=request.data)
+        if serializer.is_valid():
+            serializer.save()
+            return Response(serializer.data, status=status.HTTP_201_CREATED)
+        return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
+
+    def update(self, request, *args, **kwargs):
+        course_id = kwargs.get('pk')
+        course = Course.objects.get(pk=course_id)
+        serializer = CourseSerializer(course, data=request.data)
+        if serializer.is_valid():
+            serializer.save()
+            return Response(serializer.data, status=status.HTTP_200_OK)
+        return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
+
+    def destroy(self, request, *args, **kwargs):
+        course_id = kwargs.get('pk')
+        course = Course.objects.get(pk=course_id)
+        course.delete()
+        return Response(status=status.HTTP_204_NO_CONTENT)
+
+    def list(self, request, *args, **kwargs):
+        serializer = CourseSerializer(self.queryset, many=True)
+        return Response(serializer.data, status=status.HTTP_200_OK)
+
+    def retrieve(self, request, *args, **kwargs):
+        course_id = kwargs.get('pk')
+        course = Course.objects.get(pk=course_id)
+        serializer = CourseSerializer(course, many=False)
+        return Response(serializer.data, status=status.HTTP_200_OK)

backend/pigeonhole/apps/groups/migrations/0001_initial.py

@@ -1,10 +1,11 @@
-# Generated by Django 5.0.2 on 2024-03-02 21:03
+# Generated by Django 5.0.3 on 2024-03-10 16:00
 
 import django.db.models.deletion
 from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
+
     initial = True
 
     dependencies = [

backend/pigeonhole/apps/groups/migrations/0002_initial.py

@@ -1,9 +1,10 @@
-# Generated by Django 5.0.2 on 2024-03-02 21:03
+# Generated by Django 5.0.3 on 2024-03-10 16:00
 
 from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
+
     initial = True
 
     dependencies = [

backend/pigeonhole/apps/groups/models.py

@@ -28,4 +28,4 @@ def save(self, *args, **kwargs):
 class GroupSerializer(serializers.ModelSerializer):
     class Meta:
         model = Group
-        fields = ["group_id", "group_nr", "final_score", "project_id", "student"]
+        fields = ["group_id", "group_nr", "final_score", "project_id", "student", "feedback"]

backend/pigeonhole/apps/groups/views.py

@@ -0,0 +1,26 @@
+from rest_framework import viewsets, status
+from rest_framework.permissions import IsAuthenticated
+from rest_framework.response import Response
+
+from backend.pigeonhole.apps.groups.models import Group, GroupSerializer
+
+
+class GroupViewSet(viewsets.ModelViewSet):
+    queryset = Group.objects.all()
+    serializer_class = GroupSerializer
+    permission_classes = [IsAuthenticated]
+
+    def perform_create(self, serializer):
+        serializer.save(user=self.request.user)
+
+    def list(self, request, *args, **kwargs):
+        serializer = GroupSerializer(self.queryset, many=True)
+        return Response(serializer.data, status=status.HTTP_200_OK)
+
+    def create(self, request, *args, **kwargs):
+        serializer = GroupSerializer(data=request.data)
+        if serializer.is_valid():
+            serializer.save()
+            return Response(serializer.data, status=status.HTTP_201_CREATED)
+        else:
+            return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)

backend/pigeonhole/apps/projects/migrations/0001_initial.py

@@ -1,10 +1,11 @@
-# Generated by Django 5.0.2 on 2024-03-02 21:03
+# Generated by Django 5.0.3 on 2024-03-10 16:00
 
 import django.db.models.deletion
 from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
+
     initial = True
 
     dependencies = [
@@ -26,27 +27,25 @@ class Migration(migrations.Migration):
             name='ForbiddenExtension',
             fields=[
                 ('extension_id', models.BigAutoField(primary_key=True, serialize=False)),
-                ('extension', models.IntegerField()),
+                ('extension', models.CharField(max_length=512)),
                 ('project_id', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='projects.project')),
             ],
         ),
         migrations.CreateModel(
             name='Conditions',
             fields=[
                 ('condition_id', models.BigAutoField(primary_key=True, serialize=False)),
-                ('condition', models.CharField(max_length=256)),
-                ('deadline', models.DateTimeField()),
+                ('condition', models.TextField(max_length=256)),
                 ('test_file_location', models.CharField(max_length=512, null=True)),
                 ('test_file_type', models.CharField(max_length=256, null=True)),
-                ('submission_id', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE,
-                                                    to='projects.project')),
+                ('submission_id', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='projects.project')),
             ],
         ),
         migrations.CreateModel(
             name='AllowedExtension',
             fields=[
                 ('extension_id', models.BigAutoField(primary_key=True, serialize=False)),
-                ('extension', models.IntegerField()),
+                ('extension', models.CharField(max_length=512)),
                 ('project_id', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='projects.project')),
             ],
         ),

backend/pigeonhole/apps/projects/migrations/0002_project_deadline.py

@@ -0,0 +1,18 @@
+# Generated by Django 5.0.3 on 2024-03-10 16:09
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('projects', '0001_initial'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='project',
+            name='deadline',
+            field=models.DateTimeField(blank=True, null=True),
+        ),
+    ]

backend/pigeonhole/apps/projects/migrations/0002_remove_conditions_deadline_project_deadline.py

@@ -0,0 +1,22 @@
+# Generated by Django 5.0.3 on 2024-03-09 21:26
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('projects', '0001_initial'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='conditions',
+            name='deadline',
+        ),
+        migrations.AddField(
+            model_name='project',
+            name='deadline',
+            field=models.DateTimeField(null=True),
+        ),
+    ]

backend/pigeonhole/apps/projects/migrations/0003_alter_project_deadline.py

@@ -0,0 +1,18 @@
+# Generated by Django 5.0.3 on 2024-03-10 16:09
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('projects', '0002_project_deadline'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='project',
+            name='deadline',
+            field=models.DateTimeField(null=True),
+        ),
+    ]

backend/pigeonhole/apps/projects/migrations/0004_alter_project_deadline.py

@@ -0,0 +1,18 @@
+# Generated by Django 5.0.3 on 2024-03-10 16:11
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('projects', '0003_alter_project_deadline'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='project',
+            name='deadline',
+            field=models.DateTimeField(blank=True, null=True),
+        ),
+    ]

backend/pigeonhole/apps/projects/models.py

@@ -6,49 +6,50 @@
 
 # Create your models here.
 class Project(models.Model):
+    objects = models.Manager()
     project_id = models.BigAutoField(primary_key=True)
     course_id = models.ForeignKey(Course, on_delete=models.CASCADE)
     name = models.CharField(max_length=256)
     description = models.TextField()
+    deadline = models.DateTimeField(null=True, blank=True)
     visible = models.BooleanField(default=False)
 
 
 class ProjectSerializer(serializers.ModelSerializer):
     class Meta:
         model = Project
-        fields = ['project_id', 'course_id', 'name', 'description', 'deadline', 'visible']
+        fields = ['project_id', 'course_id', 'name', 'description', 'visible', 'deadline']
 
 
 class Conditions(models.Model):
     condition_id = models.BigAutoField(primary_key=True)
     submission_id = models.ForeignKey(Project, on_delete=models.CASCADE)
-    condition = models.CharField(max_length=256)
-    deadline = models.DateTimeField()
+    condition = models.TextField(max_length=256)
     test_file_location = models.CharField(max_length=512, null=True)
     test_file_type = models.CharField(max_length=256, null=True)
 
     objects = models.Manager()
 
     @property
     def get_forbidden_extensions(self):
-        return ForbiddenExtension.objects.filter(project_id=self.submission_id)
+        return ForbiddenExtension.objects.filter(project_id=self.project_id)
 
     @property
     def get_allowed_extensions(self):
-        return AllowedExtension.objects.filter(project_id=self.submission_id)
+        return AllowedExtension.objects.filter(project_id=self.project_id)
 
 
 class AllowedExtension(models.Model):
     extension_id = models.BigAutoField(primary_key=True)
     project_id = models.ForeignKey(Project, on_delete=models.CASCADE)
-    extension = models.IntegerField()
+    extension = models.CharField(max_length=512)
 
     objects = models.Manager()
 
 
 class ForbiddenExtension(models.Model):
     extension_id = models.BigAutoField(primary_key=True)
     project_id = models.ForeignKey(Project, on_delete=models.CASCADE)
-    extension = models.IntegerField()
+    extension = models.CharField(max_length=512)
 
     objects = models.Manager()

backend/pigeonhole/apps/projects/permissions.py

@@ -0,0 +1,23 @@
+from rest_framework import permissions
+from backend.pigeonhole.apps.users.models import Teacher, Student
+
+
+class CanAccessProject(permissions.BasePermission):
+    # Custom permission class to determine if the currect user has access
+    # to the project data.
+    def has_permission(self, request, view):
+        user = request.user
+        subject_id = view.kwargs.get('course_id')
+        # If the user is a teacher, grant access.
+        if isinstance(user, Teacher):
+            if user.course.filter(id=subject_id).exists():
+                return True
+        elif isinstance(user, Teacher) and user.is_admin:
+            return True
+        # If the user is a student, grant access only to their own projects.
+        elif isinstance(user, Student):
+            if user.course.filter(id=subject_id).exists():
+                return True
+        elif request.user.is_superuser:
+            return True
+        return False