Челаев Петр Лаб. 3 Группа 6512 #89
Open
PripyatskyPrometheus wants to merge 41 commits into
Open
Conversation
…кам чёткое без огрехов, как ыло с автоматической
Gwymlas
requested changes
May 11, 2026
| // Сервис генерации теперь делится на 5 реплик | ||
| var generators = new List<IResourceBuilder<ProjectResource>>(); | ||
| // Minio (объектное хранилище) | ||
| var minio = builder.AddContainer("minio", "minio/minio") |
There was a problem hiding this comment.
Лучше использовать готовый aspire пакет (CommunityToolkit.Aspire.Hosting.Minio), в просто поднятом контейнере не будет телеметрии и сервис дискавери
|
|
||
| // Эмулятор Amazon SQS, добавлен для корректной отработки тестов | ||
| // (до этого были постоянные ошибки тестов из-за рандомизации портов) | ||
| var sqs = builder.AddContainer("elasticmq", "softwaremill/elasticmq") |
| .WithHttpEndpoint(port: 9324, targetPort: 9324, name: "http"); | ||
|
|
||
| // Создаём 5 генераторов в цикле | ||
| var generators = new List<IResourceBuilder<ProjectResource>>(); |
There was a problem hiding this comment.
Генераторам стоит добавить ожидание брокера
Comment on lines
+21
to
+33
| public SqsBackgroundService( | ||
| IAmazonSQS sqsClient, | ||
| IMinioClient minioClient, | ||
| IConfiguration configuration, | ||
| ILogger<SqsBackgroundService> logger) | ||
| { | ||
| _sqsClient = sqsClient; | ||
| _minioClient = minioClient; | ||
| _logger = logger; | ||
|
|
||
| _queueUrl = configuration["SQS:QueueUrl"] ?? "http://localhost:9324/queue/projects"; | ||
| _bucketName = configuration["Minio:BucketName"] ?? "projects"; | ||
| } |
There was a problem hiding this comment.
SqsBackgroundService сейчас выполняет инициализацию инфраструктуры, чтение из очереди и сохраняет данные в объектное хранилище. Думаю стоит разбить на несколько служб, сейчас нарушается srp
| { | ||
| public DistributedApplication App { get; private set; } = null!; | ||
| public IAmazonS3 S3Client { get; private set; } = null!; | ||
| public string SqsUrl { get; private set; } = null!; |
Comment on lines
+56
to
+65
| try | ||
| { | ||
| await S3Client.GetBucketLocationAsync("projects"); | ||
| Console.WriteLine("Бакет projects существует"); | ||
| } | ||
| catch (AmazonS3Exception ex) when (ex.ErrorCode == "NoSuchBucket") | ||
| { | ||
| await S3Client.PutBucketAsync("projects"); | ||
| Console.WriteLine("Бакет projects создан"); | ||
| } |
There was a problem hiding this comment.
Разве файловый сервис сам не создаст бакет?
| /// Проверяет, что все поля модели программного проекта заполнены корректно | ||
| /// </summary> | ||
| [Fact] | ||
| public async Task AllFieldsPopulated() |
There was a problem hiding this comment.
Мне кажется этот и Generator_GetById_ReturnsValidProject можно объединить, один чуть шире проверяет поля чем другой
| /// Сквозной тест, проверяющий полный путь данных через GenerationService,SQS, FileService и Mini | ||
| /// </summary> | ||
| [Fact] | ||
| public async Task Minio_FileSaved() |
There was a problem hiding this comment.
Может тут и в Minio_FileContentMatchesApiResponse обращаться не к сервису генерации, а к gateway, чтобы пройти весь путь
Comment on lines
+149
to
+151
| Assert.Equal(apiProject.Name, savedProject.Name); | ||
| Assert.Equal(apiProject.Customer, savedProject.Customer); | ||
| Assert.Equal(apiProject.Budget, savedProject.Budget); |
There was a problem hiding this comment.
Возможно лучше использовать
Assert.Equivalent(apiProject, savedProject);
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ФИО: Челаев Петр
Номер группы: 6512
Номер лабораторной: 3
Номер варианта: 35
Краткое описание предметной области: Программный проект
Краткое описание добавленных фич: Реализован файловый сервис, добавлено объектное хранилище и интеграционные тесты