Skip to content

Солдатова Ксения Лаб. 3 Группа 6512#161

Open
KsSoldatova wants to merge 10 commits into
itsecd:mainfrom
KsSoldatova:main
Open

Солдатова Ксения Лаб. 3 Группа 6512#161
KsSoldatova wants to merge 10 commits into
itsecd:mainfrom
KsSoldatova:main

Conversation

@KsSoldatova
Copy link
Copy Markdown

ФИО: Солдатова Ксения
Номер группы: 6512
Номер лабораторной: 3
Номер варианта: 33
Краткое описание предметной области: Программный проект
Краткое описание добавленных фич: Добавлены объектное хранилище, файловый сервис и тесты

cloud-1 cloud-2

@github-actions github-actions Bot added In progress Код в процессе проверки Lab 3 Лабораторная №3. Интеграционное тестирование labels May 19, 2026
@github-actions github-actions Bot requested a review from Gwymlas May 19, 2026 21:07
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Удалить

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не используется, удалить

Comment on lines +90 to +95
/// <summary>
/// Проверяет, что повторный запрос того же программного проекта
/// обслуживается из кэша и не приводит к дублирующей отправке в S3
/// </summary>
[Fact]
public async Task RepeatedRequest_DoesNotDuplicateFileInS3()
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вроде этим тестом мы можем утверждать только то, что в объектном хранилище в итоге находится один объект с ключом programproject_{id}.json. Но тест не доказывает, что при повторных запросах не происходит отправка в брокер сообщений и запись в объектное хранилище с перезаписью существующего объекта
Можно просто уточнить комментарий к тесту, чтобы он не вводил в заблуждение

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

In progress Код в процессе проверки Lab 3 Лабораторная №3. Интеграционное тестирование

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants