Remove mention in UI/tests

vmcj · vmcj · commit b4018113004f · 2025-12-21T16:29:12.000+01:00
diff --git a/webapp/src/Controller/API/AccessController.php b/webapp/src/Controller/API/AccessController.php
@@ -222,7 +222,6 @@ public function getStatusAction(Request $request): Access
                         'team',
                         'roles',
                         'userid',
-                        'email',
                         'last_ip',
                         'enabled',
                     ]
diff --git a/webapp/src/Controller/Jury/UserController.php b/webapp/src/Controller/Jury/UserController.php
@@ -64,7 +64,6 @@ public function indexAction(): Response
             'username'   => ['title' => 'username', 'sort' => true, 'default_sort' => true],
             'externalid' => ['title' => 'external ID', 'sort' => true],
             'name'       => ['title' => 'name', 'sort' => true],
-            'email'      => ['title' => 'email', 'sort' => true],
             'user_roles' => ['title' => 'roles', 'sort' => true],
             'teamid'     => ['title' => '', 'sort' => false, 'render' => 'entity_id_badge'],
             'team'       => ['title' => 'team', 'sort' => true],
diff --git a/webapp/src/DataTransferObject/AddUser.php b/webapp/src/DataTransferObject/AddUser.php
@@ -14,8 +14,6 @@ class AddUser
     public function __construct(
         public readonly string $username,
         public readonly string $name,
-        #[OA\Property(format: 'email', nullable: true)]
-        public readonly ?string $email,
         #[OA\Property(nullable: true)]
         public readonly ?string $ip,
         #[OA\Property(format: 'password', nullable: true)]
diff --git a/webapp/src/DataTransferObject/UpdateUser.php b/webapp/src/DataTransferObject/UpdateUser.php
@@ -11,13 +11,12 @@ public function __construct(
         public readonly string $id,
         string $username,
         string $name,
-        ?string $email,
         ?string $ip,
         ?string $password,
         ?bool $enabled,
         ?string $teamId,
         array $roles
     ) {
-        parent::__construct($username, $name, $email, $ip, $password, $enabled, $teamId, $roles);
+        parent::__construct($username, $name, $ip, $password, $enabled, $teamId, $roles);
     }
 }
diff --git a/webapp/src/Form/Type/UserRegistrationType.php b/webapp/src/Form/Type/UserRegistrationType.php
@@ -15,7 +15,6 @@
 use Symfony\Component\DependencyInjection\Attribute\Autowire;
 use Symfony\Component\Form\AbstractType;
 use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
-use Symfony\Component\Form\Extension\Core\Type\EmailType;
 use Symfony\Component\Form\Extension\Core\Type\PasswordType;
 use Symfony\Component\Form\Extension\Core\Type\RepeatedType;
 use Symfony\Component\Form\Extension\Core\Type\SubmitType;
@@ -27,7 +26,6 @@
 use Symfony\Component\Intl\Countries;
 use Symfony\Component\OptionsResolver\OptionsResolver;
 use Symfony\Component\Validator\Constraints\Callback;
-use Symfony\Component\Validator\Constraints\Email;
 use Symfony\Component\Validator\Constraints\NotBlank;
 use Symfony\Component\Validator\Context\ExecutionContext;
 
@@ -59,15 +57,6 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
                     'autocomplete' => 'name',
                 ],
             ])
-            ->add('email', EmailType::class, [
-                'label' => false,
-                'required' => false,
-                'attr' => [
-                    'placeholder' => 'Email address (optional)',
-                    'autocomplete' => 'email',
-                ],
-                'constraints' => new Email(),
-            ])
             ->add('teamName', TextType::class, [
                 'label' => false,
                 'attr' => [
diff --git a/webapp/src/Form/Type/UserType.php b/webapp/src/Form/Type/UserType.php
@@ -10,7 +10,6 @@
 use Symfony\Bridge\Doctrine\Form\Type\EntityType;
 use Symfony\Component\DependencyInjection\Attribute\Autowire;
 use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
-use Symfony\Component\Form\Extension\Core\Type\EmailType;
 use Symfony\Component\Form\Extension\Core\Type\PasswordType;
 use Symfony\Component\Form\Extension\Core\Type\SubmitType;
 use Symfony\Component\Form\Extension\Core\Type\TextType;
@@ -53,12 +52,6 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
             'help' => 'Optional full name for the user.',
             'empty_data' => ''
         ]);
-        $builder->add('email', EmailType::class, [
-            'required' => false,
-            'attr' => [
-                'autocomplete' => 'user-email',
-            ],
-        ]);
         $builder->add('plainPassword', PasswordType::class, [
             'required' => false,
             'label' => 'Password',
diff --git a/webapp/templates/jury/user.html.twig b/webapp/templates/jury/user.html.twig
@@ -28,16 +28,6 @@
                     <th>Login</th>
                     <td class="teamid">{{ user.username }}</td>
                 </tr>
-                <tr>
-                    <th>Email</th>
-                    <td>
-                        {% if user.email %}
-                            <a href="mailto:{{ user.email }}">{{ user.email }}</a>
-                        {% else %}
-                            -
-                        {% endif %}
-                    </td>
-                </tr>
                 <tr>
                     <th>Password</th>
                     <td>
diff --git a/webapp/tests/Unit/Controller/API/GeneralInfoControllerTest.php b/webapp/tests/Unit/Controller/API/GeneralInfoControllerTest.php
@@ -110,7 +110,7 @@ public function testUserEndpoint(string $username, string $fullname, string $tea
         static::assertTrue($response['enabled']);
         static::assertNull($response['last_login_time']);
         static::assertGreaterThanOrEqual($response['first_login_time'], $response['last_api_login_time']);
-        $keysExpected = ['id', 'ip', 'last_ip', 'email'];
+        $keysExpected = ['id', 'ip', 'last_ip'];
         foreach ($keysExpected as $keyExpected) {
             static::assertArrayHasKey($keyExpected, $response);
         }
diff --git a/webapp/tests/Unit/Controller/API/UserControllerTest.php b/webapp/tests/Unit/Controller/API/UserControllerTest.php
@@ -18,7 +18,6 @@ class UserControllerTest extends AccountBaseTestCase
             "id" => "admin",
             "username" => "admin",
             "name" => "Administrator",
-            "email" => null,
             "ip" => null,
             "enabled" => true
         ],
@@ -30,7 +29,6 @@ class UserControllerTest extends AccountBaseTestCase
             "id" => "judgehost",
             "username" => "judgehost",
             "name" => "User for judgedaemons",
-            "email" => null,
             "ip" => null,
             "enabled" => true
         ],
@@ -42,7 +40,6 @@ class UserControllerTest extends AccountBaseTestCase
             "id" => "demo",
             "username" => "demo",
             "name" => "demo user for example team",
-            "email" => null,
             "ip" => null,
             "enabled" => true
         ],
diff --git a/webapp/tests/Unit/Controller/Jury/UserControllerTest.php b/webapp/tests/Unit/Controller/Jury/UserControllerTest.php
@@ -20,19 +20,16 @@ class UserControllerTest extends JuryControllerTestCase
     protected static string  $className                = User::class;
     protected static array   $DOM_elements             = ['h1' => ['Users']];
     protected static string  $addForm                  = 'user[';
-    protected static array   $addEntitiesShown         = ['name', 'username', 'email'];
+    protected static array   $addEntitiesShown         = ['name', 'username'];
     protected static array   $overviewSingleNotShown   = ['plainPassword'];
     protected static array   $addEntities              = [['username'      => 'un',
                                                            'name'          => 'Alice',
-                                                           'email'         => 'alice@example.org',
                                                            'plainPassword' => 'plainpassword',
                                                            'ipAddress'     => '10.0.0.0',
                                                            'enabled'       => '1',
                                                            'team'          => '0',
                                                            'user_roles'    => ['0' => true, '1' => true]],
                                                           ['username' => 'ABCabc123_-@', 'name' => 'Allowed username'],
-                                                          ['username' => 'nem', 'name' => 'No Email',
-                                                           'email' => ''],
                                                           ['username' => 'npw', 'name' => 'No password',
                                                            'plainPassword' => ''],
                                                           ['username' => 'specialchar', 'name' => 'Special char in password',
diff --git a/webapp/tests/Unit/Controller/PublicControllerTest.php b/webapp/tests/Unit/Controller/PublicControllerTest.php
@@ -22,7 +22,7 @@ class PublicControllerTest extends BaseTestCase
     protected static string $urlTeams        = '/jury/teams';
     protected static string $urlAffil        = '/jury/affiliations';
     protected static array  $requiredFields  = ['teamName','affiliationName','affiliationShortName','existingAffiliation'];
-    protected static array  $formFields      = ['username','name','email','teamName','affiliation','affiliationName',
+    protected static array  $formFields      = ['username','name','teamName','affiliation','affiliationName',
                                                 'affiliationShortName','affiliationCountry','existingAffiliation'];
     protected static array  $duplicateFields = ['username'=>['input'=>'selfregister','error'=>'The username \'"selfregistered"\' is already in use.'],
                                                 'teamName'=>['input'=>'Example teamname','error'=>'This team name is already in use.'],
@@ -199,7 +199,7 @@ public function testSelfRegisterWrongPassword(
         self::assertSelectorExists($selector);
     }
 
-    // username, name, email, teamName, affiliation, affiliationName
+    // username, name, teamName, affiliation, affiliationName
     // affiliationShortName, affiliationCountry, existingAffiliation
     // plainPassword
     public function selfRegisterProvider(): Generator
@@ -211,7 +211,7 @@ public function selfRegisterProvider(): Generator
                 }
                 yield[['username'=>'minimaluser', 'teamName'=>'NewTeam','affiliation'=>'none'],'shirt-recognize-bar-together', $fixtures, $category];
                 yield[['username'=>'bruteforce', 'teamName'=>'Fib(9)','affiliation'=>'none'],'01123581321', $fixtures, $category];
-                yield[['username'=>'fullUser', 'name'=>'Full User', 'email'=>'email@domain.com','teamName'=>'Trial','affiliation'=>'none'],'..........', $fixtures, $category];
+                yield[['username'=>'fullUser', 'name'=>'Full User', 'teamName'=>'Trial','affiliation'=>'none'],'..........', $fixtures, $category];
                 yield[['username'=>'student@', 'teamName'=>'Student@Uni',
                        'affiliation'=>'new','affiliationName'=>'NewUni','affiliationShortName'=>'nu'],'p@ssword_Is_long', $fixtures, $category];
                 yield[['username'=>'winner@', 'teamName'=>'FunnyTeamname',
@@ -220,7 +220,7 @@ public function selfRegisterProvider(): Generator
                 yield[['username'=>'newinstsamecountry', 'name'=>'CompetingDutchTeam', 'teamName'=>'SupperT3@m','affiliation'=>'new','affiliationName'=>'Vrije Universiteit',
                        'affiliationShortName'=>'vu','affiliationCountry'=>'NLD'],'demodemodemo', $fixtures, $category];
                 if (count($fixtures)===1) {
-                    yield[['username'=>'reusevaluesofexistinguser', 'name'=>'selfregistered user for example team','email'=>'electronic@mail.tld','teamName'=>'EasyEnough','affiliation'=>'none'],'demodemodemo', [...$fixtures, SelfRegisteredUserFixture::class],''];
+                    yield[['username'=>'reusevaluesofexistinguser', 'name'=>'selfregistered user for example team','teamName'=>'EasyEnough','affiliation'=>'none'],'demodemodemo', [...$fixtures, SelfRegisteredUserFixture::class],''];
                 }
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -222,7 +222,6 @@ public function getStatusAction(Request $request): Access`
`222`	`222`	`'team',`
`223`	`223`	`'roles',`
`224`	`224`	`'userid',`
`225`		`- 'email',`
`226`	`225`	`'last_ip',`
`227`	`226`	`'enabled',`
`228`	`227`	`]`
Original file line number	Diff line number	Diff line change
`@@ -110,7 +110,7 @@ public function testUserEndpoint(string $username, string $fullname, string $tea`
`110`	`110`	`static::assertTrue($response['enabled']);`
`111`	`111`	`static::assertNull($response['last_login_time']);`
`112`	`112`	`static::assertGreaterThanOrEqual($response['first_login_time'], $response['last_api_login_time']);`
`113`		`- $keysExpected = ['id', 'ip', 'last_ip', 'email'];`
	`113`	`+ $keysExpected = ['id', 'ip', 'last_ip'];`
`114`	`114`	`foreach ($keysExpected as $keyExpected) {`
`115`	`115`	`static::assertArrayHasKey($keyExpected, $response);`
`116`	`116`	`}`