Files
Memoh/internal/channel/manager_integration_test.go
T
Ran 6acdd191c7 Squashed commit of the following:
commit bcdb026ae43e4f95d0b2c4f9bd440a2df9d6b514
Author: Ran <16112591+chen-ran@users.noreply.github.com>
Date:   Thu Feb 12 17:10:32 2026 +0800

    chore: update DEVELOPMENT.md

commit 30281742ef
Merge: ca5c6a1 5b05f13
Author: BBQ <bbq@BBQdeMacBook-Air.local>
Date:   Thu Feb 12 15:49:17 2026 +0800

    merge(github/main): integrate fx dependency injection framework

    Merge upstream fx refactor and adapt all services to use go.uber.org/fx
    for dependency injection. Resolve conflicts in main.go, server.go,
    and service constructors while preserving our domain model changes.

    - Fix telegram adapter panic on shutdown (double close channel)
    - Fix feishu adapter processing messages after stop
    - Increase directory lookup timeout from 2s to 5s

commit ca5c6a1866
Author: BBQ <bbq@BBQdeMacBook-Air.local>
Date:   Thu Feb 12 15:33:09 2026 +0800

    refactor(core): restructure conversation, channel and message domains

    - Rename chat module to conversation with flow-based architecture
    - Move channelidentities into channel/identities subpackage
    - Add channel/route for routing logic
    - Add message service with event hub
    - Add MCP providers: container, directory, schedule
    - Refactor Feishu/Telegram adapters with directory and stream support
    - Add platform management page and channel badges in web UI
    - Update database schema for conversations, messages and channel routes
    - Add @memoh/shared package for cross-package type definitions

commit 75e2ef0467
Merge: d99ba38 01cb6c8
Author: BBQ <bbq@BBQdeMacBook-Air.local>
Date:   Thu Feb 12 14:45:49 2026 +0800

    merge(github): merge github/main, resolve index.ts URL conflict

    Keep our defensive absolute-URL check in createAuthFetcher.

commit d99ba38b7d
Merge: 860e20f 35ce7d1
Author: BBQ <bbq@BBQdeMacBook-Air.local>
Date:   Thu Feb 12 05:20:18 2026 +0800

    merge(github): merge github/main, keep our code and docs/spec

commit 860e20fe70
Author: BBQ <bbq@BBQdeMacBook-Air.local>
Date:   Wed Feb 11 22:13:27 2026 +0800

    docs(docs): add concepts and style guides for VitePress site

    - Add concepts: identity-and-binding, index (en/zh)
    - Add style: terminology (en/zh)
    - Update index and zh/index
    - Update .vitepress/config.ts

commit a75fdb8040
Author: BBQ <bbq@BBQdeMacBook-Air.local>
Date:   Wed Feb 11 17:37:16 2026 +0800

    refactor(mcp): standardize unified tool gateway on go-sdk

    Split business executors from federation sources and migrate unified tool/federation transports to the official go-sdk for stricter MCP compliance and safer session lifecycle handling. Add targeted regression tests for accept compatibility, initialization retries, pending cleanup, and include updated swagger artifacts.

commit 02b33c8e85
Author: BBQ <bbq@BBQdeMacBook-Air.local>
Date:   Wed Feb 11 15:42:21 2026 +0800

    refactor(core): finalize user-centric identity and policy cleanup

    Unify auth and chat identity semantics around user_id, enforce personal-bot owner-only authorization, and remove legacy compatibility branches in integration tests.

commit 06e8619a37
Author: BBQ <bbq@BBQdeMacBook-Air.local>
Date:   Wed Feb 11 14:47:03 2026 +0800

    refactor(core): migrate channel identity and binding across app

    Align channel identity and bind flow across backend and app-facing layers, including generated swagger artifacts and package lock updates while excluding docs content changes.
2026-02-12 17:13:03 +08:00

242 lines
6.7 KiB
Go

