"tutoren" removed from tests and veranstalter.py

d120 · Jul 24, 2024 · e2c9e73 · e2c9e73
1 parent 17b4ae8
commit e2c9e73
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 24 deletions.
diff --git a/src/feedback/tests/test_views_veranstalter.py b/src/feedback/tests/test_views_veranstalter.py
@@ -126,15 +126,11 @@ def test_post_bestellung(self):
             "freie_fragen-freifrage2": "Ist das die zweite Frage?"
         })
 
-        self.assertTemplateUsed(response_fifth_step, "formtools/wizard/tutoren.html")
-
         self.assertEqual(Tutor.objects.count(), 0)
-        response_sixth_step = c.post('/veranstalter/bestellung', {
-            "veranstalter_wizard-current_step": "tutoren",
-            "tutoren-csv_tutoren": "Müller,Max,[email protected],Bemerkung1\nMustermann,Erika,[email protected]"
-        })
 
-        self.assertTemplateUsed(response_sixth_step, "formtools/wizard/veroeffentlichen.html")
+        # step "tutoren" is removed
+
+        self.assertTemplateUsed(response_fifth_step, "formtools/wizard/veroeffentlichen.html")
 
         response_seventh_step = c.post('/veranstalter/bestellung', {'veroeffentlichen-veroeffentlichen': True,
                                     "veranstalter_wizard-current_step": "veroeffentlichen"})
@@ -152,7 +148,7 @@ def test_post_bestellung(self):
 
         self.assertTrue(self.v.evaluieren)
         self.assertEqual(self.v.primaerdozent, self.p)
-        self.assertEqual(Tutor.objects.count(), 2)
+        self.assertEqual(Tutor.objects.count(), 0) # step "tutoren" removed
         self.assertEqual(self.p.email, "[email protected]")
         self.assertEqual(self.v.anzahl, 22)
         self.assertEqual(self.v.ergebnis_empfaenger.count(), 2)
@@ -251,14 +247,11 @@ def test_post_bestellung_ein_ergebnis_empfaenger(self):
             "freie_fragen-freifrage2": "Ist das die zweite Frage?"
         })
 
-        self.assertTemplateUsed(response_fifth_step, "formtools/wizard/tutoren.html")
-
         self.assertEqual(Tutor.objects.count(), 0)
-        response_sixth_step = c.post('/veranstalter/bestellung', {
-            "veranstalter_wizard-current_step": "tutoren",
-            "tutoren-csv_tutoren": "Müller,Max,[email protected],Bemerkung1\nMustermann,Erika,[email protected]"
-        })
-        self.assertTemplateUsed(response_sixth_step, "formtools/wizard/veroeffentlichen.html")
+
+        # step 6 "tutoren" is removed
+
+        self.assertTemplateUsed(response_fifth_step, "formtools/wizard/veroeffentlichen.html")
 
         response_seventh_step = c.post('/veranstalter/bestellung', {'veroeffentlichen-veroeffentlichen': True,
                                     "veranstalter_wizard-current_step": "veroeffentlichen"})
@@ -275,7 +268,7 @@ def test_post_bestellung_ein_ergebnis_empfaenger(self):
         self.assertTemplateUsed(response_eight_step, "formtools/wizard/bestellung_done.html")
         self.assertTrue(self.v.evaluieren)
         self.assertEqual(self.v.primaerdozent, self.p2)
-        self.assertEqual(Tutor.objects.count(), 2)
+        self.assertEqual(Tutor.objects.count(), 0) # step "tutoren" removed
         self.assertEqual(self.p.email, "[email protected]")
 
     def test_post_bestellung_without_excercises(self):
@@ -362,10 +355,7 @@ def test_status_changes(self):
             "freie_fragen-freifrage2": "Ist das die zweite Frage?"
         })
 
-        c.post('/veranstalter/bestellung', {
-            "veranstalter_wizard-current_step": "tutoren",
-            "tutoren-csv_tutoren": "Müller,Max,[email protected],Bemerkung1\nMustermann,Erika,[email protected]"
-        })
+        # step "tutoren" removed
 
         c.post('/veranstalter/bestellung', {
             'veroeffentlichen-veroeffentlichen': True,

diff --git a/src/feedback/views/veranstalter.py b/src/feedback/views/veranstalter.py
@@ -87,7 +87,6 @@ def veranstalter_dashboard(request):
     "primaerdozent": "formtools/wizard/primaerdozent.html",
     "verantwortlicher_address": "formtools/wizard/address.html",
     "freie_fragen": "formtools/wizard/freiefragen.html",
-    "tutoren": "formtools/wizard/tutoren.html",
     "veroeffentlichen": "formtools/wizard/veroeffentlichen.html",
     "zusammenfassung": "formtools/wizard/zusammenfassung.html"
 }
@@ -100,7 +99,6 @@ def veranstalter_dashboard(request):
     "primaerdozent": "Primärdozent",
     "verantwortlicher_address": "Verantwortlicher",
     "freie_fragen": "Freie Fragen",
-    "tutoren": "Tutoren",
     "veroeffentlichen": "Veroeffentlichen",
     "zusammenfassung": "Zusammenfassung"
 }
@@ -173,7 +171,6 @@ class VeranstalterWizard(SessionWizardView):
         ('primaerdozent', VeranstaltungPrimaerDozentForm),
         ('verantwortlicher_address', VeranstaltungDozentDatenForm),
         ('freie_fragen', VeranstaltungFreieFragen),
-        ('tutoren', VeranstaltungTutorenForm),
         ('veroeffentlichen', VeranstaltungVeroeffentlichung),
         ('zusammenfassung', forms.Form)
     ]
@@ -184,7 +181,6 @@ class VeranstalterWizard(SessionWizardView):
         'primaerdozent': show_primaerdozent_form,
         'verantwortlicher_address': perform_evalution,
         'freie_fragen': perform_evalution,
-        'tutoren': show_tutor_form,
         'veroeffentlichen': perform_evalution,
     }