From 0c9cbff9dd67ad8c3048c83d98e62078ab451be9 Mon Sep 17 00:00:00 2001
From: Anto59290 <antonin59290@hotmail.com>
Date: Thu, 14 Nov 2024 16:50:20 +0100
Subject: [PATCH] Renomer le type d'exploitant pour le lieu
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Le type d'exploitant ne correspond pas à la réalité des choses, on veut
en réalité connaitre le site de l'inspection.
On réutilise le modèle qui était précédement utilisé pour les
prélèvements.

Au passage je corrige un test qui assert que les attributs d'un objet
sont bien égals a eux même.
---
 sv/admin.py                                   |  2 -
 ..._type_exploitant_etablissement_and_more.py | 31 ++++++++++++
 sv/models.py                                  | 18 ++-----
 sv/static/sv/fichedetection_form.js           |  6 +--
 .../sv/_fichedetection_form__lieux_form.html  | 19 +++----
 sv/tests/test_fichedetection_create.py        | 50 +++++++++----------
 sv/tests/test_fichedetection_update.py        | 10 ++--
 sv/tests/test_utils.py                        |  8 +--
 sv/views.py                                   |  9 ++--
 9 files changed, 83 insertions(+), 70 deletions(-)
 create mode 100644 sv/migrations/0035_remove_lieu_type_exploitant_etablissement_and_more.py

diff --git a/sv/admin.py b/sv/admin.py
index 9dff83ee..e4edb488 100644
--- a/sv/admin.py
+++ b/sv/admin.py
@@ -7,7 +7,6 @@
     Region,
     Lieu,
     StatutEtablissement,
-    TypeExploitant,
     PositionChaineDistribution,
     StructurePreleveur,
     SiteInspection,
@@ -33,7 +32,6 @@
 admin.site.register(Region)
 admin.site.register(Lieu)
 admin.site.register(StatutEtablissement)
-admin.site.register(TypeExploitant)
 admin.site.register(PositionChaineDistribution)
 admin.site.register(StructurePreleveur)
 admin.site.register(SiteInspection)
diff --git a/sv/migrations/0035_remove_lieu_type_exploitant_etablissement_and_more.py b/sv/migrations/0035_remove_lieu_type_exploitant_etablissement_and_more.py
new file mode 100644
index 00000000..faf06889
--- /dev/null
+++ b/sv/migrations/0035_remove_lieu_type_exploitant_etablissement_and_more.py
@@ -0,0 +1,31 @@
+# Generated by Django 5.0.8 on 2024-11-15 07:49
+
+import django.db.models.deletion
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("sv", "0034_remove_prelevement_site_inspection"),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name="lieu",
+            name="type_exploitant_etablissement",
+        ),
+        migrations.AddField(
+            model_name="lieu",
+            name="site_inspection",
+            field=models.ForeignKey(
+                blank=True,
+                null=True,
+                on_delete=django.db.models.deletion.PROTECT,
+                to="sv.siteinspection",
+                verbose_name="Site d'inspection",
+            ),
+        ),
+        migrations.DeleteModel(
+            name="TypeExploitant",
+        ),
+    ]
diff --git a/sv/models.py b/sv/models.py
index 72028ef1..38f44e66 100644
--- a/sv/models.py
+++ b/sv/models.py
@@ -170,10 +170,10 @@ class Meta:
     )
     adresse_etablissement = models.CharField(max_length=100, verbose_name="Adresse établissement", blank=True)
     siret_etablissement = models.CharField(max_length=100, verbose_name="SIRET établissement", blank=True)
