Skip to content

Commit c294810

Browse files
committed
fixup! fixed tests
1 parent b886c8d commit c294810

File tree

1 file changed

+13
-9
lines changed

1 file changed

+13
-9
lines changed

evap/staff/tests/test_views.py

+13-9
Original file line numberDiff line numberDiff line change
@@ -932,21 +932,25 @@ def setUpTestData(cls):
932932
def test_assign_questionnaires(self) -> None:
933933
page = self.app.get(self.url, user=self.manager)
934934
assign_form = page.forms["questionnaire-assign-form"]
935-
assign_form["general-" + str(self.seminar_type.id)] = [self.questionnaire_general.pk]
936-
assign_form["contributor-" + str(self.lecture_type.id)] = [self.questionnaire_contributor.pk]
935+
assign_form[f"general-{self.seminar_type.id}"] = [self.questionnaire_general.pk]
936+
assign_form[f"contributor-{self.lecture_type.id}"] = [self.questionnaire_contributor.pk]
937937
page = assign_form.submit().follow()
938938

939939
for evaluation in self.semester.evaluations.all():
940940
if evaluation.course.type == self.seminar_type:
941-
self.assertEqual(evaluation.general_contribution.questionnaires.count(), 1)
942-
self.assertEqual(evaluation.general_contribution.questionnaires.get(), self.questionnaire_general)
941+
self.assertQuerySetEqual(
942+
evaluation.general_contribution.questionnaires.all(), [self.questionnaire_general]
943+
)
943944
for contribution in evaluation.contributions.exclude(contributor=None): # contributions without general
944945
self.assertEqual(contribution.questionnaires.count(), 0)
945946
if evaluation.course.type == self.lecture_type:
946947
self.assertEqual(evaluation.general_contribution.questionnaires.count(), 0)
947948
for contribution in evaluation.contributions.exclude(contributor=None):
948-
self.assertEqual(contribution.questionnaires.count(), 1)
949-
self.assertEqual(contribution.questionnaires.get(), self.questionnaire_contributor)
949+
self.assertQuerySetEqual(
950+
contribution.questionnaires.all(), [self.questionnaire_contributor]
951+
)
952+
#self.assertEqual(contribution.questionnaires.count(), 1)
953+
#self.assertEqual(contribution.questionnaires.get(), self.questionnaire_contributor)
950954

951955

952956
class TestSemesterQuestionnaireAssignment(WebTestStaffMode):
@@ -988,9 +992,9 @@ def setUpTestData(cls):
988992
def test_questionnaire_assignment(self):
989993
page = self.app.get(self.url, user=self.manager, status=200)
990994
form = page.forms["questionnaire-assign-form"]
991-
form["general-" + str(self.course_type_1.id)] = [self.questionnaire_1.pk, self.questionnaire_2.pk]
992-
form["general-" + str(self.course_type_2.id)] = [self.questionnaire_2.pk]
993-
form["contributor-" + str(self.course_type_1.id)] = [self.questionnaire_responsible.pk]
995+
form[f"general-{self.course_type_1.id}"] = [self.questionnaire_1.pk, self.questionnaire_2.pk]
996+
form[f"general-{self.course_type_2.id}"] = [self.questionnaire_2.pk]
997+
form[f"contributor-{self.course_type_1.id}"] = [self.questionnaire_responsible.pk]
994998

995999
response = form.submit().follow()
9961000
self.assertIn("Successfully", str(response))

0 commit comments

Comments
 (0)