visionA/local-agent/visiona-agent/shutdown_notify_test.go
jim800121chen 3f0175f1a9 feat(local-agent): Phase 0.5 visionA Agent — Wails 桌面 + tunnel client + 配對 UI
從 local-tool 複製出獨立的「visionA Agent」桌面應用(A3 純橋樑:
tunnel client + 配對 UI + 設定,不開 HTTP port、不做本機裝置/推論 UI)。
Bundle ID 與 local-tool 不同(com.innovedus.visiona-agent vs visiona-local),
雙 app 可共存。fork 後不主動 sync,需要時手動 cherry-pick。

Backend / Wails Go(AB1-AB13):
- internal/tunnel:6 狀態機(Idle/Connecting/Connected/Reconnecting/Failed/Stopped)
  + Pair/Unpair/Reconnect/Disconnect binding + ClientHooks event
- internal/auth:encrypted file token store(AES-GCM + scrypt + machineID
  fallback salt + 13 tests)
- internal/config:YAML validation + atomic write + 11 tests
- internal/log:ring buffer + ExportLog 升級 zip
- visionA-backend /api/pairing/exchange:SessionTokenStore + 17 new tests
- 三平台 build 驗證(macOS DMG 160 MB / Windows EXE / Linux AppImage)
- end-to-end 5 milestone 全綠(pairing → tunnel → forward → reuse 防護
  → tunnel drop failover)

Frontend / Next.js(AF1-AF7,沿用 visionA-frontend 基礎):
- AppShell + Header + TabNav(StatusView / PairView / SettingsView 三 tab)
- ConnectionStatusBadge 5 種狀態
- TokenInput regex 驗證 + 7 種錯誤 + 0.5s auto-switch 到狀態頁
- 設定頁 4 區塊(含重新配對 AlertDialog)
- agent-api.ts 封裝 Wails bindings(mock/real 雙實作)+ 90 tests

Phase 0.7 review-driven fix(Round 2):
- A1 Session fixation 防護(RotateSessionID)
- A3 mock pairing 預設改 false(必須明確 opt-in)+ startup log
- A4 Pair 失敗後 state 清理矩陣(exchange/Save/Start fail 各自終態)
- A5 Pair/Unpair/Reconnect lifecycleMu + 50 goroutine race test
- F1 重新配對次按鈕 / F2 PairView Esc cancel / F3 Wails BrowserOpenURL
  / F4 Settings draft 持久 + 未儲存 badge

驗證:agent backend go test -race -count=3 ./... 4 packages 全綠 /
agent frontend pnpm test 119 tests 全綠

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-01 11:22:01 +08:00

177 lines
4.7 KiB
Go
Raw Permalink Blame History

This file contains ambiguous Unicode characters

This file contains Unicode characters that might be confused with other characters. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.

