Skip to content

test(client): cover domain, state and data logic - #218

Open
MITOM06 wants to merge 1 commit into
mainfrom
test/cov-flutter-logic
Open

MITOM06 wants to merge 1 commit into
mainfrom
test/cov-flutter-logic

Conversation

@MITOM06

@MITOM06 MITOM06 commented Nov 14, 2026

Copy link
Copy Markdown
Owner

Summary

Tests only — no production code changed, coverage-thresholds.txt untouched.

Flutter line coverage: 56.83% → 62.00% (16785/29534 → 18313/29538 lines; *.g.dart, *.freezed.dart, l10n/ excluded — scripts/ci/coverage-gate.sh flutter). The gate suggests flutter=61; left for the coordinator.

43 test files (33 new, 10 extended), ~290 new test cases. Full suite: 2159 pass, 70 skipped (pre-existing), 0 fail. flutter analyze: no issues.

Covered

  • core: every app_routes.dart pageBuilder (params/query/extra parsing), Dio interceptors (bearer, Accept-Language, 403 ACCOUNT_BLOCKED logout, 401 without refresh creds, network error), Otp6BoxInput, BouncingDots
  • chat/data + domain: KB / persona / memory / AI-action repositories and notifiers (incl. live KB status), upload ops (MIME per extension, token parsing), calls repo, StompStreams routing, ActiveCalls, 1-on-1 WebRTC + LiveKit signal routing (ring / busy / crossed / end / answered-elsewhere / cancel), AI stream watchdog (fake time), SpeechCapture / CallSttService, reducers (quote edits, pending-action cards), system-message parser, chat/connector/auth/admin error tables, tool display names
  • chat/presentation: CallScreen (mesh caller/callee, connect + timer, controls, ring timeout, back = hang up, mic refused, LiveKit caller + incoming without mic) via FakeWebRtcPlatform; GroupCallScreen notetaker banner / named tiles / cam toggle
  • meetings: NoteEditorNotifier (autosave, 409 conflict + merge, retry, remote refetch, best-effort save on dispose), list refresh / detail / chat history paging / cancel-end-update, queue listener summary-ready, attach picker, repository update/join/deny, value objects
  • auth / admin / integrations / reminders / skills / help: profile update (no AsyncLoading flicker), MFA guards, bot admin repo, audit labels, directory/custom MCP, reminders (+ screen), skills optimistic toggle, FAQ data

Each new area was sanity-checked by breaking the production code temporarily (tests went red) and reverting.

Bugs found

None marked skip: 'BUG'.

Still uncovered in logic layers (why)

  • core/rtc/livekit_session*.dart (~180 lines): needs a real LiveKit Room
  • core/services/notification_service.dart: Firebase / local notifications plugins
  • core/config/dev_host_discovery.resolveAndApply, parts of app_config.dart: gated by compile-time --dart-defines
  • token_usage_provider.dart: builds a private Dio (no override seam)
  • AudioplayersTonePlayer, meeting_screen_frames capture, wakelock: platform plugins
  • Dio 401 → refresh → retry path: goes through TokenManager.shared (no injection seam)

🤖 Generated with Claude Code

Raise Flutter line coverage 56.83% -> 62.00% (16785 -> 18313 / 29534+ lines,
generated files and l10n excluded) with tests only — no production change.

- core: app_routes pageBuilders, Dio interceptors (bearer, Accept-Language,
  ACCOUNT_BLOCKED logout, 401 without refresh creds), OTP input, typing dots
- chat: KB / persona / memory / AI-action repos + notifiers, upload ops,
  calls repo, STOMP stream routing, active-call banner, realtime WebRTC and
  LiveKit signal routing, AI stream watchdog, STT capture, reducers,
  system-message parser, error/tool-name mapping, CallScreen + GroupCallScreen
- meetings: note editor autosave/conflict, providers (refresh, history paging,
  cancel/end/update), queue listener, attach picker, repo, value objects
- auth/admin/integrations/reminders/skills/help: error code tables, profile
  update, MFA guards, bot admin repo, audit labels, reminders, skills, FAQ

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Nov 14, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
platform-web Ready Ready Preview Nov 14, 2026 11:52am UTC

@coderabbitai

coderabbitai Bot commented Nov 14, 2026

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: ae4a6851-6301-4cf9-bbba-614c2561f7c0
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

This branch was successfully deployed

1 active deployment
Preview — d92578da Deployed Nov 14, 2026 by vercel[bot]
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