f2efeb37b7
Tasks 5 and 6 of the Phase 2 plan, committed together because their test
fallout is entangled — several fixtures construct both stores.
StoreAndForwardStorage and SiteStorageService now take ILocalDb. Connections
come from ILocalDb.CreateConnection(), which hands out an already-open,
pragma-configured connection carrying the zb_hlc_next() UDF the capture triggers
call; a raw connection would lack the UDF and every write to a replicated table
would fail closed. Deleted with the connection strings: S&F's
EnsureDatabaseDirectoryExists and its per-open busy_timeout pragma, and the site
service's BusyTimeoutFloorSeconds normalization — LocalDb owns all of it now.
DI: AddSiteRuntime's string overload is gone (nothing left to supply), so the
Host calls the no-arg form. ScadaBridge:Database:SiteDbPath and
StoreAndForwardOptions.SqliteDbPath survive only as the migrator's source
locations in Tasks 8/9.
Two things the plan did not anticipate, both worth reading:
1. FOUND A REAL LATENT DEFECT, from Phase 1, now fixed. The plan assumed
directory creation simply moved to LocalDb along with file ownership. It did
not: the LocalDb library never creates the parent directory, and
SqliteLocalDb opens the file eagerly in its constructor — so a missing
directory is a hard boot failure ("SQLite Error 14: unable to open database
file"), not a degraded start. The default site config points at the RELATIVE
path ./data/site-localdb.db, so any site node without a pre-existing data/
directory fails to boot. The docker rig escapes only because its volume mount
happens to create /app/data — a coincidence that would have hidden this until
a bare-metal or fresh deployment. This has been latent since Phase 1 made
LocalDb:Path required; deleting S&F's EnsureDatabaseDirectoryExists here
would have widened it. Re-established the guarantee at the layer that now
owns the path (SiteLocalDbDirectory.Ensure, called before AddZbLocalDb) and
pinned it with SiteLocalDbDirectoryTests. Non-vacuity is not assumed: two
tests written against the wrong assumption failed with exactly this
SQLite Error 14 before the fix existed.
2. Test fallout was ~7x the plan's estimate. The plan named "fixtures" in one
project; the constructor change actually reaches 40 files across 7 test
projects, and most used Mode=Memory;Cache=Shared — which LocalDb has no
equivalent for, so every one had to move to a real temp file. Rather than
copy the Phase 1 TestLocalDb fixture into 7 projects, added a shared
tests/ZB.MOM.WW.ScadaBridge.TestSupport library (not a test project) so the
WAL-sidecar cleanup and the "real, not stubbed" rationale live in one place.
Retargeted rather than deleted, in both directions: the S&F WAL test now asserts
against the LocalDb-backed store (WAL genuinely is LocalDb's job), while the
directory-creation test moved to Host.Tests (that guarantee is NOT LocalDb's).
SiteStorageServiceTests.Initialize_EnablesWalJournalMode got the same treatment.
DeploymentManagerMediumFindingsTests induced a persistence failure via an
unopenable path, which no longer reaches the assertion since the fixture now
throws first; it induces the same failure shape via an uninitialized store.
Verified: full solution build 0 warnings; SiteRuntime 532, Host 318,
AuditLog 355, ExternalSystemGateway 142, HealthMonitoring 97,
StoreAndForward 153 — 1597 passed, 0 failed.
Claude-Session: https://claude.ai/code/session_01BL2Vu1ESDQ9SCN4gVKkdts
229 lines
8.2 KiB
C#
229 lines
8.2 KiB
C#
using Akka.Actor;
|
|
using Akka.TestKit.Xunit2;
|
|
using Microsoft.Extensions.Logging.Abstractions;
|
|
using ZB.MOM.WW.ScadaBridge.Commons.Messages.DebugView;
|
|
using ZB.MOM.WW.ScadaBridge.Commons.Messages.Instance;
|
|
using ZB.MOM.WW.ScadaBridge.Commons.Messages.ScriptExecution;
|
|
using ZB.MOM.WW.ScadaBridge.Commons.Messages.Streaming;
|
|
using ZB.MOM.WW.ScadaBridge.Commons.Types.Flattening;
|
|
using ZB.MOM.WW.ScadaBridge.SiteRuntime.Actors;
|
|
using ZB.MOM.WW.ScadaBridge.SiteRuntime.Persistence;
|
|
using ZB.MOM.WW.ScadaBridge.SiteRuntime.Scripts;
|
|
using ZB.MOM.WW.ScadaBridge.TestSupport;
|
|
using System.Text.Json;
|
|
|
|
namespace ZB.MOM.WW.ScadaBridge.SiteRuntime.Tests.Actors;
|
|
|
|
/// <summary>
|
|
/// Integration tests for InstanceActor with child Script/Alarm actors (WP-15, WP-16, WP-24, WP-25).
|
|
/// </summary>
|
|
public class InstanceActorIntegrationTests : TestKit, IDisposable
|
|
{
|
|
private readonly SiteStorageService _storage;
|
|
private readonly ScriptCompilationService _compilationService;
|
|
private readonly SharedScriptLibrary _sharedScriptLibrary;
|
|
private readonly SiteRuntimeOptions _options;
|
|
private readonly TestLocalDb _localDb;
|
|
|
|
public InstanceActorIntegrationTests()
|
|
{
|
|
_localDb = TestLocalDb.CreateTemp("instance-int-test");
|
|
_storage = new SiteStorageService(
|
|
_localDb.Db,
|
|
NullLogger<SiteStorageService>.Instance);
|
|
_storage.InitializeAsync().GetAwaiter().GetResult();
|
|
_compilationService = new ScriptCompilationService(
|
|
NullLogger<ScriptCompilationService>.Instance);
|
|
_sharedScriptLibrary = new SharedScriptLibrary(
|
|
_compilationService, NullLogger<SharedScriptLibrary>.Instance);
|
|
_options = new SiteRuntimeOptions
|
|
{
|
|
MaxScriptCallDepth = 10,
|
|
ScriptExecutionTimeoutSeconds = 30
|
|
};
|
|
|
|
}
|
|
|
|
void IDisposable.Dispose()
|
|
{
|
|
// TestKit teardown first, so no in-flight actor can reach a disposed ILocalDb;
|
|
// then dispose the database before deleting — the master connection anchors the WAL.
|
|
Shutdown();
|
|
var path = _localDb.Path;
|
|
_localDb.Dispose();
|
|
TestLocalDb.DeleteFiles(path);
|
|
}
|
|
|
|
private IActorRef CreateInstanceWithScripts(
|
|
string instanceName,
|
|
IReadOnlyList<ResolvedScript>? scripts = null,
|
|
IReadOnlyList<ResolvedAlarm>? alarms = null)
|
|
{
|
|
var config = new FlattenedConfiguration
|
|
{
|
|
InstanceUniqueName = instanceName,
|
|
Attributes =
|
|
[
|
|
new ResolvedAttribute { CanonicalName = "Temperature", Value = "98.6", DataType = "Double" },
|
|
new ResolvedAttribute { CanonicalName = "Status", Value = "Running", DataType = "String" }
|
|
],
|
|
Scripts = scripts ?? [],
|
|
Alarms = alarms ?? []
|
|
};
|
|
|
|
return ActorOf(Props.Create(() => new InstanceActor(
|
|
instanceName,
|
|
JsonSerializer.Serialize(config),
|
|
_storage,
|
|
_compilationService,
|
|
_sharedScriptLibrary,
|
|
null, // no stream manager
|
|
_options,
|
|
NullLogger<InstanceActor>.Instance)));
|
|
}
|
|
|
|
[Fact]
|
|
public void InstanceActor_CreatesScriptActors_FromConfig()
|
|
{
|
|
var scripts = new[]
|
|
{
|
|
new ResolvedScript
|
|
{
|
|
CanonicalName = "GetValue",
|
|
Code = "42"
|
|
}
|
|
};
|
|
|
|
var actor = CreateInstanceWithScripts("Pump1", scripts);
|
|
|
|
// Verify script actor is reachable via CallScript
|
|
actor.Tell(new ScriptCallRequest("GetValue", null, 0, "corr-1"));
|
|
var result = ExpectMsg<ScriptCallResult>(TimeSpan.FromSeconds(10));
|
|
Assert.True(result.Success);
|
|
Assert.Equal(42, result.ReturnValue);
|
|
}
|
|
|
|
[Fact]
|
|
public void InstanceActor_ScriptCallRequest_UnknownScript_ReturnsError()
|
|
{
|
|
var actor = CreateInstanceWithScripts("Pump1");
|
|
|
|
actor.Tell(new ScriptCallRequest("NonExistent", null, 0, "corr-2"));
|
|
var result = ExpectMsg<ScriptCallResult>(TimeSpan.FromSeconds(5));
|
|
Assert.False(result.Success);
|
|
Assert.Contains("not found", result.ErrorMessage);
|
|
}
|
|
|
|
[Fact]
|
|
public void InstanceActor_WP24_StateMutationsSerializedThroughMailbox()
|
|
{
|
|
// WP-24: Instance Actor processes messages sequentially.
|
|
// Verify that rapid attribute changes don't corrupt state.
|
|
var actor = CreateInstanceWithScripts("Pump1");
|
|
|
|
// Send many rapid set commands
|
|
for (int i = 0; i < 50; i++)
|
|
{
|
|
actor.Tell(new SetStaticAttributeCommand(
|
|
$"corr-{i}", "Pump1", "Temperature", $"{i}", DateTimeOffset.UtcNow));
|
|
}
|
|
|
|
// Each static write replies with a SetStaticAttributeResponse; drain all
|
|
// 50 — the FIFO mailbox guarantees they are processed in order.
|
|
for (int i = 0; i < 50; i++)
|
|
{
|
|
ExpectMsg<SetStaticAttributeResponse>(TimeSpan.FromSeconds(5));
|
|
}
|
|
|
|
// The last value should be the final one
|
|
actor.Tell(new GetAttributeRequest(
|
|
"corr-final", "Pump1", "Temperature", DateTimeOffset.UtcNow));
|
|
var response = ExpectMsg<GetAttributeResponse>(TimeSpan.FromSeconds(5));
|
|
Assert.True(response.Found);
|
|
Assert.Equal("49", response.Value?.ToString());
|
|
}
|
|
|
|
[Fact]
|
|
public void InstanceActor_WP25_DebugViewSubscribe_ReturnsSnapshot()
|
|
{
|
|
var actor = CreateInstanceWithScripts("Pump1");
|
|
|
|
// Wait for initialization
|
|
Thread.Sleep(500);
|
|
|
|
actor.Tell(new SubscribeDebugViewRequest("Pump1", "debug-1"));
|
|
var snapshot = ExpectMsg<DebugViewSnapshot>(TimeSpan.FromSeconds(5));
|
|
|
|
Assert.Equal("Pump1", snapshot.InstanceUniqueName);
|
|
Assert.True(snapshot.AttributeValues.Count >= 2); // Temperature + Status
|
|
Assert.True(snapshot.SnapshotTimestamp > DateTimeOffset.MinValue);
|
|
}
|
|
|
|
[Fact]
|
|
public void InstanceActor_WP25_DebugViewSubscribe_NoLongerForwardsEventsViaClusterClient()
|
|
{
|
|
// Events now flow via gRPC (SiteStreamManager → StreamRelayActor → gRPC),
|
|
// not via ClusterClient. Subscribing returns a snapshot but ongoing events
|
|
// are NOT forwarded to the subscriber actor.
|
|
var actor = CreateInstanceWithScripts("Pump1");
|
|
|
|
// Subscribe to debug view — should still get snapshot
|
|
actor.Tell(new SubscribeDebugViewRequest("Pump1", "debug-2"));
|
|
ExpectMsg<DebugViewSnapshot>(TimeSpan.FromSeconds(5));
|
|
|
|
// Change an attribute
|
|
actor.Tell(new AttributeValueChanged(
|
|
"Pump1", "Temperature", "Temperature", "200", "Good", DateTimeOffset.UtcNow));
|
|
|
|
// Should NOT receive change notification (old ClusterClient path removed)
|
|
ExpectNoMsg(TimeSpan.FromSeconds(1));
|
|
}
|
|
|
|
[Fact]
|
|
public void InstanceActor_WP25_DebugViewUnsubscribe_StopsNotifications()
|
|
{
|
|
var actor = CreateInstanceWithScripts("Pump1");
|
|
|
|
// Subscribe
|
|
actor.Tell(new SubscribeDebugViewRequest("Pump1", "debug-3"));
|
|
ExpectMsg<DebugViewSnapshot>(TimeSpan.FromSeconds(5));
|
|
|
|
// Unsubscribe
|
|
actor.Tell(new UnsubscribeDebugViewRequest("Pump1", "debug-3"));
|
|
|
|
// Change attribute
|
|
actor.Tell(new AttributeValueChanged(
|
|
"Pump1", "Temperature", "Temperature", "300", "Good", DateTimeOffset.UtcNow));
|
|
|
|
// Should NOT receive change notification
|
|
ExpectNoMsg(TimeSpan.FromSeconds(1));
|
|
}
|
|
|
|
[Fact]
|
|
public void InstanceActor_CreatesAlarmActors_FromConfig()
|
|
{
|
|
var alarms = new[]
|
|
{
|
|
new ResolvedAlarm
|
|
{
|
|
CanonicalName = "HighTemp",
|
|
TriggerType = "RangeViolation",
|
|
TriggerConfiguration = "{\"attributeName\":\"Temperature\",\"min\":0,\"max\":100}",
|
|
PriorityLevel = 1
|
|
}
|
|
};
|
|
|
|
var actor = CreateInstanceWithScripts("Pump1", alarms: alarms);
|
|
|
|
// Wait for initialization
|
|
Thread.Sleep(500);
|
|
|
|
// Verify alarm actor was created by checking the debug snapshot includes the alarm
|
|
actor.Tell(new DebugSnapshotRequest("Pump1", "snap-alarm"));
|
|
var snapshot = ExpectMsg<DebugViewSnapshot>(TimeSpan.FromSeconds(5));
|
|
|
|
Assert.Single(snapshot.AlarmStates);
|
|
Assert.Equal("HighTemp", snapshot.AlarmStates[0].AlarmName);
|
|
}
|
|
}
|