mirror of
https://github.com/snowy-yang/qqbotsdk.git
synced 2026-10-21 15:14:48 +00:00
snowy-yang
13a1d971be
实测 QQ 服务端在一个连接建立约 1 小时后会自己断开。此前只能等它断：
要么被看门狗判成静默超时、要么等读写失败，都算"异常断开"，观感与恢复
路径都不理想。现改为临到期前主动收尾重连——是正常断开 + RESUME 续传，
期间消息由网关补发。
实现：receive_helper 记录寿命截止点（_SERVER_LIFETIME - _RECONNECT_MARGIN，
默认 60min - 10min = 50min），收帧超时取"静默阈值"与"寿命余量"两个截止里
的先到者——静默过久仍抛 TimeoutError 交给看门狗，寿命到点则 return 正常
结束本轮连接，run() 见存活超过 _STABLE_SECONDS 便立刻重连（沿用 b37a53f
的"健康断开零等待"路径），RESUME 带上 session_id 续传。
_connect 的存活秒数天然 >= 50min > _STABLE_SECONDS，主动重连必被判定为
健康断开，无需额外标志位。
测试：新增 test_proactive_reconnect_before_server_lifetime（真实 ws 端到端，
把寿命阈值调到 0.5s 让这一刻立即发生）；已验证把寿命阈值关掉后该测试会
超时失败，不是空转到通过。另给 tests/test_reconnect.py 补 CHANGELOG 条目
（b37a53f 漏记），并同步 connecter 模块 docstring / ARCHITECTURE / GUIDE。
90 测试全绿，ruff check/format + pyright 干净。
227 lines
8.4 KiB
Python
227 lines
8.4 KiB
Python
"""WebSocket 接入的端到端集成测试：起一个假网关（token 接口 + gateway + ws），
|
||
真实跑通 HELLO → IDENTIFY → READY → 心跳 ACK → 业务事件分发的完整时序。"""
|
||
|
||
import asyncio
|
||
import json
|
||
from typing import Any
|
||
|
||
import pytest
|
||
from aiohttp import ClientSession, WSMsgType, web
|
||
|
||
from qqbotsdk.config import Config
|
||
from qqbotsdk.connecter import WebsocketConnecter
|
||
from qqbotsdk.emitter import EventEmitter
|
||
from qqbotsdk.model import Opcode
|
||
from qqbotsdk.payloads import GroupAtMessage
|
||
from qqbotsdk.queue import EventQueue
|
||
from qqbotsdk.session import Session
|
||
from qqbotsdk.token import AccessToken
|
||
from qqbotsdk.ws_protocol import WebsocketProtocol
|
||
|
||
|
||
class FakeGateway:
|
||
"""模拟 QQ 开放平台的鉴权、gateway 与 ws 协议行为。"""
|
||
|
||
def __init__(self) -> None:
|
||
self.identifies: list[dict] = []
|
||
self.heartbeats = 0
|
||
self.resumes: list[dict] = []
|
||
self.connections = 0
|
||
self.runner: web.AppRunner | None = None
|
||
self.port = 0
|
||
# 测试用同步点
|
||
self.ready_sent = asyncio.Event()
|
||
self.got_dispatch = asyncio.Event()
|
||
self.second_connection = asyncio.Event()
|
||
self.resume_received = asyncio.Event()
|
||
|
||
async def start(self) -> None:
|
||
app = web.Application()
|
||
app.router.add_post("/app/getAppAccessToken", self._token)
|
||
app.router.add_get("/gateway", self._gateway)
|
||
app.router.add_get("/ws", self._ws)
|
||
self.runner = web.AppRunner(app)
|
||
await self.runner.setup()
|
||
site = web.TCPSite(self.runner, "127.0.0.1", 0)
|
||
await site.start()
|
||
assert site._server is not None
|
||
self.port = site._server.sockets[0].getsockname()[1]
|
||
|
||
async def stop(self) -> None:
|
||
assert self.runner is not None
|
||
await self.runner.cleanup()
|
||
|
||
async def _token(self, request: web.Request) -> web.Response:
|
||
return web.json_response({"access_token": "test-token", "expires_in": 9999999})
|
||
|
||
async def _gateway(self, request: web.Request) -> web.Response:
|
||
return web.json_response({"url": f"http://127.0.0.1:{self.port}/ws"})
|
||
|
||
async def _ws(self, request: web.Request) -> web.WebSocketResponse:
|
||
ws = web.WebSocketResponse()
|
||
await ws.prepare(request)
|
||
self.connections += 1
|
||
if self.connections >= 2:
|
||
self.second_connection.set()
|
||
|
||
await ws.send_json({"op": 10, "d": {"heartbeat_interval": 300}})
|
||
s = 0
|
||
|
||
async for msg in ws:
|
||
if msg.type != WSMsgType.TEXT:
|
||
break
|
||
payload: dict[str, Any] = json.loads(msg.data)
|
||
op = payload.get("op")
|
||
if op == 2: # IDENTIFY
|
||
self.identifies.append(payload)
|
||
s += 1
|
||
await ws.send_json(
|
||
{
|
||
"op": 0,
|
||
"s": s,
|
||
"t": "READY",
|
||
"d": {"session_id": "S1", "user": {"id": "bot"}},
|
||
}
|
||
)
|
||
self.ready_sent.set()
|
||
# READY 之后立刻推一条业务事件
|
||
s += 1
|
||
await ws.send_json(
|
||
{
|
||
"op": 0,
|
||
"s": s,
|
||
"t": "GROUP_AT_MESSAGE_CREATE",
|
||
"d": {"content": "ping", "group_openid": "G1"},
|
||
}
|
||
)
|
||
elif op == 6: # RESUME
|
||
self.resumes.append(payload)
|
||
self.resume_received.set()
|
||
elif op == 1: # HEARTBEAT
|
||
self.heartbeats += 1
|
||
await ws.send_json({"op": 11})
|
||
return ws
|
||
|
||
|
||
def make_config(gw: FakeGateway) -> Config:
|
||
return Config(
|
||
app_id="app",
|
||
app_secret="secret",
|
||
base_url=f"http://127.0.0.1:{gw.port}",
|
||
timeout=None, # type: ignore[arg-type]
|
||
connecter="websocket",
|
||
intents_file="no-such-intents.toml", # 测试环境无配置，intents=0
|
||
webhook_host="127.0.0.1",
|
||
webhook_port=0,
|
||
webhook_path="/x",
|
||
)
|
||
|
||
|
||
def build_connecter(
|
||
config: Config,
|
||
) -> tuple[ClientSession, EventEmitter, WebsocketConnecter]:
|
||
http = ClientSession(timeout=config.timeout)
|
||
token = AccessToken(config, http)
|
||
session = Session()
|
||
emitter = EventEmitter(EventQueue())
|
||
protocol = WebsocketProtocol(config, session, token)
|
||
connecter = WebsocketConnecter(
|
||
config, http, token, session, emitter.queue, protocol
|
||
)
|
||
return http, emitter, connecter
|
||
|
||
|
||
@pytest.mark.asyncio
|
||
async def test_websocket_end_to_end():
|
||
gw = FakeGateway()
|
||
await gw.start()
|
||
try:
|
||
config = make_config(gw)
|
||
http = ClientSession(timeout=config.timeout)
|
||
token = AccessToken(config, http)
|
||
queue = EventQueue()
|
||
session = Session()
|
||
emitter = EventEmitter(queue)
|
||
protocol = WebsocketProtocol(config, session, token)
|
||
connecter = WebsocketConnecter(config, http, token, session, queue, protocol)
|
||
|
||
received: list[GroupAtMessage] = []
|
||
# 协议帧（HELLO/HEARTBEAT_ACK）绝不该触达 handler——只有 op=0 业务事件入队
|
||
protocol_frames: list[object] = []
|
||
|
||
@emitter.on("GROUP_AT_MESSAGE_CREATE")
|
||
async def on_msg(msg: GroupAtMessage) -> None:
|
||
received.append(msg)
|
||
gw.got_dispatch.set()
|
||
|
||
@emitter.on(Opcode.HELLO)
|
||
async def on_hello(_) -> None:
|
||
protocol_frames.append("HELLO")
|
||
|
||
@emitter.on(Opcode.HEARTBEAT_ACK)
|
||
async def on_ack(_) -> None:
|
||
protocol_frames.append("HEARTBEAT_ACK")
|
||
|
||
dispatch_task = asyncio.create_task(emitter.dispatch())
|
||
connect_task = asyncio.create_task(connecter.run())
|
||
try:
|
||
await asyncio.wait_for(gw.ready_sent.wait(), timeout=10)
|
||
await asyncio.wait_for(gw.got_dispatch.wait(), timeout=10)
|
||
await asyncio.sleep(0.5) # 留出至少一个心跳周期（300ms）
|
||
|
||
# READY 已被协议层捕获，session_id 用于后续 RESUME
|
||
assert session.session_id == "S1"
|
||
# 心跳定时器真实运行，服务端收到了 HEARTBEAT
|
||
assert gw.heartbeats >= 1
|
||
# IDENTIFY 形态正确（token 前缀 / shard / intents）
|
||
identify = gw.identifies[0]
|
||
assert identify["op"] == 2
|
||
assert identify["d"]["token"] == "QQBot test-token"
|
||
assert identify["d"]["intents"] == 0
|
||
assert identify["d"]["shard"] == [0, 1]
|
||
# 业务事件以 dataclass 形式分发到 handler
|
||
assert len(received) == 1
|
||
assert received[0].content == "ping"
|
||
assert received[0].group_openid == "G1"
|
||
# 协议帧（HELLO/HEARTBEAT_ACK）没进分发路径
|
||
assert protocol_frames == []
|
||
finally:
|
||
dispatch_task.cancel()
|
||
connect_task.cancel()
|
||
await asyncio.gather(dispatch_task, connect_task, return_exceptions=True)
|
||
await http.close()
|
||
finally:
|
||
await gw.stop()
|
||
|
||
|
||
@pytest.mark.asyncio
|
||
async def test_proactive_reconnect_before_server_lifetime():
|
||
"""服务端约 1 小时后会无预警掐断连接：应在临到期前主动重连。
|
||
|
||
把寿命阈值压到极短让这一刻在测试里立刻发生。第二次连接必须是 RESUME
|
||
——说明是正常收尾后按会话续传（服务端补发期间消息），而不是被掐断后
|
||
从头 IDENTIFY。
|
||
"""
|
||
gw = FakeGateway()
|
||
await gw.start()
|
||
try:
|
||
config = make_config(gw)
|
||
http, _emitter, connecter = build_connecter(config)
|
||
# 0.5s 寿命 - 0.3s 余量 = 连接存活约 0.2s 后主动重连；"稳定"阈值一并调小，
|
||
# 让这次主动重连被判定为正常收尾（立即重连，不退避）
|
||
connecter._SERVER_LIFETIME = 0.5
|
||
connecter._RECONNECT_MARGIN = 0.3
|
||
connecter._STABLE_SECONDS = 0.1
|
||
connect_task = asyncio.create_task(connecter.run())
|
||
try:
|
||
await asyncio.wait_for(gw.resume_received.wait(), timeout=10)
|
||
assert gw.connections >= 2 # 确实重连过
|
||
assert len(gw.identifies) == 1 # 只 IDENTIFY 过一次
|
||
assert gw.resumes[0]["d"]["session_id"] == "S1" # 续传同一会话
|
||
finally:
|
||
connect_task.cancel()
|
||
await asyncio.gather(connect_task, return_exceptions=True)
|
||
await http.close()
|
||
finally:
|
||
await gw.stop()
|