sadmadrus / chessBox

chessBox
Apache License 2.0
0 stars 0 forks source link

added simple validation (lines 1-235) #27

Closed loskutovanl closed 1 year ago

loskutovanl commented 1 year ago

[WIP] simple validation добавлена простая валидация движения фигуры Simple - # 1 из VALIDATION.md

Принимаю пожелания по улучшею читаемости и логики и ошибкам.

sadmadrus commented 1 year ago

Пока нет описания сервиса, можно подготовить функцию запроса к сервису и подождать немного

nekr0z commented 1 year ago

Линтер по делу ругается: все его рекомендации стоит воплотить — это повысит читаемость кода ;)

nekr0z commented 1 year ago

Упс, в тесте старое имя функции. Я как раз полез править, а ты уже смёржила :)

loskutovanl commented 1 year ago

Немного реорганизовала структуру для удобства и переименовала файлы. Добавила тесты для всех фигур. В процессе нашла баг для движения пешек и исправила его.

loskutovanl commented 1 year ago

Небольшие правки в Simple и спецификации после юнит тестов.

Блок Advanced и ниже смотреть ещё рано. Не готово. Частично начала делать из-за спонтанно возникшего свободного времени.

loskutovanl commented 1 year ago

В блоке Simple готов сервис и методы движения фигур для его обработки. Добавлены тесты на все функции и юнит-тесты. Не вижу открытых критичных вопросов по simple, поэтому предлагаю смержить ветку.

p.s. в активном процессе работа над advanced, но поскольку это уже другое направление, то для ее ревью я лучше создам отдельный PR. Пока что ничего, связанного с advanced, смотреть и проверять не нужно, оно еще не готово.

loskutovanl commented 1 year ago

Оставила только куски по Simple в ветке. Готовы к слиянию!