Explicitly indicate that eml files are testdata

This is to prevent potential confusion
kiesraad · Apr 26, 2024 · f2863c7 · f2863c7
1 parent cc91e5a
commit f2863c7
Show file tree

Hide file tree

Showing 8 changed files with 11 additions and 7 deletions.
diff --git a/README.md b/README.md
@@ -25,7 +25,7 @@ Hieronder een korte beschrijving van de controles die onderdeel zijn van HCP. De
 | `check_potentially_switched_candidates` | Controleert of voorkeursstemmen tussen twee kandidaten op dezelfde lijst mogelijk verwisseld zijn. Dat wil zeggen dat een van de kandidaten veel meer (in te stellen hoeveel) stemmen heeft gekregen dan verwacht terwijl een ander veel minder gekregen heeft dan verwacht. | In `b.csv` de paren kandidaten waarvoor dit het geval is, gescheiden door een nieuwe regel (`\n`) in het formaat: *"Mogelijke verwisseling op lijst `i` (`lijstnaam`). Kandidaat `j` had `v_j` stemmen maar verwachting was `e_j`. Kandidaat `k` had `v_k` stemmen maar verwachting was `e_k`"*
 
 ## .odt verwerking
-Naast de checks die hierboven beschreven zijn, kan ook het proces-verbaal dat bij een telbestand meegeleverd wordt geparsed worden. Het doel hiervan is om stembureau's te identificeren die al een hertelling uitgevoerd hebben. Als dit het geval is, dan wordt in `a.csv` de waarde "x of ja" toegevoegd aan de kolom "Al hergeteld". In alle gevallen moet zowel het nummer als de naam van het stembureau genoteerd staan om zeker te weten dat we deze koppelen aan het juiste stembureau in de EML. Is deze koppeling om welke reden dan ook niet mogelijk, dan gaan we er van uit dat er **niet** herteld is.
+Naast de checks die hierboven beschreven zijn, kan ook het proces-verbaal dat bij een telbestand meegeleverd wordt geparsed worden. Het doel hiervan is om stembureau's te identificeren die al een hertelling uitgevoerd hebben. Als dit het geval is, dan wordt in `a.csv` de waarde "x of ja" toegevoegd aan de kolom "Al herteld". In alle gevallen moet zowel het nummer als de naam van het stembureau genoteerd staan om zeker te weten dat we deze koppelen aan het juiste stembureau in de EML. Is deze koppeling om welke reden dan ook niet mogelijk, dan gaan we er van uit dat er **niet** herteld is.
 
 De informatie is afkomstig uit:
 #### [Model_Na31-1.odt](https://www.rijksoverheid.nl/onderwerpen/verkiezingen/documenten/publicaties/2022/11/18/model-na-31-1)

diff --git a/...Telling_TK2023_gemeente_Dordrecht.eml.xml → ...Telling_TK2023_gemeente_Dordrecht.eml.xml b/...Telling_TK2023_gemeente_Dordrecht.eml.xml → ...Telling_TK2023_gemeente_Dordrecht.eml.xml
@@ -1,3 +1,4 @@
+<!-- This is TESTDATA and not a real election result -->
 <EML xmlns="urn:oasis:names:tc:evs:schema:eml"
     xmlns:ds="http://www.w3.org/2000/09/xmldsig#"
     xmlns:kr="http://www.kiesraad.nl/extensions"

diff --git a/test/data/TK2023_DORDRECHT/Model_Na31-1.odt → ...ST_DATA_TK2023_DORDRECHT/Model_Na31-1.odt b/test/data/TK2023_DORDRECHT/Model_Na31-1.odt → ...ST_DATA_TK2023_DORDRECHT/Model_Na31-1.odt
diff --git a/...g_EP2024_gemeente_Steenwijkerland.eml.xml → ...g_EP2024_gemeente_Steenwijkerland.eml.xml b/...g_EP2024_gemeente_Steenwijkerland.eml.xml → ...g_EP2024_gemeente_Steenwijkerland.eml.xml
@@ -1,3 +1,4 @@
+<!-- This is TESTDATA and not a real election result -->
 <EML xmlns="urn:oasis:names:tc:evs:schema:eml"
     xmlns:ds="http://www.w3.org/2000/09/xmldsig#"
     xmlns:kr="http://www.kiesraad.nl/extensions"

diff --git a/test/data/emls/empty_party_name.eml.xml b/test/data/emls/empty_party_name.eml.xml
@@ -1,3 +1,4 @@
+<!-- This is TESTDATA and not a real election result -->
 <EML xmlns="urn:oasis:names:tc:evs:schema:eml"
     xmlns:ds="http://www.w3.org/2000/09/xmldsig#"
     xmlns:kr="http://www.kiesraad.nl/extensions"

diff --git a/test/data/emls/invalid_eml_id.eml.xml b/test/data/emls/invalid_eml_id.eml.xml
@@ -1,3 +1,4 @@
+<!-- This is TESTDATA and not a real election result -->
 <EML xmlns="urn:oasis:names:tc:evs:schema:eml"
     xmlns:ds="http://www.w3.org/2000/09/xmldsig#"
     xmlns:kr="http://www.kiesraad.nl/extensions"

diff --git a/test/test_e2e.py b/test/test_e2e.py
@@ -8,10 +8,8 @@ def test_create_csv_files_a_b():
 
     This test case only includes one polling station with two inexplicable differences, which does show up in the .odt
     """
-    path_to_eml = (
-        "./test/data/TK2023_DORDRECHT/Telling_TK2023_gemeente_Dordrecht.eml.xml"
-    )
-    path_to_odt = "./test/data/TK2023_DORDRECHT/Model_Na31-1.odt"
+    path_to_eml = "./test/data/FAKE_TEST_DATA_TK2023_DORDRECHT/Fake_test_data_Telling_TK2023_gemeente_Dordrecht.eml.xml"
+    path_to_odt = "./test/data/FAKE_TEST_DATA_TK2023_DORDRECHT/Model_Na31-1.odt"
 
     temp_out_a = "./test/data/a.csv"
     temp_out_b = "./test/data/b.csv"
@@ -66,7 +64,9 @@ def test_create_csv_files_a_b_2():
     This test case is slightly more involved and tries to trigger each warning at least once,
     except for the switched candidates.
     """
-    path_to_eml = "./test/data/e2e/Telling_EP2024_gemeente_Steenwijkerland.eml.xml"
+    path_to_eml = (
+        "./test/data/e2e/Fake_test_data_Telling_EP2024_gemeente_Steenwijkerland.eml.xml"
+    )
 
     temp_out_a = "./test/data/a.csv"
     temp_out_b = "./test/data/b.csv"

diff --git a/test/test_eml.py b/test/test_eml.py
@@ -9,7 +9,7 @@
 )
 
 test_eml_paths = [
-    "./test/data/TK2023_DORDRECHT/Telling_TK2023_gemeente_Dordrecht.eml.xml",
+    "./test/data/FAKE_TEST_DATA_TK2023_DORDRECHT/Fake_test_data_Telling_TK2023_gemeente_Dordrecht.eml.xml",
     # "./test/data/emls/empty_party_name.eml.xml",
 ]