Conversation
…я данных соответствовал серверу
github-actions
bot
added
In progress
| using System.Text; | ||
| using System.Text.Json.Nodes; | ||
|
|
||
| namespace ResidentialBuilding.EventSink.Controller; |
There was a problem hiding this comment.
Неймспейс не соответствует директории, в которой лежит исходник
| using ResidentialBuilding.EventSink.Service.Storage; | ||
| using System.Text; | ||
|
|
||
| namespace ResidentialBuilding.EventSink.Controller; |
| } | ||
|
|
||
| logger.LogInformation("Subscription was successfully confirmed."); | ||
| return Ok(); |
There was a problem hiding this comment.
Вот этот return Ok(); я бы, наверное, заменил на break;
| logger.LogError(ex, "Exception occurred while processing SNS notifications."); | ||
| } | ||
|
|
||
| return Ok(); |
There was a problem hiding this comment.
Потому что тут есть return Ok();
| ?? throw new KeyNotFoundException( | ||
| "S3 bucket name was not found in configuration"); |
There was a problem hiding this comment.
Это ReSharper в Rider-е предлагает так переносить строки? JetBrains с каждым релизом все дальше от бога 😢
There was a problem hiding this comment.
да, это ide так переносит, с форматированием у них в этой ide почему-то беда
|
|
||
| namespace ResidentialBuilding.EventSink.Service.Storage; | ||
|
|
||
| public class AwsFileService(IAmazonS3 client, IConfiguration configuration, ILogger<AwsFileService> logger) |
| @@ -0,0 +1,9 @@ | |||
| namespace ResidentialBuilding.EventSink.Service.Messaging; | |||
There was a problem hiding this comment.
Похоже, что проблема с неймспейсами у тебя по всему проекту, потому что проект раньше назывался по-другому
There was a problem hiding this comment.
Если честно, я не вижу разницы между первыми тремя юнит-тестами с точки зрения проверки работоспособности пайплайна обработки.
Все они выстроены одинаково - поднимается оркестратор, отправляется запрос в гейтвей, через период времени запрашивается файловый сервис, его ответ ассертится.
При этом некоторый краевой случай -проверка, что запрос по одному идентификатору вернет одни и те же данные- рассматривается только в тесте номер 3.
Я бы порекомендовал лучше покрыть тестами негативный сценарий - ведь у тебя в коде происходит валидация id
…н return Ok() в конце метода; в AwsFileService добавил summary и сделал человеческий перенос строк
2 participants
ФИО: Елагин Денис
Номер группы: 6513
Номер лабораторной: 3
Номер варианта: 38
Краткое описание предметной области: Объект жилого строительства
Краткое описание добавленных фич: добавил s3 хранилище и брокер по варианту, изменил генератор так, чтобы там е, где происходит кэширование происходила отправка в брокер, сервис работы с файлами получает сообщения из брокера и сохраняет в s3 хранилище, также сделаны интеграционные тесты