Skip to content

Commit 57fe0f0

Browse files
committed
style(quiz): fixed linting issues
1 parent 84616b6 commit 57fe0f0

8 files changed

Lines changed: 24 additions & 36 deletions

File tree

server/api/apps/shared_models/admin.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,4 +22,3 @@ class QuestionAdmin(admin.ModelAdmin):
2222

2323
admin.site.register(Tag)
2424
admin.site.register(QuestionResponse)
25-
Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +0,0 @@
1-
from .quiz_models import Question, Tag, Answer
2-
from .statistics_models import QuestionResponse

server/api/apps/shared_models/models/quiz_models.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ class QuestionType(models.TextChoices):
1010
NUMERIC = "NA", _("Numerical Answer")
1111
SHORT_ANSWER = "SA", _("Short Answer")
1212

13-
1413
text = models.TextField(blank=True, default="")
1514
question_type = models.CharField(max_length=2,
1615
choices=QuestionType.choices,
@@ -25,7 +24,7 @@ def __str__(self):
2524
class Tag(models.Model):
2625
name = models.CharField(max_length=defines.TAG_NAME_MAXLEN)
2726
question = models.ManyToManyField(Question)
28-
27+
2928
def __str__(self):
3029
return self.name
3130

@@ -34,7 +33,6 @@ class Answer(models.Model):
3433
text = models.CharField(max_length=defines.ANSWER_TEXT_MAXLEN)
3534
is_correct = models.BooleanField(default=False)
3635
question = models.ForeignKey(Question, on_delete=models.CASCADE)
37-
36+
3837
def __str__(self):
3938
return self.text
40-

server/api/apps/shared_models/models/statistics_models.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
from django.db import models
2-
from . import defines
3-
from .quiz_models import *
2+
from .quiz_models import Question, Answer
43
from django.contrib.auth.models import User
54

65
from datetime import timedelta
@@ -18,4 +17,3 @@ class QuestionResponse(models.Model):
1817

1918
def __str__(self):
2019
return str((self.question, self.selected_answer))
21-
Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
1-
from rest_framework import routers, serializers, viewsets
2-
from ..models.quiz_models import *
1+
from rest_framework import serializers
2+
from ..models.quiz_models import Question, Tag, Answer
33

44

55
class QuestionSerializer(serializers.HyperlinkedModelSerializer):
66
class Meta:
77
model = Question
88
fields = ["id", "text", "question_type", "date_created"]
99

10-
10+
1111
class TagSerializer(serializers.HyperlinkedModelSerializer):
1212
class Meta:
1313
model = Tag
@@ -18,4 +18,3 @@ class AnswerSerializer(serializers.HyperlinkedModelSerializer):
1818
class Meta:
1919
model = Answer
2020
fields = ["id", "text", "is_correct", "question"]
21-

server/api/apps/shared_models/serializers/statistics_serializers.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
from rest_framework import routers, serializers, viewsets
2-
from ..models.statistics_models import *
1+
from rest_framework import serializers
2+
from ..models.statistics_models import QuestionResponse
33

44

55
class QuestionResponseSerializer(serializers.HyperlinkedModelSerializer):
@@ -11,4 +11,3 @@ class Meta:
1111
"is_correct",
1212
"time_taken",
1313
"date_created"]
14-

server/api/apps/shared_models/tests/test_quiz.py

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -2,20 +2,21 @@
22
from django.utils import timezone
33
from ..models.quiz_models import Question, Tag, Answer
44

5-
import datetime
5+
MC = Question.QuestionType.MULTICHOICE
6+
NA = Question.QuestionType.NUMERIC
7+
SA = Question.QuestionType.SHORT_ANSWER
68

79

810
class QuestionTestCase(TestCase):
911
def setUp(self):
1012
self.creation_time = timezone.now()
13+
1114
Question.objects.create(text="Test multiple choice question",
12-
question_type=
13-
Question.QuestionType.MULTICHOICE)
15+
question_type=MC)
1416
Question.objects.create(text="Test numerical answer question",
15-
question_type=Question.QuestionType.NUMERIC)
17+
question_type=NA)
1618
Question.objects.create(text="Test short answer question",
17-
question_type=
18-
Question.QuestionType.SHORT_ANSWER)
19+
question_type=SA)
1920

2021
def test_text(self):
2122
mc = Question.objects.get(text="Test multiple choice question")
@@ -51,8 +52,7 @@ def test_date_created(self):
5152
class TagTestCase(TestCase):
5253
def setUp(self):
5354
q = Question.objects.create(text="This is a physics question",
54-
question_type=
55-
Question.QuestionType.NUMERIC)
55+
question_type=NA)
5656
t = Tag.objects.create(name="Unit 3 Physics")
5757
q.tag_set.add(t)
5858

@@ -67,8 +67,7 @@ def test(self):
6767
class AnswerTestCase(TestCase):
6868
def setUp(self):
6969
q = Question.objects.create(text="Question?",
70-
question_type=
71-
Question.QuestionType.SHORT_ANSWER)
70+
question_type=SA)
7271
Answer.objects.create(text="Answer!", question=q, is_correct=True)
7372

7473
def test(self):
@@ -80,4 +79,3 @@ def test(self):
8079
self.assertTrue(a.is_correct)
8180

8281
self.assertEquals(a.question, q)
83-

server/api/apps/shared_models/tests/test_statistics.py

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,23 +3,23 @@
33
from ..models.quiz_models import Question, Answer
44
from ..models.statistics_models import QuestionResponse
55

6-
import datetime
6+
from datetime import timedelta
7+
8+
MC = Question.QuestionType.MULTICHOICE
79

810

911
class QuestionTestCase(TestCase):
1012
def setUp(self):
1113
self.creation_time = timezone.now()
1214
q = Question.objects.create(text="Question?",
13-
question_type=
14-
Question.QuestionType.MULTICHOICE)
15+
question_type=MC)
1516
a = Answer.objects.create(text="Answer!",
1617
question=q,
1718
is_correct=True)
1819
QuestionResponse.objects.create(question=q,
1920
selected_answer=a,
2021
is_correct=a.is_correct,
21-
time_taken=
22-
datetime.timedelta(seconds=5))
22+
time_taken=timedelta(seconds=5))
2323

2424
def test(self):
2525
q = Question.objects.get(text="Question?")
@@ -32,7 +32,6 @@ def test(self):
3232
self.assertEquals(qr.selected_answer, a)
3333
self.assertEquals(qr.is_correct, a.is_correct)
3434
self.assertEquals(qr.is_correct, True)
35-
self.assertEquals(qr.time_taken, datetime.timedelta(seconds=5))
35+
self.assertEquals(qr.time_taken, timedelta(seconds=5))
3636
self.assertLess((qr.date_created-self.creation_time).total_seconds(),
37-
0.1)
38-
37+
0.1)

0 commit comments

Comments
 (0)