Change output text for topdesk import

Makes sure that the text is non-ambiguous
kiesraad · Apr 25, 2024 · 7d39790 · 7d39790
1 parent 4e334dd
commit 7d39790
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 6 deletions.
diff --git a/src/eml_types.py b/src/eml_types.py
@@ -164,14 +164,26 @@ def render(self, recounted: bool) -> str:
         sentence = Sentence()
 
         if summary_type == SummaryType.A:
-            if self.inexplicable_difference:
+            if self.inexplicable_difference and not self.explanation_sum_difference:
                 sentence.add(
-                    f"een onverklaard verschil van {self.inexplicable_difference}"
+                    "een onverklaard verschil tussen het aantal toegelaten kiezers en "
+                    f"het aantal getelde stembiljetten van {self.inexplicable_difference}"
                 )
-            if self.explanation_sum_difference:
+            elif self.explanation_sum_difference and not self.inexplicable_difference:
                 sentence.add(
-                    f"een aantal ontbrekende verklaringen van {self.explanation_sum_difference}"
+                    "een onverklaard verschil tussen het aantal toegelaten kiezers en het "
+                    f"aantal getelde stembiljetten van {self.explanation_sum_difference}. "
+                    "In het proces-verbaal tellen de verklaringen die gegeven zijn niet op tot "
+                    "het verschil tussen het aantal toegelaten kiezers en het aantal getelde stembiljetten"
                 )
+            elif self.explanation_sum_difference and self.inexplicable_difference:
+                sentence.add(
+                    "een onverklaard verschil tussen het aantal toegelaten kiezers en het aantal "
+                    f"getelde stembiljetten van {self.inexplicable_difference + self.explanation_sum_difference}. "
+                    f"In het proces-verbaal is ingevuld dat er {self.inexplicable_difference} keer geen verklaring "
+                    "is voor het verschil. De verklaringen die gegeven zijn tellen niet op tot het totale verschil"
+                )
+
         elif summary_type == SummaryType.B:
             if self.zero_votes:
                 sentence.add("een aantal uitgebrachte stemmen van 0")

diff --git a/test/test_e2e.py b/test/test_e2e.py
@@ -91,8 +91,8 @@ def test_create_csv_files_a_b_2():
             "Gemeentenummer;1708\n"
             "\n"
             "Verkiezingnummer;Kieskringnummer;Gemeentenummer;Gemeentenaam;Stembureaunummer;Stembureaunaam;Aantal geen verklaring voor verschil;Aantal ontbrekende verklaringen voor verschil;Al herteld;Samenvatting\n"
-            "EP2024;alle;1708;Steenwijkerland;1;Stembureau Holterberg;1;405;;Er is een onverklaard verschil van 1. Daarnaast is er een aantal ontbrekende verklaringen van 405. Er is niet herteld.\n"
-            'EP2024;alle;1708;Steenwijkerland;3;"Stembureau ""De Waarschuwing""";108;;;Er is een onverklaard verschil van 108. Er is niet herteld.\n'
+            "EP2024;alle;1708;Steenwijkerland;1;Stembureau Holterberg;1;405;;Er is een onverklaard verschil tussen het aantal toegelaten kiezers en het aantal getelde stembiljetten van 406. In het proces-verbaal is ingevuld dat er 1 keer geen verklaring is voor het verschil. De verklaringen die gegeven zijn tellen niet op tot het totale verschil. Er is niet herteld.\n"
+            'EP2024;alle;1708;Steenwijkerland;3;"Stembureau ""De Waarschuwing""";108;;;Er is een onverklaard verschil tussen het aantal toegelaten kiezers en het aantal getelde stembiljetten van 108. Er is niet herteld.\n'
         )
         assert content == expected