test: [#100] add tests for all TODO in repository/WorkspaceRepository #101
Conversation
| } | ||
|
|
||
| @ParameterizedTest | ||
| @ValueSource(strings = "wks-test") |
There was a problem hiding this comment.
Давай тут тоже будем использовать методы EntitiesFactory чтобы подставить не одно значение, а стрим из существующих воркспейсов
alexeiak
force-pushed
the
tests_branch
branch
2 times, most recently
from
dab66ef
to
b5221ce
Compare
All TODOs in package /service have done.
Add test for createWorkspace() on exception situation
Add text about project
… excluding conurency test for putWorkspaceUpdate()
|
После git pull пришлось сделать force push, но всё по инструкции, лишь метаданные предыдущих коммитов обновились. До конца недели планирую всё доделать: некоторые правки и тест для конкурентных транзакций в web/WorkspaceController putWorkspaceUpdate(). |
|
Я правильно понимаю, что работа над ПР пока не закончена и будут еще коммиты? |
|
Ребят, чтобы ПР не было СУПЕРМЕГА большим, давайте вносить поэтапно. |
service/WorkspaceServiceIT, repository/WorkspaceRepositoryIT
Только что отправил коммит по рефакторингу. Осталось написать доп. тест на ситуацию с конкурентными транзакциями, но я ещё не приступил к вниканию, как это по-правильному сделать. В коде обозначена TODO, поэтому, даже на этот момент всё уже красиво. |
Отправляю так: один коммит — один класс или пакет.
По завершению напишу сюда в комментариях.