package main
// shutdown_notify_test.go — MAJ-4 補丁notifyShutdownImminent helper 單元測試
//
// 驗證:
// 1. port <= 0 → 直接 return不發 HTTP不 panic
// 2. reason=quit → 真的打到 /api/system/shutdown-notify?reason=quit
// 3. server 無回應sink hole→ timeout 後不阻塞 caller
// 4. server 回 5xx / 非 200 → 依然不阻塞 callerbest-effort
import (
"context"
"net"
"net/http"
"net/http/httptest"
"net/url"
"strconv"
"strings"
"sync"
"testing"
"time"
)
func TestNotifyShutdownImminent_ZeroPortIsNoop(t *testing.T) {
// port = 0 / 負數 → 直接 return沒 panic 就代表正確
notifyShutdownImminent(context.Background(), 0, "quit")
notifyShutdownImminent(context.Background(), -1, "quit")
}
func TestNotifyShutdownImminent_SendsPostWithReason(t *testing.T) {
var (
mu sync.Mutex
seenPath string
seenQuery url.Values
seenMethod string
requestDone = make(chan struct{}, 1)
)
srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
mu.Lock()
seenPath = r.URL.Path
seenQuery = r.URL.Query()
seenMethod = r.Method
mu.Unlock()
w.WriteHeader(http.StatusOK)
_, _ = w.Write([]byte(`{"ok":true,"reason":"quit"}`))
select {
case requestDone <- struct{}{}:
default:
}
}))
defer srv.Close()
port := portFromURL(t, srv.URL)
notifyShutdownImminent(context.Background(), port, "quit")
select {
case <-requestDone:
case <-time.After(2 * time.Second):
t.Fatalf("server never received request")
}
mu.Lock()
defer mu.Unlock()
if seenMethod != http.MethodPost {
t.Errorf("want POST, got %q", seenMethod)
}
if seenPath != "/api/system/shutdown-notify" {
t.Errorf("wrong path: %q", seenPath)
}
if seenQuery.Get("reason") != "quit" {
t.Errorf("wrong reason: %q", seenQuery.Get("reason"))
}
}
func TestNotifyShutdownImminent_SendsReasonRestart(t *testing.T) {
var seenReason string
var mu sync.Mutex
srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
mu.Lock()
seenReason = r.URL.Query().Get("reason")
mu.Unlock()
w.WriteHeader(http.StatusOK)
}))
defer srv.Close()
notifyShutdownImminent(context.Background(), portFromURL(t, srv.URL), "restart")
mu.Lock()
defer mu.Unlock()
if seenReason != "restart" {
t.Errorf("want reason=restart, got %q", seenReason)
}
}
func TestNotifyShutdownImminent_TimeoutDoesNotBlock(t *testing.T) {
// 短 timeout確保 caller 不會被卡住
orig := notifyShutdownImminentTimeout
notifyShutdownImminentTimeout = 150 * time.Millisecond
defer func() { notifyShutdownImminentTimeout = orig }()
// Server 回應得「比 client timeout 慢一倍」,因此 client 應該先 timeout 回來
srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
time.Sleep(500 * time.Millisecond)
w.WriteHeader(http.StatusOK)
}))
defer srv.Close()
start := time.Now()
done := make(chan struct{})
go func() {
notifyShutdownImminent(context.Background(), portFromURL(t, srv.URL), "quit")
close(done)
}()
select {
case <-done:
if elapsed := time.Since(start); elapsed > 400*time.Millisecond {
t.Errorf("client timeout 沒生效 — elapsed=%v應 < 400ms", elapsed)
}
case <-time.After(1 * time.Second):
t.Fatalf("notifyShutdownImminent 被卡住超過 1s")
}
}
func TestNotifyShutdownImminent_ServerErrorIsBestEffort(t *testing.T) {
// server 回 500 → caller 不該 panic也不該有任何可見副作用
srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
http.Error(w, "boom", http.StatusInternalServerError)
}))
defer srv.Close()
notifyShutdownImminent(context.Background(), portFromURL(t, srv.URL), "quit")
}
func TestNotifyShutdownImminent_ConnectionRefusedIsIgnored(t *testing.T) {
// 啟一個 listener 後立刻關閉 → port 上沒服務 → caller 應靜默失敗
l, err := net.Listen("tcp", "127.0.0.1:0")
if err != nil {
t.Fatalf("listen: %v", err)
}
port := l.Addr().(*net.TCPAddr).Port
_ = l.Close()
done := make(chan struct{})
go func() {
notifyShutdownImminent(context.Background(), port, "quit")
close(done)
}()
select {
case <-done:
case <-time.After(2 * time.Second):
t.Fatalf("connection refused path should return quickly")
}
}
// portFromURL 從 httptest.Server.URL例 http://127.0.0.1:54321抽出 port。
func portFromURL(t *testing.T, rawURL string) int {
t.Helper()
u, err := url.Parse(rawURL)
if err != nil {
t.Fatalf("parse url: %v", err)
}
portStr := u.Port()
if portStr == "" {
t.Fatalf("no port in URL %q", rawURL)
}
if strings.ContainsAny(portStr, "/") {
t.Fatalf("bad port %q", portStr)
}
p, err := strconv.Atoi(portStr)
if err != nil {
t.Fatalf("bad port: %v", err)
}
return p
}