Skip to content

fix(mobile): clean up stale tests, fix pubspec, remove dead docs

Jérôme Revillard requested to merge fix/mobile-stale-tests-pubspec-cleanup into main

Summary

  • Delete api_service_test.dart — ApiService was refactored into OpenAPI client providers, test referenced a non-existent class (2 compilation failures)
  • Migrate auth_notifier_test.dart from ApiService/apiServiceProvider to FakeAuthenticationApi/authenticationApiProvider, fix 2 race conditions in logout tests
  • Fix keycloak_config_test.dart expected values to match actual configs (devConfig dart-define defaults vs e2eConfig hardcoded emulator host)
  • Move flutter_launcher_icons from dependencies to dev_dependencies in pubspec.yaml
  • Remove obsolete ChatbotProxy pitfall from mobile CLAUDE.md

Test plan

  • flutter test — 198/198 passed (was 120/125 before fix)
  • flutter analyze lib/ test/services/ test/config/ — No issues found
  • flutter build apk --flavor dev --debug — Build successful

Merge request reports