diff --git a/core/modules/user/tests/src/Kernel/UserRequirementsTest.php b/core/modules/user/tests/src/Kernel/UserRequirementsTest.php new file mode 100644 index 00000000000..88b406bc30e --- /dev/null +++ b/core/modules/user/tests/src/Kernel/UserRequirementsTest.php @@ -0,0 +1,65 @@ +container->get('module_handler')->loadInclude('user', 'install'); + $this->installEntitySchema('user'); + } + + /** + * Tests that the requirements check can detect conflicting user emails. + * + * @see \Drupal\Tests\user\Kernel\UserValidationTest::testValidation + */ + public function testConflictingUserEmails(): void { + + $output = \user_requirements('runtime'); + $this->assertArrayNotHasKey('conflicting emails', $output); + + $this->createUser([], 'User A', FALSE, ['mail' => 'unique@example.com']); + $this->createUser([], 'User B', FALSE, ['mail' => 'UNIQUE@example.com']); + + $output = \user_requirements('runtime'); + $this->assertArrayHasKey('conflicting emails', $output); + } + + /** + * Tests that the requirements check does not incorrectly flag blank emails. + */ + public function testBlankUserEmails(): void { + + $output = \user_requirements('runtime'); + $this->assertArrayNotHasKey('conflicting emails', $output); + + $this->createUser([], 'User A', FALSE, ['mail' => '']); + $this->createUser([], 'User B', FALSE, ['mail' => '']); + + $output = \user_requirements('runtime'); + $this->assertArrayNotHasKey('conflicting emails', $output); + } + +} diff --git a/core/modules/user/tests/src/Kernel/UserValidationTest.php b/core/modules/user/tests/src/Kernel/UserValidationTest.php index 77cf851cf60..6ad1d2475bf 100644 --- a/core/modules/user/tests/src/Kernel/UserValidationTest.php +++ b/core/modules/user/tests/src/Kernel/UserValidationTest.php @@ -137,6 +137,14 @@ class UserValidationTest extends KernelTestBase { $this->assertCount(1, $violations, 'Violation found when email already exists.'); $this->assertEquals('mail', $violations[0]->getPropertyPath()); $this->assertEquals('The email address existing@example.com is already taken.', $violations[0]->getMessage()); + + // Ensure case-insensitive uniqueness of email. + $user->set('mail', 'EXISTING@example.com'); + $violations = $user->validate(); + $this->assertCount(1, $violations, 'Violation found when email already exists.'); + $this->assertEquals('mail', $violations[0]->getPropertyPath()); + $this->assertEquals('The email address EXISTING@example.com is already taken.', $violations[0]->getMessage()); + $user->set('mail', NULL); $violations = $user->validate(); $this->assertCount(1, $violations, 'Email addresses may not be removed'); diff --git a/core/modules/user/user.install b/core/modules/user/user.install index 35ff4fe9d0e..8ef18ab6715 100644 --- a/core/modules/user/user.install +++ b/core/modules/user/user.install @@ -120,6 +120,7 @@ function user_requirements($phase): array { $query = \Drupal::database()->select('users_field_data'); $query->addExpression('LOWER(mail)', 'lower_mail'); + $query->isNotNull('mail'); $query->groupBy('lower_mail'); $query->having('COUNT(uid) > :matches', [':matches' => 1]); $conflicts = $query->countQuery()->execute()->fetchField();