Replaced the "ab_server PCCC upstream-broken" skip gate with the actual
root cause: libplctag's ab_server rejects empty CIP routing paths at the
unconnected-send layer before the PCCC dispatcher runs. Real SLC/
MicroLogix/PLC-5 hardware accepts empty paths (no backplane); ab_server
does not. With `/1,0` in place, N (Int16), F (Float32), and L (Int32)
file reads + writes round-trip cleanly across all three compose profiles.
## Fixture changes
- `AbLegacyServerFixture.cs`:
- Drop `AB_LEGACY_TRUST_WIRE` env var + the reachable-but-untrusted
skip branch. Fixture now only skips on TCP unreachability.
- Add `AB_LEGACY_CIP_PATH` env var (default `1,0`) + expose `CipPath`
property. Set `AB_LEGACY_CIP_PATH=` (empty) against real hardware.
- Shorter skip messages on the `[AbLegacyFact]` / `[AbLegacyTheory]`
attributes — one reason: endpoint not reachable.
- `AbLegacyReadSmokeTests.cs`:
- Device URI built from `sim.CipPath` instead of hardcoded empty path.
- New `AB_LEGACY_COMPOSE_PROFILE` env var filters the parametric
theory to the running container's family. Only one container binds
`:44818` at a time, so cross-family params would otherwise fail.
- `Slc500_write_then_read_round_trip` skips cleanly when the running
profile isn't `slc500`.
## E2E + seed + docs
- `scripts/e2e/test-ablegacy.ps1` — drop the `AB_LEGACY_TRUST_WIRE`
skip gate; synopsis calls out the `/1,0` vs empty cip-path split
between the Docker fixture and real hardware.
- `scripts/e2e/e2e-config.sample.json` — sample gateway flipped from
the hardware placeholder (`192.168.1.10`) to the Docker fixture
(`127.0.0.1/1,0`); comment rewritten.
- `scripts/e2e/README.md` — AB Legacy expected-matrix row goes from
SKIP to PASS.
- `scripts/smoke/seed-ablegacy-smoke.sql` — default HostAddress points
at the Docker fixture + header / footer text reflect the new state.
- `tests/.../Docker/README.md` — "Known limitations" section rewritten
to describe the cip-path gate (not a dispatcher gap); env-var table
picks up `AB_LEGACY_CIP_PATH` + `AB_LEGACY_COMPOSE_PROFILE`.
- `docs/drivers/AbLegacy-Test-Fixture.md` + `docs/drivers/README.md`
+ `docs/DriverClis.md` — flip status from blocked to functional;
residual bit-file-write gap (B3:0/5 → 0x803D0000) documented.
## Residual gap
Bit-file writes (`B3:0/5` style) surface `0x803D0000` against
`ab_server --plc=SLC500`; bit reads work. Non-blocking for smoke
coverage — N/F/L round-trip is enough. Real hardware / RSEmulate 500
for bit-write fidelity. Documented in `Docker/README.md` §"Known
limitations" + the `AbLegacy-Test-Fixture.md` follow-ups list.
## Verified
- Full-solution build: 0 errors, 334 pre-existing warnings.
- Integration suite passes per-profile with
`AB_LEGACY_COMPOSE_PROFILE=<slc500|micrologix|plc5>` + matching
compose container up.
- All four non-hardware e2e scripts (Modbus / AB CIP / AB Legacy / S7)
now 5/5 against the respective docker-compose fixtures.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
122 lines
5.6 KiB
C#
122 lines
5.6 KiB
C#
using Shouldly;
|
|
using Xunit;
|
|
using ZB.MOM.WW.OtOpcUa.Core.Abstractions;
|
|
using ZB.MOM.WW.OtOpcUa.Driver.AbLegacy.PlcFamilies;
|
|
|
|
namespace ZB.MOM.WW.OtOpcUa.Driver.AbLegacy.IntegrationTests;
|
|
|
|
/// <summary>
|
|
/// End-to-end smoke tests against the <c>ab_server</c> PCCC Docker container.
|
|
/// Promotes the AB Legacy driver from unit-only coverage (<c>FakeAbLegacyTag</c>)
|
|
/// to wire-level: real libplctag PCCC stack over real TCP against the ab_server
|
|
/// simulator. Parametrised over all three families (SLC 500 / MicroLogix / PLC-5)
|
|
/// via <c>[AbLegacyTheory]</c> + <c>[MemberData]</c>.
|
|
/// </summary>
|
|
[Collection(AbLegacyServerCollection.Name)]
|
|
[Trait("Category", "Integration")]
|
|
[Trait("Simulator", "ab_server-PCCC")]
|
|
public sealed class AbLegacyReadSmokeTests(AbLegacyServerFixture sim)
|
|
{
|
|
// Only one ab_server container binds :44818 at a time and `--plc=SLC500` only
|
|
// answers SLC-mode PCCC, etc. When `AB_LEGACY_COMPOSE_PROFILE` is set, the theory
|
|
// filters to that profile alone so the suite matches the running container. Unset
|
|
// (the default for real-hardware runs) parameterises across every family the driver
|
|
// supports.
|
|
public static IEnumerable<object[]> Profiles
|
|
{
|
|
get
|
|
{
|
|
var only = Environment.GetEnvironmentVariable("AB_LEGACY_COMPOSE_PROFILE");
|
|
var profiles = KnownProfiles.All.Where(p =>
|
|
string.IsNullOrEmpty(only) ||
|
|
string.Equals(p.ComposeProfile, only, StringComparison.OrdinalIgnoreCase));
|
|
return profiles.Select(p => new object[] { p });
|
|
}
|
|
}
|
|
|
|
[AbLegacyTheory]
|
|
[MemberData(nameof(Profiles))]
|
|
public async Task Driver_reads_seeded_N_file_from_ab_server_PCCC(AbLegacyServerProfile profile)
|
|
{
|
|
if (sim.SkipReason is not null) Assert.Skip(sim.SkipReason);
|
|
|
|
// PCCC semantics allow an empty cip-path (real SLC/PLC-5 hardware takes nothing
|
|
// after the `/`), but libplctag's ab_server requires a non-empty path at the
|
|
// CIP unconnected-send layer before the PCCC dispatcher runs. Default `1,0`
|
|
// against the Docker fixture; set AB_LEGACY_CIP_PATH= (empty) against real HW.
|
|
var deviceUri = $"ab://{sim.Host}:{sim.Port}/{sim.CipPath}";
|
|
await using var drv = new AbLegacyDriver(new AbLegacyDriverOptions
|
|
{
|
|
Devices = [new AbLegacyDeviceOptions(deviceUri, profile.Family)],
|
|
Tags = [
|
|
new AbLegacyTagDefinition(
|
|
Name: "IntCounter",
|
|
DeviceHostAddress: deviceUri,
|
|
Address: "N7:0",
|
|
DataType: AbLegacyDataType.Int),
|
|
],
|
|
Timeout = TimeSpan.FromSeconds(5),
|
|
Probe = new AbLegacyProbeOptions { Enabled = false },
|
|
}, driverInstanceId: $"ablegacy-smoke-{profile.Family}");
|
|
|
|
await drv.InitializeAsync("{}", TestContext.Current.CancellationToken);
|
|
|
|
var snapshots = await drv.ReadAsync(
|
|
["IntCounter"], TestContext.Current.CancellationToken);
|
|
|
|
snapshots.Single().StatusCode.ShouldBe(AbLegacyStatusMapper.Good,
|
|
$"N7:0 read must succeed against the {profile.Family} compose profile");
|
|
drv.GetHealth().State.ShouldBe(DriverState.Healthy);
|
|
}
|
|
|
|
[AbLegacyFact]
|
|
public async Task Slc500_write_then_read_round_trip_on_N7_scratch_register()
|
|
{
|
|
if (sim.SkipReason is not null) Assert.Skip(sim.SkipReason);
|
|
|
|
// Skip when the running compose profile isn't SLC500 — ab_server's `--plc=`
|
|
// flag selects exactly one family per process, so a write against a plc5-mode
|
|
// container with SLC500 semantics always fails at the wire.
|
|
var only = Environment.GetEnvironmentVariable("AB_LEGACY_COMPOSE_PROFILE");
|
|
if (!string.IsNullOrEmpty(only) &&
|
|
!string.Equals(only, "slc500", StringComparison.OrdinalIgnoreCase))
|
|
{
|
|
Assert.Skip($"Test targets the SLC500 compose profile; AB_LEGACY_COMPOSE_PROFILE='{only}'.");
|
|
}
|
|
|
|
// PCCC semantics allow an empty cip-path (real SLC/PLC-5 hardware takes nothing
|
|
// after the `/`), but libplctag's ab_server requires a non-empty path at the
|
|
// CIP unconnected-send layer before the PCCC dispatcher runs. Default `1,0`
|
|
// against the Docker fixture; set AB_LEGACY_CIP_PATH= (empty) against real HW.
|
|
var deviceUri = $"ab://{sim.Host}:{sim.Port}/{sim.CipPath}";
|
|
await using var drv = new AbLegacyDriver(new AbLegacyDriverOptions
|
|
{
|
|
Devices = [new AbLegacyDeviceOptions(deviceUri, AbLegacyPlcFamily.Slc500)],
|
|
Tags = [
|
|
new AbLegacyTagDefinition(
|
|
Name: "Scratch",
|
|
DeviceHostAddress: deviceUri,
|
|
Address: "N7:5",
|
|
DataType: AbLegacyDataType.Int,
|
|
Writable: true),
|
|
],
|
|
Timeout = TimeSpan.FromSeconds(5),
|
|
Probe = new AbLegacyProbeOptions { Enabled = false },
|
|
}, driverInstanceId: "ablegacy-smoke-rw");
|
|
|
|
await drv.InitializeAsync("{}", TestContext.Current.CancellationToken);
|
|
|
|
const short probe = 0x1234;
|
|
var writeResults = await drv.WriteAsync(
|
|
[new WriteRequest("Scratch", probe)],
|
|
TestContext.Current.CancellationToken);
|
|
writeResults.Single().StatusCode.ShouldBe(AbLegacyStatusMapper.Good,
|
|
"PCCC N7:5 write must succeed end-to-end");
|
|
|
|
var readResults = await drv.ReadAsync(
|
|
["Scratch"], TestContext.Current.CancellationToken);
|
|
readResults.Single().StatusCode.ShouldBe(AbLegacyStatusMapper.Good);
|
|
Convert.ToInt32(readResults.Single().Value).ShouldBe(probe);
|
|
}
|
|
}
|