fix(Import cantines): reparer le cas où on veut ajouter des gestionna…

…ires mais on n'a pas des colonnes vides en plus (#4754)

api/tests/files/canteen_manager_import.csv

@@ -0,0 +1 @@
+21340172201787,A canteen,,54460,,700,14000,,site,conceded,,manager@example.com

api/tests/test_import_diagnostics.py

@@ -285,6 +285,22 @@ def test_import_only_canteens(self, _):
         self.assertEqual(Canteen.objects.count(), 1)
         self.assertEqual(Canteen.objects.first().economic_model, None)
 
+    @authenticate
+    def test_import_canteens_with_managers(self, _):
+        """
+        Should be able to import canteens from a file that doesn't contain any diagnostic fields
+        """
+        manager = UserFactory(email="manager@example.com")
+        with open("./api/tests/files/canteen_manager_import.csv") as diag_file:
+            response = self.client.post(reverse("import_diagnostics"), {"file": diag_file})
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
+        body = response.json()
+        self.assertEqual(body["count"], 0)
+        self.assertEqual(len(body["errors"]), 0, body["errors"])
+        self.assertEqual(Diagnostic.objects.count(), 0)
+        self.assertEqual(Canteen.objects.count(), 1)
+        self.assertIn(manager, Canteen.objects.first().managers.all())
+
     @authenticate
     def test_staff_import(self, _):
         """

api/views/diagnosticimport.py

@@ -332,7 +332,7 @@ def _validate_canteen(row):
     def _get_manager_emails_to_notify(self, row):
         try:
             manager_emails = []
-            if len(row) > self.manager_column_idx + 1 and row[self.manager_column_idx]:
+            if len(row) > self.manager_column_idx and row[self.manager_column_idx]:
                 manager_emails = ImportDiagnosticsView._get_manager_emails(row[self.manager_column_idx])
         except Exception:
             raise ValidationError(