Refactor collection and group suggestion handling to improve name comparison logic
Updated the CreateCollectionFromSuggestionCommandHandler and CreateGroupFromSuggestionCommandHandler to compare names in memory rather than using SQL queries for case-insensitive checks. This change addresses potential discrepancies with SQL's case sensitivity, particularly with special characters. Additionally, modified unit tests to use array syntax for assertions, enhancing readability and consistency across the test suite.
This commit is contained in:
+14
-5
@@ -21,11 +21,20 @@ public sealed class CreateCollectionFromSuggestionCommandHandler(
|
||||
if (suggestion is null)
|
||||
return Result.Failure<Guid>(CollectionErrors.SuggestionNotFound);
|
||||
|
||||
var taken = await dbContext.Collections.AnyAsync(
|
||||
c => c.Name.ToLower() == suggestion.Name.ToLower(),
|
||||
cancellationToken
|
||||
);
|
||||
if (taken)
|
||||
// Сравниваем имена в памяти, а не запросом: коллекций десятки, зато проверка идёт по тем же
|
||||
// правилам, что и пометка «уже есть» в списке предложений. SQL-регистронезависимость
|
||||
// (lower()) в тонких местах вроде «ё» с ней расходится, и предложение то показывалось бы
|
||||
// доступным, то отбивалось конфликтом.
|
||||
var names = await dbContext.Collections.Select(c => c.Name).ToListAsync(cancellationToken);
|
||||
if (
|
||||
names.Any(name =>
|
||||
string.Equals(
|
||||
name.Trim(),
|
||||
suggestion.Name,
|
||||
StringComparison.CurrentCultureIgnoreCase
|
||||
)
|
||||
)
|
||||
)
|
||||
return Result.Failure<Guid>(CollectionErrors.SuggestionAlreadyCreated);
|
||||
|
||||
var collection = Collection.Create(suggestion.Name);
|
||||
|
||||
+13
-6
@@ -22,12 +22,19 @@ public sealed class CreateGroupFromSuggestionCommandHandler(
|
||||
if (suggestion is null)
|
||||
return Result.Failure<Guid>(GroupErrors.SuggestionNotFound);
|
||||
|
||||
// Имя — ключ узнаваемости группы, и две «Мультфильм» в списке слота неразличимы.
|
||||
var taken = await dbContext.Groups.AnyAsync(
|
||||
g => g.Name.ToLower() == suggestion.Name.ToLower(),
|
||||
cancellationToken
|
||||
);
|
||||
if (taken)
|
||||
// Имя — ключ узнаваемости группы, и две «Мультфильм» в списке слота неразличимы. Сравниваем
|
||||
// в памяти, а не запросом: групп десятки, зато проверка идёт по тем же правилам, что и
|
||||
// пометка «уже есть» в списке предложений, — SQL-регистронезависимость с ней расходится.
|
||||
var names = await dbContext.Groups.Select(g => g.Name).ToListAsync(cancellationToken);
|
||||
if (
|
||||
names.Any(name =>
|
||||
string.Equals(
|
||||
name.Trim(),
|
||||
suggestion.Name,
|
||||
StringComparison.CurrentCultureIgnoreCase
|
||||
)
|
||||
)
|
||||
)
|
||||
return Result.Failure<Guid>(GroupErrors.SuggestionAlreadyCreated);
|
||||
|
||||
var group = Group.Create(suggestion.Name);
|
||||
|
||||
Reference in New Issue
Block a user