fix Pierre-Sassoulas#134 with test

The code is taken from Pierre-Sassoulas#168, the test is based on fixture from Pierre-Sassoulas#177.
maximloginov · Oct 30, 2022 · dddf3df · dddf3df
1 parent 5099715
commit dddf3df
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/survey/templates/survey/question.html b/survey/templates/survey/question.html
@@ -9,7 +9,7 @@
     </thead> -->
     <tbody>
 {% for form in response_form %}
-	{% if form.field.widget.attrs.category == category.name or not form.field.widget.attrs.category %}
+	{% if form.field.widget.attrs.category == category.name or category.name == 'No category' and not form.field.widget.attrs.category %}
         <tr class="{% if form.errors%} danger {% endif %}">
             <td>
 	            {{ form.label|safe }}

diff --git a/survey/tests/views/test_survey_category.py b/survey/tests/views/test_survey_category.py
@@ -14,3 +14,12 @@ def test_category_description_none_not_shown(self):
         """
         response = self.client.get(reverse("survey-detail", args=(13,)))
         self.assertNotContains(response, "None")
+
+    def test_without_category_not_duplicated(self):
+        """
+        Checks that question without category is not duplicated when there is
+        a question with category in the same survey.
+
+        """
+        response = self.client.get(reverse("survey-detail", args=(13,)))
+        self.assertContains(response, "Question without category", count=1)