ДЗ №7: Web API #8
No reviewers
Labels
No labels
No milestone
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
fbki.Levitan/PersonalAccount!8
Loading…
Reference in a new issue
No description provided.
Delete branch "task/7_web-api"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Да. В целом все верно. Нужно немного доработать код.
@ -0,0 +37,4 @@{table.PrimaryKey("PK_journal_rows", x => x.id);table.ForeignKey(name: "FK_journal_rows_organizations_organization_id",Нет. Не нужно связей делать. Это "плоская" таблица. Назначение - "очень быстро вставить данные".
@ -24,3 +61,2 @@foreach (var entry in entries)Console.WriteLine(entry);await syncService.SyncBatchAsync(fetchSettings);Нет. Нужно весь журнал загрузить из клиента на сервер. У Вас только одна пачка загружается.
Так же , сделайте замер (по логам, например) с какой скоростью у Вас весь журнал был загружен.
@ -0,0 +5,4 @@using PersonalAccount.Api.Services;using PersonalAccount.Domain.Data;using PersonalAccount.Domain.Models;using Testcontainers.PostgreSql;Советую заменить на InMemory
https://learn.microsoft.com/en-us/ef/core/providers/in-memory/?tabs=dotnet-core-cli
Если разворачивать Ваш проект в Docker контейнере, то потребуется root права (контейнер в контейнере). т.е. таким образом Ваш проект будет зависимый. Лучше это исключить. Есть альтернативы.
@ -0,0 +60,4 @@// ── PushAsync ─────────────────────────────────────────────────────────────────[Test]public async Task PushAsync_WithNewRows_PersistsRowsAndReturnsTrue()Добавьте пжл описание к тестам
https://gitverse.ru/VolovikovAlexander/PersonalAccount2026/content/master/_docs/CONTRIBUTING.md
Для чего каждый тест используется.
Тесты пишутся для "коллег" на будущее.
@ -0,0 +129,4 @@// ── Helpers ───────────────────────────────────────────────────────────────────private static List<JournalRowDto> BuildRows(long startCode, int count) =>У Вас тут только продаже. Нет оплат, сдачи и прочее.
Советую заменить
https://gitverse.ru/VolovikovAlexander/PersonalAccount2026/content/master/Tst/PersonalAccount.UnitTests/ReportDataCreator.cs
На прошлом занятии мы разбирали данный кейс.