红尘
d73ff9b0fb
Deploy Site / deploy-vercel (push) Has been cancelled
Deploy Site / deploy-docs (push) Has been cancelled
Docker / shell lint / Lint Dockerfile (hadolint) (push) Has been cancelled
Docker / shell lint / Lint docker/ shell scripts (shellcheck) (push) Has been cancelled
Docker Build and Publish / build-amd64 (push) Has been cancelled
Docker Build and Publish / build-arm64 (push) Has been cancelled
Lint (ruff + ty) / ruff + ty diff (push) Has been cancelled
Lint (ruff + ty) / ruff enforcement (blocking) (push) Has been cancelled
Lint (ruff + ty) / Windows footguns (blocking) (push) Has been cancelled
Nix Lockfile Fix / auto-fix-main (push) Has been cancelled
Nix Lockfile Fix / fix (push) Has been cancelled
Nix / nix (macos-latest) (push) Has been cancelled
Nix / nix (ubuntu-latest) (push) Has been cancelled
OSV-Scanner / Scan lockfiles (push) Has been cancelled
Build Skills Index / build-index (push) Has been cancelled
Tests / test (1) (push) Has been cancelled
Tests / test (2) (push) Has been cancelled
Tests / test (3) (push) Has been cancelled
Tests / test (4) (push) Has been cancelled
Tests / test (5) (push) Has been cancelled
Tests / test (6) (push) Has been cancelled
Tests / e2e (push) Has been cancelled
uv.lock check / uv lock --check (push) Has been cancelled
Docker Build and Publish / merge (push) Has been cancelled
Build Skills Index / trigger-deploy (push) Has been cancelled
Tests / save-durations (push) Has been cancelled
68 lines
3.1 KiB
Python
68 lines
3.1 KiB
Python
"""Tests for Discord Opus codec loading — must use ctypes.util.find_library."""
|
|
|
|
import inspect
|
|
import types
|
|
|
|
|
|
class TestOpusFindLibrary:
|
|
"""Opus loading must try ctypes.util.find_library first, with platform fallback."""
|
|
|
|
def test_uses_find_library_first(self):
|
|
"""find_library must be the primary lookup strategy."""
|
|
from plugins.platforms.discord.adapter import DiscordAdapter
|
|
source = inspect.getsource(DiscordAdapter.connect)
|
|
assert "find_library" in source, \
|
|
"Opus loading must use ctypes.util.find_library"
|
|
|
|
def test_homebrew_fallback_is_conditional(self):
|
|
"""Homebrew paths must only be tried when find_library returns None."""
|
|
from plugins.platforms.discord.adapter import DiscordAdapter
|
|
source = inspect.getsource(DiscordAdapter.connect)
|
|
# Homebrew fallback must exist
|
|
assert "/opt/homebrew" in source or "homebrew" in source, \
|
|
"Opus loading should have macOS Homebrew fallback"
|
|
# find_library must appear BEFORE any Homebrew path
|
|
fl_idx = source.index("find_library")
|
|
hb_idx = source.index("/opt/homebrew")
|
|
assert fl_idx < hb_idx, \
|
|
"find_library must be tried before Homebrew fallback paths"
|
|
# Fallback must be guarded by platform check
|
|
assert "sys.platform" in source or "darwin" in source, \
|
|
"Homebrew fallback must be guarded by macOS platform check"
|
|
|
|
def test_windows_bundled_discord_opus_dll_is_discovered(self, monkeypatch, tmp_path):
|
|
"""Native Windows installs should try discord.py's bundled opus DLL."""
|
|
import plugins.platforms.discord.adapter as adapter
|
|
|
|
opus_py = tmp_path / "discord" / "opus.py"
|
|
bundled = opus_py.parent / "bin" / "libopus-0.x64.dll"
|
|
bundled.parent.mkdir(parents=True)
|
|
opus_py.write_text("# fake discord.opus module\n")
|
|
bundled.write_bytes(b"fake dll")
|
|
|
|
discord_stub = types.SimpleNamespace(
|
|
opus=types.SimpleNamespace(__file__=str(opus_py))
|
|
)
|
|
monkeypatch.setattr(adapter.sys, "platform", "win32")
|
|
monkeypatch.setattr(adapter.struct, "calcsize", lambda _fmt: 8)
|
|
|
|
assert adapter._find_discord_windows_bundled_opus(discord_stub) == str(
|
|
bundled.resolve()
|
|
)
|
|
|
|
def test_opus_decode_error_logged(self):
|
|
"""Opus decode failure must log the error, not silently return."""
|
|
from plugins.platforms.discord.adapter import VoiceReceiver
|
|
source = inspect.getsource(VoiceReceiver._on_packet)
|
|
assert "logger" in source, \
|
|
"_on_packet must log Opus decode errors"
|
|
assert "self._decoders.pop" in source, \
|
|
"_on_packet must reset the Opus decoder after decode failures"
|
|
# Must not have bare `except Exception:\n return`
|
|
lines = source.split("\n")
|
|
for i, line in enumerate(lines):
|
|
if "except Exception" in line and i + 1 < len(lines):
|
|
next_line = lines[i + 1].strip()
|
|
assert next_line != "return", \
|
|
f"_on_packet has bare 'except Exception: return' at line {i+1}"
|