Files
dingtalk-workspace-cli/internal/event/busctl/discover_test.go
aa6abc5ed6 feat(event): add personal event subscriptions (#589)
* dws event

* fix consume fail

* test: add stream ticket injection probe

* feat: add portal ticket stream mode

* user event

* fix: allow portal ticket normal without app secret

* event

* user event

* eventType filter

* refactor(event): 优化IPC端点路径处理和改进相关测试

- 用dwsevent.IPCEndpoint替代原先根据GOOS判断的路径逻辑
- 新增event包实现Unix socket路径长度限制及长路径fallback机制
- 添加endpoint_test.go覆盖路径短长及唯一性的单元测试
- 修改busctl模块使用统一的IPC端点获取方法,避免重复实现
- transport_unix.go新增checkSocketPath函数检查路径长度,防止EINVAL错误
- 在监听和连接Unix socket时加入路径限制检查,提升错误明晰度
- 去除多个文件中无用的runtime导入,简化代码依赖

* opt

* event skill

* default value

* install script event

* fix: remove subscribe id event fanout filter

* fix(personal): 修正指定发送人消息描述错误

* more im event

* filter subId

* fix: align personal event schema with stream payload

* fix: avoid duplicate app helper name

* feat: simplify personal event schemas

* feat: simplify event schema output

* docs: refine dingtalk event skill references

* feat: align personal event consume flags

* fix event stop and status visibility

* hide app event public entrypoints

* hide incomplete personal sender event

* remove external event reference comments

* chore(event): prepare official release

* fix(event): harden personal stream lifecycle

---------

Co-authored-by: 玉澜 <yulan.wqy@alibaba-inc.com>
Co-authored-by: zhengyubai <zhengyubai618@gmail.com>
2026-07-10 17:54:43 +08:00

281 lines
6.8 KiB
Go

// Copyright 2026 Alibaba Group
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
package busctl
import (
"errors"
"net"
"os"
"path/filepath"
"runtime"
"sync"
"sync/atomic"
"testing"
"time"
"github.com/DingTalk-Real-AI/dingtalk-workspace-cli/internal/event/transport"
)
func skipOnWindows(t *testing.T) {
t.Helper()
if runtime.GOOS == "windows" {
t.Skip("uses Unix socket; Windows transport covered by transport_windows_test.go")
}
}
func shortTempDir(t *testing.T) string {
t.Helper()
dir, err := os.MkdirTemp("/tmp", "dws-busctl-")
if err != nil {
t.Fatalf("mktemp: %v", err)
}
t.Cleanup(func() { _ = os.RemoveAll(dir) })
return dir
}
// startStubBus brings up a transport.Listener at sockPath and accepts
// connections in a goroutine, discarding the data. Returns a closer.
// Used as a stand-in for the real bus daemon in discover unit tests.
func startStubBus(t *testing.T, sockPath string) func() {
t.Helper()
l, err := transport.Listen(sockPath)
if err != nil {
t.Fatalf("startStubBus listen: %v", err)
}
done := make(chan struct{})
go func() {
for {
conn, err := l.Accept()
if err != nil {
close(done)
return
}
go func(c net.Conn) {
buf := make([]byte, 256)
for {
if _, err := c.Read(buf); err != nil {
_ = c.Close()
return
}
}
}(conn)
}
}()
return func() {
_ = l.Close()
<-done
}
}
func TestDiscover_BusAlreadyRunning_DirectDial(t *testing.T) {
skipOnWindows(t)
dir := shortTempDir(t)
sock := filepath.Join(dir, "bus.sock")
closer := startStubBus(t, sock)
defer closer()
var spawnCalled atomic.Bool
fakeSpawn := func(SpawnConfig) (int, error) {
spawnCalled.Store(true)
return 0, nil
}
conn, err := Discover(DiscoverConfig{
WorkDir: dir,
IPCEndpoint: sock,
ClientID: "ding_x",
Spawn: fakeSpawn,
})
if err != nil {
t.Fatalf("Discover: %v", err)
}
defer conn.Close()
if spawnCalled.Load() {
t.Fatal("Spawn must not be called when bus is already running")
}
}
func TestDiscover_NoBus_SpawnSucceeds(t *testing.T) {
skipOnWindows(t)
dir := shortTempDir(t)
sock := filepath.Join(dir, "bus.sock")
// fakeSpawn launches the stub bus *during* the spawn call to mirror
// the real flow (bus is up by the time spawn returns).
var closer func()
t.Cleanup(func() {
if closer != nil {
closer()
}
})
fakeSpawn := func(SpawnConfig) (int, error) {
closer = startStubBus(t, sock)
return 12345, nil
}
conn, err := Discover(DiscoverConfig{
WorkDir: dir,
IPCEndpoint: sock,
ClientID: "ding_x",
Spawn: fakeSpawn,
})
if err != nil {
t.Fatalf("Discover: %v", err)
}
defer conn.Close()
}
func TestDiscover_SpawnHardErrorFails(t *testing.T) {
skipOnWindows(t)
dir := shortTempDir(t)
sock := filepath.Join(dir, "bus.sock")
hardErr := errors.New("exec failed: not found")
fakeSpawn := func(SpawnConfig) (int, error) { return 0, hardErr }
_, err := Discover(DiscoverConfig{
WorkDir: dir,
IPCEndpoint: sock,
ClientID: "ding_x",
Spawn: fakeSpawn,
DialDeadline: 100 * time.Millisecond,
})
if err == nil {
t.Fatal("Discover should fail when Spawn returns non-ErrSpawnFailed error")
}
if !errors.Is(err, hardErr) {
t.Fatalf("err = %v, want wrap of %v", err, hardErr)
}
}
func TestDiscover_SpawnReportsFailButPeerBusComesUp(t *testing.T) {
skipOnWindows(t)
dir := shortTempDir(t)
sock := filepath.Join(dir, "bus.sock")
// Simulates the race: our Spawn loses (returns ErrSpawnFailed) but
// during retry-dial a peer bus shows up.
go func() {
time.Sleep(80 * time.Millisecond)
closer := startStubBus(t, sock)
t.Cleanup(closer)
}()
fakeSpawn := func(SpawnConfig) (int, error) { return 0, ErrSpawnFailed }
conn, err := Discover(DiscoverConfig{
WorkDir: dir,
IPCEndpoint: sock,
ClientID: "ding_x",
Spawn: fakeSpawn,
DialDeadline: 2 * time.Second,
DialBackoff: 20 * time.Millisecond,
})
if err != nil {
t.Fatalf("Discover should retry-dial after ErrSpawnFailed: %v", err)
}
defer conn.Close()
}
func TestDiscover_DeadlineExceeded(t *testing.T) {
skipOnWindows(t)
dir := shortTempDir(t)
sock := filepath.Join(dir, "bus.sock") // never created
fakeSpawn := func(SpawnConfig) (int, error) { return 0, ErrSpawnFailed }
start := time.Now()
_, err := Discover(DiscoverConfig{
WorkDir: dir,
IPCEndpoint: sock,
ClientID: "ding_x",
Spawn: fakeSpawn,
DialDeadline: 150 * time.Millisecond,
DialBackoff: 20 * time.Millisecond,
})
if err == nil {
t.Fatal("Discover should fail when bus never comes up within deadline")
}
if elapsed := time.Since(start); elapsed < 100*time.Millisecond || elapsed > 1*time.Second {
t.Errorf("deadline-driven exit took %s, expected ~150ms", elapsed)
}
}
func TestDiscover_ConcurrentCallersOnlyOneSpawn(t *testing.T) {
skipOnWindows(t)
dir := shortTempDir(t)
sock := filepath.Join(dir, "bus.sock")
// Use a mutex-guarded one-shot Spawn that actually starts the stub bus
// on first call. Concurrent callers may race: the first wins (returns
// success), the rest see ErrSpawnFailed but still retry-dial successfully.
var spawnMu sync.Mutex
var spawnCount atomic.Int32
var closer func()
t.Cleanup(func() {
if closer != nil {
closer()
}
})
fakeSpawn := func(SpawnConfig) (int, error) {
spawnMu.Lock()
defer spawnMu.Unlock()
spawnCount.Add(1)
if closer == nil {
closer = startStubBus(t, sock)
return 99, nil
}
return 0, ErrSpawnFailed
}
const N = 5
var wg sync.WaitGroup
errs := make(chan error, N)
conns := make([]net.Conn, 0, N)
connsMu := sync.Mutex{}
for i := 0; i < N; i++ {
wg.Add(1)
go func() {
defer wg.Done()
conn, err := Discover(DiscoverConfig{
WorkDir: dir,
IPCEndpoint: sock,
ClientID: "ding_x",
Spawn: fakeSpawn,
DialDeadline: 2 * time.Second,
})
if err != nil {
errs <- err
return
}
connsMu.Lock()
conns = append(conns, conn)
connsMu.Unlock()
}()
}
wg.Wait()
close(errs)
for err := range errs {
t.Errorf("Discover concurrent caller: %v", err)
}
for _, c := range conns {
_ = c.Close()
}
// Note: spawnCount can be 1..N because all goroutines fail dial first
// and call Spawn. The point is they all SUCCESSFULLY connected to the
// single bus that the first Spawn brought up.
if len(conns) != N {
t.Fatalf("only %d/%d callers got a conn", len(conns), N)
}
}