Skip to content

Редизайн страницы входа на Bootstrap + тесты - #14

Merged
vladcheck merged 5 commits into
mainfrom
vladcheck/ui-login
Oct 6, 2026
Merged

vladcheck merged 5 commits into
mainfrom
vladcheck/ui-login

Conversation

@vladcheck

Copy link
Copy Markdown
Owner

Что сделано

  • Редизайн LoginForm.fxml на BootstrapFX: панель panel panel-primary, кнопка btn btn-primary btn-block, ошибка — alert alert-danger.
  • LoginFormController переписан под паттерн из docs/examples/how-to-interact-with-api-from-ui-via-middleware.md: обычные TextField/PasswordField + errorLabel, клиентская валидация (пустые поля, формат email), вызов AuthService.login(), ошибка сервера — в errorLabel, успех — переход на ResourceList.
  • Program: BootstrapFX-стили подключены глобально, стартовой страницей стала форма входа.
  • В pom.xml добавлен mockito-core 5.23.0 (scope test).

Тесты

  • Unit (ui.LoginFormControllerTest, TestFX + мок AuthService): 5 тестов — пустые поля, пустой пароль, неверный email (API не вызывается, навигации нет), успешный вход (вызов login + переход на ResourceList, ошибка скрыта), ошибка сервера (текст в errorLabel, навигации нет).
  • Integration (ui.LoginIntegrationTest, без моков — реальный AuthServiceImpl + ApiClient): неверные креды против живого API отклоняются с ServiceException. Тест пропускается (assume), если бэкенд на localhost:8080 недоступен.

Проверки

  • mvn verify: 33 теста, 0 failures (1 skipped — нет бэкенда), spotless и checkstyle чистые.

@vladcheck
vladcheck force-pushed the vladcheck/ui-login branch 2 times, most recently from 5e0ff14 to c111375 Compare October 6, 2026 11:24
@vladcheck
vladcheck requested review from Deatheh and miyagi18 October 6, 2026 11:26
@vladcheck vladcheck self-assigned this Oct 6, 2026
@vladcheck
vladcheck merged commit 00dbeb2 into main Oct 6, 2026
8 checks passed
@vladcheck
vladcheck deleted the vladcheck/ui-login branch October 6, 2026 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant