From 5573e5c90239baff555f032cd8f7fcf589a483a3 Mon Sep 17 00:00:00 2001
From: P-Jeremy <jemyplu@gmail.com>
Date: Mon, 13 Jan 2025 16:28:49 +0100
Subject: [PATCH] feat(admin): allow email null when editing a user with
 userName connection

---
 admin/app/components/users/user-overview.gjs                | 6 +++++-
 admin/tests/acceptance/authenticated/users/get-test.js      | 2 +-
 .../integration/components/users/user-overview-test.gjs     | 6 ++++--
 3 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/admin/app/components/users/user-overview.gjs b/admin/app/components/users/user-overview.gjs
index 583cda1da38..8417dce71a5 100644
--- a/admin/app/components/users/user-overview.gjs
+++ b/admin/app/components/users/user-overview.gjs
@@ -82,6 +82,10 @@ export default class UserOverview extends Component {
     return hasBeenAnonymised || isPixAgent;
   }
 
+  get isEmailRequired() {
+    return this.args.user.username ? null : 'obligatoire';
+  }
+
   _initForm() {
     this.form.firstName = this.args.user.firstName;
     this.form.lastName = this.args.user.lastName;
@@ -201,7 +205,7 @@ export default class UserOverview extends Component {
             {{#if this.canModifyEmail}}
               <div class="form-field">
                 <PixInput
-                  @requiredLabel="obligatoire"
+                  @requiredLabel={{this.isEmailRequired}}
                   @errorMessage={{this.form.emailError.message}}
                   @validationStatus={{this.form.emailError.status}}
                   @value={{this.form.email}}
diff --git a/admin/tests/acceptance/authenticated/users/get-test.js b/admin/tests/acceptance/authenticated/users/get-test.js
index f3a14f34baf..f1bf95447f2 100644
--- a/admin/tests/acceptance/authenticated/users/get-test.js
+++ b/admin/tests/acceptance/authenticated/users/get-test.js
@@ -83,7 +83,7 @@ module('Acceptance | authenticated/users/get', function (hooks) {
       // when
       await fillIn(screen.getByLabelText('Prénom *', { exact: false }), 'john');
       await fillIn(screen.getByLabelText(/Nom */), 'doe');
-      await fillIn(screen.getByLabelText('Adresse e-mail *', { exact: false }), 'john.doe@example.net');
+      await fillIn(screen.getByLabelText('Adresse e-mail', { exact: false }), 'john.doe@example.net');
       await fillIn(screen.getByLabelText('Identifiant *', { exact: false }), 'john.doe0101');
       await click(screen.getByRole('button', { name: 'Langue' }));
 
diff --git a/admin/tests/integration/components/users/user-overview-test.gjs b/admin/tests/integration/components/users/user-overview-test.gjs
index ccf8c670f10..cb82c396c05 100644
--- a/admin/tests/integration/components/users/user-overview-test.gjs
+++ b/admin/tests/integration/components/users/user-overview-test.gjs
@@ -515,7 +515,7 @@ module('Integration | Component | users | user-overview', function (hooks) {
           assert.dom(screen.getByRole('textbox', { name: 'Identifiant *' })).hasValue(user.username);
         });
 
-        test('displays email', async function (assert) {
+        test('displays not required email', async function (assert) {
           // given
           const user = EmberObject.create({
             lastName: 'Harry',
@@ -529,7 +529,9 @@ module('Integration | Component | users | user-overview', function (hooks) {
           await clickByName('Modifier');
 
           // then
-          assert.dom(screen.getByRole('textbox', { name: 'Adresse e-mail *' })).exists();
+          const emailInput = screen.getByRole('textbox', { name: 'Adresse e-mail' });
+          assert.dom(emailInput).exists();
+          assert.dom(emailInput).hasNoAttribute('required');
         });
       });