Backoffice : trier les contacts lors de l'export

Stabilise le test et rend le comportement de l'export plus prévisible pour un
humain.

apps/transport/lib/transport_web/controllers/backoffice/contact_controller.ex

@@ -83,6 +83,7 @@ defmodule TransportWeb.Backoffice.ContactController do
       left join notification_subscription ns_reuser on ns_reuser.contact_id = c.id and ns_reuser.role = 'reuser'
       group by 1
     ) t on t.contact_id = id
+    order by id
     """
 
     csv_header =

apps/transport/test/transport_web/controllers/backoffice/contact_controller_test.exs

@@ -337,6 +337,8 @@ defmodule TransportWeb.Backoffice.ContactControllerTest do
 
     assert Enum.count(csv_content) == 2
 
+    assert [relevant, irrelevant] = csv_content
+
     assert %{
              "id" => to_string(contact.id),
              "first_name" => contact.first_name,
@@ -365,10 +367,10 @@ defmodule TransportWeb.Backoffice.ContactControllerTest do
              "reuser_resources_changed" => "false",
              "is_producer" => "true",
              "is_reuser" => "false"
-           } == csv_content |> hd()
+           } == relevant
 
     # `array_agg` with only null values is properly encoded
-    assert %{"organization_names" => ""} = Enum.at(csv_content, 1)
+    assert %{"organization_names" => ""} = irrelevant
   end
 
   defp sample_contact_args(%{} = args \\ %{}) do