-    type_exploitant_etablissement = models.ForeignKey(
-        "TypeExploitant",
+    site_inspection = models.ForeignKey(
+        "SiteInspection",
         on_delete=models.PROTECT,
-        verbose_name="Type d'exploitant établissement",
+        verbose_name="Site d'inspection",
         blank=True,
         null=True,
     )
@@ -202,18 +202,6 @@ def __str__(self):
         return self.libelle
 
 
-class TypeExploitant(models.Model):
-    class Meta:
-        verbose_name = "Type d'exploitant"
-        verbose_name_plural = "Types d'exploitant"
-        db_table = "sv_type_exploitant"
-
-    libelle = models.CharField(max_length=100, verbose_name="Libellé")
-
-    def __str__(self):
-        return self.libelle
-
-
 class PositionChaineDistribution(models.Model):
     class Meta:
         verbose_name = "Position dans la chaîne de distribution"
diff --git a/sv/static/sv/fichedetection_form.js b/sv/static/sv/fichedetection_form.js
index 1cf4267c..e2b584c7 100644
--- a/sv/static/sv/fichedetection_form.js
+++ b/sv/static/sv/fichedetection_form.js
@@ -146,7 +146,7 @@ document.addEventListener('alpine:init', () => {
             adresseEtablissement: '',
             siretEtablissement: '',
             codeInppEtablissement: '',
-            typeEtablissementId: '',
+            siteInspectionId: '',
             positionEtablissementId: '',
         },
 
@@ -233,7 +233,7 @@ document.addEventListener('alpine:init', () => {
                         adresseEtablissement: lieu.adresse_etablissement,
                         siretEtablissement: lieu.siret_etablissement,
                         codeInppEtablissement: lieu.code_inpp_etablissement,
-                        typeEtablissementId: lieu.type_exploitant_etablissement_id,
+                        siteInspectionId: lieu.site_inspection_id,
                         positionEtablissementId: lieu.position_chaine_distribution_etablissement_id,
                     };
                 });
@@ -393,7 +393,7 @@ document.addEventListener('alpine:init', () => {
                 adresseEtablissement: '',
                 siretEtablissement: '',
                 codeInppEtablissement: '',
-                typeEtablissementId: '',
+                siteInspectionId: '',
                 positionEtablissementId: '',
 
             };
diff --git a/sv/templates/sv/_fichedetection_form__lieux_form.html b/sv/templates/sv/_fichedetection_form__lieux_form.html
index 9cd71f4c..148ec3fe 100644
--- a/sv/templates/sv/_fichedetection_form__lieux_form.html
+++ b/sv/templates/sv/_fichedetection_form__lieux_form.html
@@ -28,6 +28,16 @@
                             <input x-model="lieuForm.codeINSEE" type="hidden" id="code-insee">
                             <input x-model="lieuForm.departementNom" type="hidden" id="departement">
 
+                            <p>
+                                <label class="fr-label" for="site-inspection">Site d'inspection</label>
+                                <select x-model="lieuForm.siteInspectionId" class="fr-select" data-testid="lieu-form-type-etablissement" id="site-inspection">
+                                    <option value="">----</option>
+                                    {% for site_inspection in sites_inspections %}
+                                        <option value="{{ site_inspection.id }}">{{ site_inspection.libelle }}</option>
+                                    {% endfor %}
+                                </select>
+                            </p>
+
                             <p>
                                 <label class="fr-label" for="coordonnees-gps-lambert-93-latitude">Coordonnées GPS (Lambert 93)</label>
                                 <input x-model="lieuForm.coordGPSLambert93Latitude" style="flex: 0.55; margin-right: 0.5rem;" type="number" step="1" min="6000000" max="7200000" id="coordonnees-gps-lambert-93-latitude" class="fr-input" placeholder="Latitude">
@@ -83,15 +93,6 @@
                                     <label class="fr-label" for="code-inpp-etablissement">Code INPP</label>
                                     <input x-model="lieuForm.codeInppEtablissement" type="text" class="fr-input" id="code-inpp-etablissement">
                                 </p>
-                                <p>
-                                    <label class="fr-label" for="type-etablissement">Type</label>
-                                    <select x-model="lieuForm.typeEtablissementId" class="fr-select" data-testid="lieu-form-type-etablissement" id="type-etablissement">
-                                        <option value="">----</option>
-                                        {% for type_etablissement in types_etablissement %}
-                                            <option value="{{ type_etablissement.id }}">{{ type_etablissement.libelle }}</option>
-                                        {% endfor %}
-                                    </select>
-                                </p>
                                 <p>
                                     <label class="fr-label" for="position-etablissement">Position</label>
                                     <select x-model="lieuForm.positionEtablissementId" class="fr-select" data-testid="lieu-form-position-etablissement" id="position-etablissement">
diff --git a/sv/tests/test_fichedetection_create.py b/sv/tests/test_fichedetection_create.py
index 619b22bf..45da2082 100644
--- a/sv/tests/test_fichedetection_create.py
+++ b/sv/tests/test_fichedetection_create.py
@@ -15,10 +15,10 @@
     OrganismeNuisible,
     Lieu,
     Departement,
-    TypeExploitant,
     PositionChaineDistribution,
     Region,
     StructurePreleveur,
+    SiteInspection,
 )
 
 from sv.constants import REGIONS, DEPARTEMENTS
@@ -228,7 +228,7 @@ def test_create_fiche_detection_with_lieu(
     fill_commune,
 ):
     dept = baker.make(Departement)
-    type_exploitant = baker.make(TypeExploitant)
+    site_inspection = baker.make(SiteInspection)
     position = baker.make(PositionChaineDistribution)
     lieu = baker.prepare(
         Lieu,
@@ -240,7 +240,7 @@ def test_create_fiche_detection_with_lieu(
         siret_etablissement="12345678901234",
         departement=dept,
         is_etablissement=True,
-        type_exploitant_etablissement=type_exploitant,
+        site_inspection=site_inspection,
         position_chaine_distribution_etablissement=position,
         _fill_optional=True,
     )
@@ -258,14 +258,14 @@ def test_create_fiche_detection_with_lieu(
     lieu_form_elements.coord_gps_wgs84_latitude_input.fill(str(lieu.wgs84_latitude))
     lieu_form_elements.coord_gps_wgs84_longitude_input.fill(str(lieu.wgs84_longitude))
     lieu_form_elements.is_etablissement_checkbox.click()
-    lieu_form_elements.nom_etablissement_input.fill(lieu.nom)
+    lieu_form_elements.nom_etablissement_input.fill(lieu.nom_etablissement)
     lieu_form_elements.activite_etablissement_input.fill(lieu.activite_etablissement)
     lieu_form_elements.pays_etablissement_input.fill(lieu.pays_etablissement)
     lieu_form_elements.raison_sociale_etablissement_input.fill(lieu.raison_sociale_etablissement)
     lieu_form_elements.adresse_etablissement_input.fill(lieu.adresse_etablissement)
     lieu_form_elements.siret_etablissement_input.fill(lieu.siret_etablissement)
     lieu_form_elements.code_inpp_etablissement_input.fill(lieu.code_inpp_etablissement)
-    lieu_form_elements.type_etablissement_input.select_option(str(lieu.type_exploitant_etablissement.id))
+    lieu_form_elements.lieu_site_inspection_input.select_option(str(lieu.site_inspection.id))
     lieu_form_elements.position_etablissement_input.select_option(
         str(lieu.position_chaine_distribution_etablissement.id)
     )
@@ -276,26 +276,26 @@ def test_create_fiche_detection_with_lieu(
     page.wait_for_timeout(1000)
 
     fiche_detection = FicheDetection.objects.get()
-    lieu = fiche_detection.lieux.get()
-    assert lieu.nom == lieu.nom
-    assert lieu.wgs84_latitude == lieu.wgs84_latitude
-    assert lieu.wgs84_longitude == lieu.wgs84_longitude
-    assert lieu.lambert93_latitude == lieu.lambert93_latitude
-    assert lieu.lambert93_longitude == lieu.lambert93_longitude
-    assert lieu.adresse_lieu_dit == lieu.adresse_lieu_dit
-    assert lieu.commune == "Lille"
-    assert lieu.code_insee == "59350"
-    assert lieu.departement == Departement.objects.get(nom="Nord")
-    assert lieu.is_etablissement == lieu.is_etablissement
-    assert lieu.nom_etablissement == lieu.nom_etablissement
-    assert lieu.activite_etablissement == lieu.activite_etablissement
-    assert lieu.pays_etablissement == lieu.pays_etablissement
-    assert lieu.raison_sociale_etablissement == lieu.raison_sociale_etablissement
-    assert lieu.adresse_etablissement == lieu.adresse_etablissement
-    assert lieu.siret_etablissement == lieu.siret_etablissement
-    assert lieu.code_inpp_etablissement == lieu.code_inpp_etablissement
-    assert lieu.type_exploitant_etablissement == lieu.type_exploitant_etablissement
-    assert lieu.position_chaine_distribution_etablissement == lieu.position_chaine_distribution_etablissement
+    lieu_from_db = fiche_detection.lieux.get()
+    assert lieu_from_db.nom == lieu.nom
+    assert lieu_from_db.wgs84_latitude == lieu.wgs84_latitude
+    assert lieu_from_db.wgs84_longitude == lieu.wgs84_longitude
+    assert lieu_from_db.lambert93_latitude == lieu.lambert93_latitude
+    assert lieu_from_db.lambert93_longitude == lieu.lambert93_longitude
+    assert lieu_from_db.adresse_lieu_dit == lieu.adresse_lieu_dit
+    assert lieu_from_db.commune == "Lille"
+    assert lieu_from_db.code_insee == "59350"
+    assert lieu_from_db.departement == Departement.objects.get(nom="Nord")
+    assert lieu_from_db.is_etablissement == lieu.is_etablissement
+    assert lieu_from_db.nom_etablissement == lieu.nom_etablissement
+    assert lieu_from_db.activite_etablissement == lieu.activite_etablissement
+    assert lieu_from_db.pays_etablissement == lieu.pays_etablissement
+    assert lieu_from_db.raison_sociale_etablissement == lieu.raison_sociale_etablissement
+    assert lieu_from_db.adresse_etablissement == lieu.adresse_etablissement
+    assert lieu_from_db.siret_etablissement == lieu.siret_etablissement
+    assert lieu_from_db.code_inpp_etablissement == lieu.code_inpp_etablissement
+    assert lieu_from_db.site_inspection == lieu.site_inspection
+    assert lieu_from_db.position_chaine_distribution_etablissement == lieu.position_chaine_distribution_etablissement
 
 
 def test_structure_contact_is_add_to_contacts_list_when_fiche_detection_is_created(
diff --git a/sv/tests/test_fichedetection_update.py b/sv/tests/test_fichedetection_update.py
index 7dae6968..083e957d 100644
--- a/sv/tests/test_fichedetection_update.py
+++ b/sv/tests/test_fichedetection_update.py
@@ -11,13 +11,13 @@
     Lieu,
     Prelevement,
     Departement,
-    TypeExploitant,
     PositionChaineDistribution,
     OrganismeNuisible,
     LaboratoireAgree,
     LaboratoireConfirmationOfficielle,
     StructurePreleveur,
     Etat,
+    SiteInspection,
 )
 from ..models import (
     Region,
@@ -354,7 +354,7 @@ def test_update_lieu(
     """Test que les modifications des descripteurs d'un lieu existant sont bien enregistrées en base de données."""
 
     dept = baker.make(Departement)
-    type_exploitant = baker.make(TypeExploitant)
+    site_inspection = baker.make(SiteInspection)
     position = baker.make(PositionChaineDistribution)
     new_lieu = baker.prepare(
         Lieu,
@@ -366,7 +366,7 @@ def test_update_lieu(
         siret_etablissement="12345678901234",
         departement=dept,
         is_etablissement=True,
-        type_exploitant_etablissement=type_exploitant,
+        site_inspection=site_inspection,
         position_chaine_distribution_etablissement=position,
         _fill_optional=True,
         _save_related=True,
@@ -388,7 +388,7 @@ def test_update_lieu(
     lieu_form_elements.raison_sociale_etablissement_input.fill(new_lieu.raison_sociale_etablissement)
     lieu_form_elements.adresse_etablissement_input.fill(new_lieu.adresse_etablissement)
     lieu_form_elements.siret_etablissement_input.fill(new_lieu.siret_etablissement)
-    lieu_form_elements.type_etablissement_input.select_option(str(new_lieu.type_exploitant_etablissement.id))
+    lieu_form_elements.lieu_site_inspection_input.select_option(str(new_lieu.site_inspection.id))
     lieu_form_elements.position_etablissement_input.select_option(
         str(new_lieu.position_chaine_distribution_etablissement.id)
     )
@@ -414,7 +414,7 @@ def test_update_lieu(
     assert lieu_from_db.raison_sociale_etablissement == new_lieu.raison_sociale_etablissement
     assert lieu_from_db.adresse_etablissement == new_lieu.adresse_etablissement
     assert lieu_from_db.siret_etablissement == new_lieu.siret_etablissement
-    assert lieu_from_db.type_exploitant_etablissement == new_lieu.type_exploitant_etablissement
+    assert lieu_from_db.site_inspection == new_lieu.site_inspection
     assert (
         lieu_from_db.position_chaine_distribution_etablissement == new_lieu.position_chaine_distribution_etablissement
     )
diff --git a/sv/tests/test_utils.py b/sv/tests/test_utils.py
index a7607260..5b4eae3e 100644
--- a/sv/tests/test_utils.py
+++ b/sv/tests/test_utils.py
@@ -334,12 +334,8 @@ def code_inpp_etablissement_input(self) -> Locator:
         return self.page.get_by_label("Code INPP")
 
     @property
-    def type_etablissement_label(self) -> Locator:
-        return self.page.get_by_text("Type")
-
-    @property
-    def type_etablissement_input(self) -> Locator:
-        return self.page.get_by_label("Type")
+    def lieu_site_inspection_input(self) -> Locator:
+        return self.page.get_by_label("Site d'inspection")
 
     @property
     def position_etablissement_label(self) -> Locator:
diff --git a/sv/views.py b/sv/views.py
index bb649206..7c8f7edc 100644
--- a/sv/views.py
+++ b/sv/views.py
@@ -57,7 +57,6 @@
     LaboratoireAgree,
     LaboratoireConfirmationOfficielle,
     Departement,
-    TypeExploitant,
     PositionChaineDistribution,
     FicheZoneDelimitee,
     ZoneInfestee,
@@ -177,7 +176,7 @@ def get_context_data(self, **kwargs):
             queryset = LaboratoireConfirmationOfficielle.objects.all().order_by("nom")
         context["laboratoires_confirmation_officielle"] = queryset
         context["resultats_prelevement"] = Prelevement.Resultat.choices
-        context["types_etablissement"] = TypeExploitant.objects.all().order_by("libelle")
+        context["sites_inspections"] = list(SiteInspection.objects.all().values("id", "nom").order_by("nom"))
         context["positions_chaine_distribution"] = PositionChaineDistribution.objects.all().order_by("libelle")
 
         status_code_to_id = {s.code: s.id for s in status}
@@ -375,7 +374,7 @@ def create_lieux(self, lieux, fiche):
                 adresse_etablissement=lieu["adresseEtablissement"],
                 siret_etablissement=lieu["siretEtablissement"],
                 code_inpp_etablissement=lieu["codeInppEtablissement"],
-                type_exploitant_etablissement_id=lieu["typeEtablissementId"],
+                site_inspection_id=lieu["siteInspectionId"],
                 position_chaine_distribution_etablissement_id=lieu["positionEtablissementId"],
             )
             lieu_instance.save()
@@ -547,7 +546,7 @@ def update_lieux(self, lieux, fiche_detection):
                 lieu.adresse_etablissement = loc["adresseEtablissement"]
                 lieu.siret_etablissement = loc["siretEtablissement"]
                 lieu.code_inpp_etablissement = loc["codeInppEtablissement"]
-                lieu.type_exploitant_etablissement_id = loc["typeEtablissementId"]
+                lieu.site_inspection_id = loc["siteInspectionId"]
                 lieu.position_chaine_distribution_etablissement_id = loc["positionEtablissementId"]
             else:
                 lieu.nom_etablissement = ""
@@ -556,7 +555,7 @@ def update_lieux(self, lieux, fiche_detection):
                 lieu.raison_sociale_etablissement = ""
                 lieu.adresse_etablissement = ""
                 lieu.siret_etablissement = ""
-                lieu.type_exploitant_etablissement_id = None
+                lieu.site_inspection_id = None
                 lieu.position_chaine_distribution_etablissement_id = None
             lieu.save()
             loc["lieu_pk"] = lieu.pk