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>
32 lines
864 B
C#
32 lines
864 B
C#
using RioJoy.Core.Protocol;
|
|
using Xunit;
|
|
|
|
namespace RioJoy.Core.Tests.Protocol;
|
|
|
|
public class RioRepliesTests
|
|
{
|
|
[Fact]
|
|
public void VersionInfo_ParsesMajorMinor()
|
|
{
|
|
VersionInfo v = VersionInfo.Parse(new byte[] { 0, 3 });
|
|
Assert.Equal(0, v.Major);
|
|
Assert.Equal(3, v.Minor);
|
|
Assert.Equal("0.3", v.ToString());
|
|
}
|
|
|
|
[Fact]
|
|
public void CheckStatus_ParsesTypeAndNumber()
|
|
{
|
|
CheckStatus s = CheckStatus.Parse(new byte[] { (byte)RioStatusType.BoardMissing, 7 });
|
|
Assert.Equal(RioStatusType.BoardMissing, s.Type);
|
|
Assert.Equal(7, s.Number);
|
|
}
|
|
|
|
[Fact]
|
|
public void Parse_RejectsWrongLength()
|
|
{
|
|
Assert.Throws<ArgumentException>(() => VersionInfo.Parse(new byte[] { 1 }));
|
|
Assert.Throws<ArgumentException>(() => CheckStatus.Parse(new byte[] { 1, 2, 3 }));
|
|
}
|
|
}
|