Skip to content

fix(ci): start unit tests with an empty host app - #63

Merged
502Proxy merged 3 commits into
mainfrom
fix/unit-test-host
Oct 28, 2026
Merged

502Proxy merged 3 commits into
mainfrom
fix/unit-test-host

Conversation

@502Proxy

Copy link
Copy Markdown
Owner

Closes #62

What changes and why?

Seit dem Menüleisten-Symbol hängt „Build & Tests (macOS)“ („The test runner hung before establishing connection“). Probe-Läufe zeigen: Ohne MenuBarExtra-Szene ist die CI grün, mit leerer Test-Host-App ebenfalls grün (Details in #62).

  • Neuer Einstieg App/AppLauncher.swift (@main): Unter XCTest startet er eine leere App, sonst StudyAppApp.
  • Startarbeit (Datenbank, Sync, Netzwerk, Menüleiste) läuft beim Testen nicht mehr; die Tests bauen ihre Objekte ohnehin selbst (AppContainer(inMemory:)).
  • docs/ARCHITECTURE.md beschreibt den Einstieg.

Checklist

  • Linked issue, labels and milestone set
  • scripts/qa.sh passes locally (SwiftLint + tests): CI dieses PRs; die gleiche Änderung lief auf probe/test-host grün
  • New logic has tests in StudyAppTests: nicht nötig, reiner Einstieg
  • UI tried in the app (light and dark): bitte prüfen, dass die App mit Menüleisten-Symbol normal startet
  • Data model changes: keine
  • User-visible change: keine
  • Docs in docs/ updated where behaviour changed

Generated by Claude Code

The menu bar scene kept the test host from finishing its launch on CI,
so the test runner never connected. A separate entry point now starts an
empty app under XCTest; the tests build what they need themselves.
Refs #62

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TmmqmKmX8VDSfoPbzTS1ew
Closes #62

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TmmqmKmX8VDSfoPbzTS1ew
@502Proxy 502Proxy added this to the 1.12 milestone Oct 28, 2026 — with Claude
@502Proxy 502Proxy added the bug Something does not work as intended label Oct 28, 2026 — with Claude
@502Proxy
502Proxy merged commit 5297320 into main Oct 28, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something does not work as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(ci): Unit-Tests mit leerer Host-App starten

2 participants