CydandClaude Opus 4.8
b3cb764f4d
Port the RIO wire protocol from legacy/riovjoy2.cpp into testable C#: - Protocol/: command + length table, 7-bit checksum, packet builder, and a streaming receive-side framing state machine (PacketParser) that mirrors the legacy ReadCommBlock framing/resync (high-bit-mid-packet abort). Typed RIO->PC decodes: AnalogReport (14-bit sign-extend), VersionInfo, CheckStatus; lamp-state composition. - Serial/: RioSerialLink drives an async receive loop with ACK/NAK reply policy (legacy force-accept vs. opt-in VerifyInboundChecksum), the analog poll timer, and the >5s reset-recovery watchdog. IRioTransport abstracts the COM port; the SerialPort-backed transport does 9600 8N1 + DTR reset pulse, and acquire/release is just create/dispose (foundation for native-game serial yield). - tests/RioJoy.Core.Tests: 54 xUnit tests covering checksum, framing/resync, builder round-trips, analog sign-extension + sentinel rejection, lamp combos, and the read loop driven against an in-memory fake transport. Hardware verification (version/check/analog against a cabinet) remains; it can't be done off-device. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
52 lines
1.5 KiB
C#
52 lines
1.5 KiB
C#
using RioJoy.Core.Protocol;
|
|
using Xunit;
|
|
|
|
namespace RioJoy.Core.Tests.Protocol;
|
|
|
|
public class RioCommandTableTests
|
|
{
|
|
[Theory]
|
|
[InlineData(RioCommand.CheckRequest, 0)]
|
|
[InlineData(RioCommand.VersionRequest, 0)]
|
|
[InlineData(RioCommand.AnalogRequest, 0)]
|
|
[InlineData(RioCommand.ResetRequest, 1)]
|
|
[InlineData(RioCommand.LampRequest, 2)]
|
|
[InlineData(RioCommand.CheckReply, 2)]
|
|
[InlineData(RioCommand.VersionReply, 2)]
|
|
[InlineData(RioCommand.AnalogReply, 10)]
|
|
[InlineData(RioCommand.ButtonPressed, 1)]
|
|
[InlineData(RioCommand.ButtonReleased, 1)]
|
|
[InlineData(RioCommand.KeyPressed, 2)]
|
|
[InlineData(RioCommand.KeyReleased, 2)]
|
|
[InlineData(RioCommand.TestModeChange, 1)]
|
|
public void PayloadLengths_MatchProtocol(RioCommand command, int expected)
|
|
{
|
|
Assert.Equal(expected, RioCommandTable.PayloadLength(command));
|
|
}
|
|
|
|
[Fact]
|
|
public void Bounds_AreInclusive()
|
|
{
|
|
Assert.Equal(0x80, RioCommandTable.Base);
|
|
Assert.Equal(0x8C, RioCommandTable.Max);
|
|
Assert.Equal(13, RioCommandTable.Count);
|
|
}
|
|
|
|
[Theory]
|
|
[InlineData(0x7F, false)]
|
|
[InlineData(0x80, true)]
|
|
[InlineData(0x8C, true)]
|
|
[InlineData(0x8D, false)]
|
|
[InlineData(0xFC, false)]
|
|
public void IsCommand_RespectsRange(byte b, bool expected)
|
|
{
|
|
Assert.Equal(expected, RioCommandTable.IsCommand(b));
|
|
}
|
|
|
|
[Fact]
|
|
public void PayloadLength_ThrowsForNonCommand()
|
|
{
|
|
Assert.Throws<ArgumentOutOfRangeException>(() => RioCommandTable.PayloadLength((byte)0x8D));
|
|
}
|
|
}
|