Revert: Директор Tansu — снова удаляем дубликат из CompanyDirectors
Возврат к удалению совпавшей строки CompanyDirectors вместо
последовательной постановки после блока (см. 40d741c) — решили не
показывать одного и того же человека дважды в списке согласования.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
gorbenko.pavel
@@ -221,11 +221,8 @@ public sealed class TemplateRouteService(
|
||||
HasUnevaluatableManagerCondition(step, positionMissing) ? PositionMissingWarning : null,
|
||||
cancellationToken)));
|
||||
|
||||
var flattenedSteps = stepResults.SelectMany(x => x).ToList();
|
||||
var (toCollapse, directorTansuAfterCompanyDirectors) = SplitDirectorTansuAfterCompanyDirectors(flattenedSteps);
|
||||
var collapsedSteps = CollapseByExecutorEmail(toCollapse)
|
||||
.Concat(directorTansuAfterCompanyDirectors)
|
||||
.ToList();
|
||||
var flattenedSteps = RemoveCompanyDirectorsDuplicatedByDirectorTansu(stepResults.SelectMany(x => x).ToList());
|
||||
var collapsedSteps = CollapseByExecutorEmail(flattenedSteps);
|
||||
|
||||
return Result.Ok(new ResolveTemplateRouteResultDto(route.Id, route.RequiresControl, collapsedSteps));
|
||||
}
|
||||
@@ -233,53 +230,29 @@ public sealed class TemplateRouteService(
|
||||
/// <summary>
|
||||
/// Директор Tansu (HeadType.DirectorTansu) сплошь и рядом совпадает с директором
|
||||
/// ОДНОЙ из компаний инициатора (HeadType.CompanyDirectors) — обычно с ТОО TANSU
|
||||
/// Construction. Оба шага в маршруте законны сами по себе, и оба должны попасть в
|
||||
/// результат — это НЕ дубликат одного согласования, а два разных: сначала блок
|
||||
/// директоров компаний параллельно, а затем — отдельно — Директор Tansu, даже если
|
||||
/// физически это тот же человек. Поэтому строку DirectorTansu с таким совпадением
|
||||
/// НЕ пускаем через обычное схлопывание по email (см. CollapseByExecutorEmail) —
|
||||
/// оно бы решило, что это конфликт PositionCode одного согласующего, и либо
|
||||
/// смержило бы её с строкой CompanyDirectors, либо просто выбрало одну из них.
|
||||
/// Вместо этого явно переставляем её OrderNo на «после всего блока
|
||||
/// CompanyDirectors» — не параллельно ему, а следом.
|
||||
/// Construction. Оба шага в маршруте законны сами по себе, но если так, из
|
||||
/// CompanyDirectors нужно убрать именно ЭТУ строку — не смешивать её с шагом
|
||||
/// DirectorTansu через обычное схлопывание по email (см. CollapseByExecutorEmail):
|
||||
/// тот считает разные PositionCode у одного email конфликтом и пишет
|
||||
/// предупреждение, а здесь это не конфликт, а один и тот же человек — оставляем
|
||||
/// только шаг DirectorTansu, а дубликат из CompanyDirectors удаляем.
|
||||
/// </summary>
|
||||
/// <returns>
|
||||
/// (toCollapse, directorTansuAfterCompanyDirectors): первый список идёт через
|
||||
/// обычное схлопывание по email, второй — строки DirectorTansu, которые нужно
|
||||
/// просто дописать в конец результата без схлопывания.
|
||||
/// </returns>
|
||||
private static (IReadOnlyList<ResolvedRouteStepDto> ToCollapse, IReadOnlyList<ResolvedRouteStepDto> DirectorTansuAfterCompanyDirectors)
|
||||
SplitDirectorTansuAfterCompanyDirectors(IReadOnlyList<ResolvedRouteStepDto> steps)
|
||||
private static IReadOnlyList<ResolvedRouteStepDto> RemoveCompanyDirectorsDuplicatedByDirectorTansu(
|
||||
IReadOnlyList<ResolvedRouteStepDto> steps)
|
||||
{
|
||||
var companyDirectorSteps = steps.Where(s => s.PositionCode == HeadType.CompanyDirectors).ToList();
|
||||
var companyDirectorEmails = companyDirectorSteps
|
||||
.Where(s => !string.IsNullOrWhiteSpace(s.ExecutorEmail))
|
||||
var directorTansuEmails = steps
|
||||
.Where(s => s.PositionCode == HeadType.DirectorTansu && !string.IsNullOrWhiteSpace(s.ExecutorEmail))
|
||||
.Select(s => s.ExecutorEmail!)
|
||||
.ToHashSet(StringComparer.OrdinalIgnoreCase);
|
||||
|
||||
if (companyDirectorEmails.Count == 0)
|
||||
return (steps, []);
|
||||
if (directorTansuEmails.Count == 0)
|
||||
return steps;
|
||||
|
||||
var maxCompanyDirectorsOrderNo = companyDirectorSteps.Max(s => s.OrderNo);
|
||||
|
||||
var toCollapse = new List<ResolvedRouteStepDto>();
|
||||
var afterBlock = new List<ResolvedRouteStepDto>();
|
||||
|
||||
foreach (var step in steps)
|
||||
{
|
||||
if (step.PositionCode == HeadType.DirectorTansu &&
|
||||
!string.IsNullOrWhiteSpace(step.ExecutorEmail) &&
|
||||
companyDirectorEmails.Contains(step.ExecutorEmail))
|
||||
{
|
||||
afterBlock.Add(step with { OrderNo = maxCompanyDirectorsOrderNo + 1 });
|
||||
}
|
||||
else
|
||||
{
|
||||
toCollapse.Add(step);
|
||||
}
|
||||
}
|
||||
|
||||
return (toCollapse, afterBlock);
|
||||
return steps
|
||||
.Where(s => s.PositionCode != HeadType.CompanyDirectors
|
||||
|| string.IsNullOrWhiteSpace(s.ExecutorEmail)
|
||||
|| !directorTansuEmails.Contains(s.ExecutorEmail))
|
||||
.ToList();
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
|
||||
+11
-17
@@ -900,12 +900,12 @@ public sealed class TemplateRouteServiceTests : IClassFixture<PostgresDatabaseFi
|
||||
resolved.Value.Steps[0].Warning.ShouldBeNull();
|
||||
}
|
||||
|
||||
[Fact(DisplayName = "Резолв: если Директор Tansu совпадает с одним из Директоров компаний, он идёт не параллельно блоку, а сразу после него")]
|
||||
public async Task ResolveAsync_DirectorTansuMatchesOneOfCompanyDirectors_SequencedAfterCompanyDirectorsBlock()
|
||||
[Fact(DisplayName = "Резолв: если Директор Tansu совпадает с одним из Директоров компаний, эта строка убирается из CompanyDirectors")]
|
||||
public async Task ResolveAsync_DirectorTansuMatchesOneOfCompanyDirectors_RemovesDuplicateFromCompanyDirectors()
|
||||
{
|
||||
// Arrange — директор ТОО TANSU Construction (из шага CompanyDirectors, OrderNo 5)
|
||||
// и Директор Tansu (из шага DirectorTansu, сам сконфигурирован с OrderNo 10) —
|
||||
// один и тот же человек с одним и тем же email.
|
||||
// и Директор Tansu (из шага DirectorTansu, OrderNo 10) — один и тот же человек
|
||||
// с одним и тем же email.
|
||||
var portalClient = new Mock<ITnsuPortalClient>();
|
||||
portalClient
|
||||
.Setup(x => x.GetEmployeeByEmailAsync(It.IsAny<string>(), It.IsAny<CancellationToken>()))
|
||||
@@ -949,21 +949,15 @@ public sealed class TemplateRouteServiceTests : IClassFixture<PostgresDatabaseFi
|
||||
new ResolveTemplateRouteRequest(requestTypeId, "someone@tnsu.kz", null, RouteStepStatus.Approval),
|
||||
CancellationToken.None);
|
||||
|
||||
// Assert — все три строки остаются (директор не пропадает и не схлопывается
|
||||
// молча), но Директор Tansu не параллелен блоку CompanyDirectors (OrderNo 5),
|
||||
// а идёт сразу после него — OrderNo 6, а не свой сконфигурированный 10.
|
||||
// Assert — вместо трёх строк (2 CompanyDirectors + 1 DirectorTansu) остаётся
|
||||
// две: KazPromService-директор и Директор Tansu, без предупреждения о конфликте.
|
||||
resolved.IsSuccess.ShouldBeTrue();
|
||||
resolved.Value.Steps.Count.ShouldBe(3);
|
||||
var tansuStep = resolved.Value.Steps.Single(s => s.PositionCode == HeadType.DirectorTansu);
|
||||
tansuStep.ExecutorEmail.ShouldBe("director.tansu@tnsu.kz");
|
||||
tansuStep.OrderNo.ShouldBe(6);
|
||||
resolved.Value.Steps.Count.ShouldBe(2);
|
||||
var tansuStep = resolved.Value.Steps.Single(s => s.ExecutorEmail == "director.tansu@tnsu.kz");
|
||||
tansuStep.PositionCode.ShouldBe(HeadType.DirectorTansu);
|
||||
tansuStep.OrderNo.ShouldBe(10);
|
||||
tansuStep.Warning.ShouldBeNull();
|
||||
|
||||
var companyDirectorSteps = resolved.Value.Steps.Where(s => s.PositionCode == HeadType.CompanyDirectors).ToList();
|
||||
companyDirectorSteps.Count.ShouldBe(2);
|
||||
companyDirectorSteps.ShouldAllBe(s => s.OrderNo == 5);
|
||||
companyDirectorSteps.ShouldContain(s => s.ExecutorEmail == "director.tansu@tnsu.kz");
|
||||
companyDirectorSteps.ShouldContain(s => s.ExecutorEmail == "mechanic.kps@tnsu.kz");
|
||||
resolved.Value.Steps.ShouldContain(s => s.ExecutorEmail == "mechanic.kps@tnsu.kz" && s.PositionCode == HeadType.CompanyDirectors);
|
||||
}
|
||||
|
||||
[Fact(DisplayName = "Резолв шага с типом руководителя Chief возвращает warning, если руководитель не найден")]
|
||||
|
||||
Reference in New Issue
Block a user