package channel
import (
"context"
"fmt"
"io"
"log/slog"
"strings"
"sync"
"testing"
"time"
)
type fakeConfigStore struct {
effectiveConfig ChannelConfig
channelIdentityConfig ChannelIdentityBinding
configsByType map[ChannelType][]ChannelConfig
boundChannelIdentityID string
}
func (f *fakeConfigStore) ResolveEffectiveConfig(ctx context.Context, botID string, channelType ChannelType) (ChannelConfig, error) {
return f.effectiveConfig, nil
}
func (f *fakeConfigStore) GetChannelIdentityConfig(ctx context.Context, channelIdentityID string, channelType ChannelType) (ChannelIdentityBinding, error) {
if f.channelIdentityConfig.ID == "" && len(f.channelIdentityConfig.Config) == 0 {
return ChannelIdentityBinding{}, fmt.Errorf("channel user config not found")
}
return f.channelIdentityConfig, nil
}
func (f *fakeConfigStore) UpsertChannelIdentityConfig(ctx context.Context, channelIdentityID string, channelType ChannelType, req UpsertChannelIdentityConfigRequest) (ChannelIdentityBinding, error) {
return f.channelIdentityConfig, nil
}
func (f *fakeConfigStore) ListConfigsByType(ctx context.Context, channelType ChannelType) ([]ChannelConfig, error) {
if f.configsByType == nil {
return nil, nil
}
return f.configsByType[channelType], nil
}
func (f *fakeConfigStore) ResolveChannelIdentityBinding(ctx context.Context, channelType ChannelType, criteria BindingCriteria) (string, error) {
if f.boundChannelIdentityID == "" {
return "", fmt.Errorf("channel user binding not found")
}
return f.boundChannelIdentityID, nil
}
type fakeInboundProcessorIntegration struct {
resp *OutboundMessage
err error
gotCfg ChannelConfig
gotMsg InboundMessage
}
func (f *fakeInboundProcessorIntegration) HandleInbound(ctx context.Context, cfg ChannelConfig, msg InboundMessage, sender StreamReplySender) error {
f.gotCfg = cfg
f.gotMsg = msg
if f.err != nil {
return f.err
}
if f.resp == nil {
return nil
}
if sender == nil {
return fmt.Errorf("sender missing")
}
return sender.Send(ctx, *f.resp)
}
type fakeAdapter struct {
channelType ChannelType
mu sync.Mutex
started []ChannelConfig
sent []OutboundMessage
stops int
}
func (f *fakeAdapter) Type() ChannelType {
return f.channelType
}
func (f *fakeAdapter) Descriptor() Descriptor {
return Descriptor{Type: f.channelType, DisplayName: "Fake", Capabilities: ChannelCapabilities{Text: true}}
}
func (f *fakeAdapter) ResolveTarget(channelIdentityConfig map[string]any) (string, error) {
value := strings.TrimSpace(ReadString(channelIdentityConfig, "target"))
if value == "" {
return "", fmt.Errorf("missing target")
}
return "resolved:" + value, nil
}
func (f *fakeAdapter) NormalizeTarget(raw string) string { return strings.TrimSpace(raw) }
func (f *fakeAdapter) Connect(ctx context.Context, cfg ChannelConfig, handler InboundHandler) (Connection, error) {
f.mu.Lock()
f.started = append(f.started, cfg)
f.mu.Unlock()
stop := func(context.Context) error {
f.mu.Lock()
f.stops++
f.mu.Unlock()
return nil
}
return NewConnection(cfg, stop), nil
}
func (f *fakeAdapter) Send(ctx context.Context, cfg ChannelConfig, msg OutboundMessage) error {
f.mu.Lock()
f.sent = append(f.sent, msg)
f.mu.Unlock()
return nil
}
func TestManagerHandleInboundIntegratesAdapter(t *testing.T) {
t.Parallel()
log := slog.New(slog.NewTextHandler(io.Discard, &slog.HandlerOptions{}))
store := &fakeConfigStore{}
processor := &fakeInboundProcessorIntegration{
resp: &OutboundMessage{
Target: "123",
Message: Message{
Text: "ok",
},
},
}
reg := NewRegistry()
adapter := &fakeAdapter{channelType: ChannelType("test")}
manager := NewManager(log, reg, store, processor)
manager.RegisterAdapter(adapter)
cfg := ChannelConfig{
ID: "cfg-1",
BotID: "bot-1",
ChannelType: ChannelType("test"),
Credentials: map[string]any{"botToken": "token"},
UpdatedAt: time.Now(),
}
err := manager.handleInbound(context.Background(), cfg, InboundMessage{
Channel: ChannelType("test"),
Message: Message{Text: "hi"},
BotID: "bot-1",
ReplyTarget: "123",
Conversation: Conversation{
ID: "chat-1",
Type: "p2p",
},
})
if err != nil {
t.Fatalf("expected no error, got %v", err)
}
if processor.gotMsg.Conversation.ID != "chat-1" || processor.gotMsg.Message.PlainText() != "hi" || processor.gotMsg.BotID != "bot-1" {
t.Fatalf("unexpected inbound message: %+v", processor.gotMsg)
}
adapter.mu.Lock()
defer adapter.mu.Unlock()
if len(adapter.sent) != 1 {
t.Fatalf("expected 1 send, got %d", len(adapter.sent))
}
if adapter.sent[0].Target != "123" || adapter.sent[0].Message.PlainText() != "ok" {
t.Fatalf("unexpected outbound message: %+v", adapter.sent[0])
}
}
func TestManagerSendUsesBinding(t *testing.T) {
t.Parallel()
log := slog.New(slog.NewTextHandler(io.Discard, &slog.HandlerOptions{}))
store := &fakeConfigStore{
effectiveConfig: ChannelConfig{
ID: "cfg-1",
BotID: "bot-1",
ChannelType: ChannelType("test"),
Credentials: map[string]any{"botToken": "token"},
UpdatedAt: time.Now(),
},
channelIdentityConfig: ChannelIdentityBinding{
ID: "binding-1",
Config: map[string]any{"target": "alice"},
},
}
reg := NewRegistry()
adapter := &fakeAdapter{channelType: ChannelType("test")}
manager := NewManager(log, reg, store, &fakeInboundProcessorIntegration{})
manager.RegisterAdapter(adapter)
err := manager.Send(context.Background(), "bot-1", ChannelType("test"), SendRequest{
ChannelIdentityID: "user-1",
Message: Message{
Text: "hello",
},
})
if err != nil {
t.Fatalf("expected no error, got %v", err)
}
adapter.mu.Lock()
defer adapter.mu.Unlock()
if len(adapter.sent) != 1 {
t.Fatalf("expected 1 send, got %d", len(adapter.sent))
}
if adapter.sent[0].Target != "resolved:alice" || adapter.sent[0].Message.PlainText() != "hello" {
t.Fatalf("unexpected outbound message: %+v", adapter.sent[0])
}
}
func TestManagerReconcileStartsAndStops(t *testing.T) {
t.Parallel()
log := slog.New(slog.NewTextHandler(io.Discard, &slog.HandlerOptions{}))
store := &fakeConfigStore{}
reg := NewRegistry()
adapter := &fakeAdapter{channelType: ChannelType("test")}
manager := NewManager(log, reg, store, &fakeInboundProcessorIntegration{})
manager.RegisterAdapter(adapter)
cfg := ChannelConfig{
ID: "cfg-1",
BotID: "bot-1",
ChannelType: ChannelType("test"),
Credentials: map[string]any{"botToken": "token"},
UpdatedAt: time.Now(),
}
manager.reconcile(context.Background(), []ChannelConfig{cfg})
manager.reconcile(context.Background(), nil)
adapter.mu.Lock()
defer adapter.mu.Unlock()
if len(adapter.started) != 1 {
t.Fatalf("expected 1 start, got %d", len(adapter.started))
}
if adapter.stops != 1 {
t.Fatalf("expected 1 stop, got %d", adapter.stops)
}
}