Files
UA-.NETStandard/tests/Opc.Ua.Vision.Tests/MachineVisionRuntimeTests.Storage.cs
Marc Schier 2836da7ec8 Fix sessionless ResultTransfer ownership bypass and other issues. (#4834)
# Description

Correct ownership, endpoint validation, policy propagation, subscription
delivery, file handling and generated-model behavior across the stack,
tools and samples. Add focused regression tests and update the
corresponding guides and sample documentation.

### Previously included corrections

- Require an active, matching session to generate or access transient
ResultTransfer file handles. Cover both sessionless generation and reads
against a handle owned by an active session, using live session owners
in the Machinery tests.
- Keep physical filesystem operations within their configured mount and
apply the same confinement checks throughout file operations.
- Validate LDS semaphore paths consistently during server registration
and registration-store maintenance.

### Individual issues addressed

| # | Component | Issue and correction |
|---|---|---|
| 1 | AI jobs and transfers | Jobs and transfers did not consistently
retain their creator's identity and session. Apply ownership checks to
descendants, file handles, subscriptions and execution, and recheck
deployment permissions before dispatch. |
| 2 | Password-management client | Password-bearing calls could use
signing-only channels or follow a replacement connection. Capture an
encrypted transport generation and keep dispatch bound to it; preserve
ordinary non-password edits. |
| 3 | GDS and certificate-push clients | Connections omitted hostname
verification. Validate the original configured DNS name or IP address
before session activation. |
| 4 | Certificate push | Private-key uploads could use signing-only
calls. Require an encrypted, generation-bound dispatch while preserving
certificate-only updates and restoring the caller's identity afterward.
|
| 5 | MQTT client | The .NET Framework custom-trust callback could
discard TLS hostname, certificate-availability and usage errors. Retain
those checks while supporting configured issuer trust. |
| 6 | DTLS handshake | Certificate trust was not tied to the configured
peer address. Match the certificate's DNS or IP identity to the intended
endpoint. |
| 7 | Replicated address spaces | Serialized nodes lost per-node and
descendant permission metadata. Preserve role permissions, user-role
permissions and access restrictions, including updates to existing
replica nodes. |
| 8 | MCP PubSub imports | Imports without groups skipped checks on the
effective broker destination. Check the actual endpoint before resolving
credentials or activating the connection. |
| 9 | Reference-server accounts | A disabled username account could
authenticate again with its previous roles. Reject disabled accounts
during fresh authentication. |
| 10 | Console reference client | The sample disabled hostname checks
when connecting to an otherwise trusted peer. Retain the original
requested host and validate it before activation. |
| 11 | Capture replay and export | Replay and raw export could accept
non-capture files or include key material. Require complete PCAP/PCAPNG
containers and exclude key sidecars and decryption-secret blocks. |
| 12 | PubSub adapter fallback | The fallback client configuration
automatically accepted peer certificates. Apply normal trust and
hostname validation before activation or reads. |
| 13 | Kafka transport | Explicit TLS enablement was checked but did not
reach both final Confluent configurations. Use the same effective
protocol for validation, producers and consumers. |
| 14 | PubSub actions | Configuration rebuilds could widen a
connection-local action handler's scope. Preserve its original
connection association and reject mismatched dispatch. |
| 15 | PubSub readers | Reader-level message-protection settings were
accepted but ignored. Apply the effective group and reader settings
before accepting and publishing received values. |
| 16 | PubSub key groups | Different `SecurityGroupId` values could
share one connection-wide key context. Reject mixed groups on a
connection and require separate connections for distinct groups. |
| 17 | MCP session registrations | Session-name replacement or reuse
could retain another registration's credential bindings. Bind access to
the registration, caller and authenticated endpoint, and preserve that
identity when borrowing sessions. |
| 18 | Viewer HTTP host | Requests reached the viewer's OPC UA session
without consistent Host, Origin and port checks. Validate the request
destination and browser origin before using the session, while retaining
supported local MCP clients. |
| 19 | Vision file operations | File operations did not consistently
apply protected-file checks, and remote handles lacked caller and
connection context. Apply the shared checks and use host-side opaque
handles bound to caller, target and generation. |
| 20 | DTLS record integrity | The integrity-only MAC used a truncated
sequence number. Authenticate the full reconstructed sequence number and
epoch so records remain distinct across rollover. |
| 21 | AMB client sample | Connection setup omitted hostname validation.
Preserve the originally requested host and verify the peer identity
before activation. |
| 22 | Robot sample writes | The R2 `SpeedOverride` callback could
target R1 state. Bind each callback to its own robot and apply that
robot's write policy. |
| 23 | AI result properties | Optional result children inherited
remotely writable defaults. Preserve their declaration restrictions and
keep server-produced results read-only. |
| 24 | WSS peer-trust updates | Existing channels did not consistently
respond to peer-trust changes. Revalidate complete chains for forward
and reverse WSS channels and close only those that no longer validate. |
| 25 | GDS revocation checks | Caller-provided certificates could cause
issuer retrieval from embedded URLs. Resolve validation from configured
local trust, issuers and CRLs without fetching caller-provided
locations. |
| 26 | GMS observations | Published correction observations retained
writable model defaults. Reject remote writes while preserving
publisher-side updates. |
| 27 | Production-dataset monitoring | Subscription admission and
delivery could use the reporter's context instead of each subscriber's
permissions. Evaluate the subscriber at admission and on every delivery,
including cancellation and disposal paths. |
| 28 | WoT MQTT destinations | Hostless MQTT forms could imply a
localhost destination before loopback checks. Reject them before
credential lookup or connection attempts and validate the effective
endpoint. |
| 29 | XRegistry opaque reads | Content-ID reads and notifications did
not consistently honor encrypted-read requirements or the currently
accepted transport. Recheck the bound endpoint and reject transport
downgrades before rebinding, including when protected values are already
queued. |
| 30 | PubSub capture dissection | Key loading did not consistently
check the configured capture-root boundary. Check each actual key path
immediately before opening it. |
| 31 | PubSub publication queues | Imported WriterGroup queues were not
fully checked against publish-topic grants. Check the destination
actually used by each group. |
| 32 | MCP PubSub mutations | A caller-supplied mode could select
subscribe grants for an operation that publishes. Derive the required
direction and parent context from the actual mutation. |
| 33 | Generated write masks | Model import and generation could widen
an explicit `UserWriteMask`. Preserve it through import, copy, merge,
inheritance and dependency metadata. |
| 34 | Generated method flags | An explicit `UserExecutable=false` could
be lost. Preserve its specified state and resulting method behavior
through the same model paths. |
| 35 | Generated role identifiers | Unresolved role identifiers could
cause permission entries to be omitted. Preserve namespace-qualified
role IDs and report a diagnostic when a policy cannot be represented. |
| 36 | Aggregation-server defaults | The `localhost` default did not
reliably constrain listener binding. Use numeric loopback for the
default endpoint and command-line fallback. |
| 37 | Certificate-validation cache | Cached results could skip current
leaf or issuer validity checks. Recheck validity windows on cache hits
and keep hostname validation independent of cached chain trust. |
| 38 | Software-update folders | Revision names and marker contents
containing dot segments could select paths outside the software root.
Normalize and check every derived path against that root. |
| 39 | USDA arrays | Array elements were not encoded for their string or
asset context. Escape them as data and preserve valid composition and
round-trip behavior. |
| 40 | Adapter hot reload | A changed dataset definition with the same
name could keep old fields, subscriptions or cached values active.
Compare complete immutable definitions and retire old state before
replacement. |
| 41 | Adapter index ranges | Subscription publication could ignore
`IndexRange` and return the complete value. Preserve the requested
projection and its status and timestamps; reject unsupported range
providers. |
| 42 | PubSub metadata and targets | Metadata from one connection could
change another connection's target mapping. Scope metadata to its
connection and provenance, and resolve targets using actual typed field
IDs rather than field names or unintended positional fallback. |
| 43 | Scales product locks | Product locks were not applied across
every remote mutation path. Apply them consistently to value writes,
filling calculations and related recipe/controller operations while
retaining permitted updates. |
| 44 | Event permission caching | Cached event decisions could outlive
identity, role or node-policy changes. Revalidate the current
subscriber, event type and source permissions instead of reusing a stale
decision. |
| 45 | WoT Modbus destinations | Hostless Modbus forms could imply
localhost before loopback checks. Reject them before connecting and
validate the effective endpoint. |
| 46 | MCP local PKI tools | Peer-trust mutations did not consistently
require the caller's `CertificateAdministration` grant. Check the
capability before trust and untrust operations. |
| 47 | Robot-control HTTP bridge | The bridge lacked consistent Host,
Origin and port validation and caller-specific session grants. Apply
those checks before borrowing the OPC UA session and retain supported
local clients. |
| 48 | Subscription transfers | Transferred subscriptions could retain
cached notifications after the destination's permissions changed.
Require authorization continuity between source and destination
identities and roles, invalidate stale retransmission data when
continuity fails, and preserve retransmission behavior for compatible
transfers. |

### Compatibility notes

- Replication uses the versioned NSRP v1 policy envelope. Upgrade peers
together and re-encode or reseed legacy topology from the authoritative
model; missing permission metadata is not inferred from old records.
- Integrity-only DTLS peers must be upgraded together for the full
sequence/epoch MAC. AEAD behavior is unchanged.
- Distinct PubSub key groups require separate connections.
- Custom sessions that cannot capture the actual encrypted transport,
and subscription providers that cannot honor a requested range, are
rejected rather than silently changing the requested behavior.
- Subscription transfers retain retransmission data only when the
destination authorization remains compatible with the source snapshot;
otherwise the old sequence numbers and cached data are invalidated.

## Related Issues

No separate tracking issue is linked. The individual corrections are
listed above.

## Checklist

_Put an `x` in the boxes that apply. You can complete these step by step
after opening the PR._

- [ ] I have signed the
[CLA](https://opcfoundation.org/license/cla/ContributorLicenseAgreementv1.0.pdf)
and read the
[CONTRIBUTING](https://github.com/OPCFoundation/UA-.NETStandard/blob/master/CONTRIBUTING.md)
doc.
- [x] I have added tests that prove my fix is effective or that my
feature works and increased code coverage.
- [ ] I have added all necessary documentation.
- [x] I have verified that my changes do not introduce (new) build or
analyzer warnings.
- [ ] I ran **all** tests locally using the **UA.slnx** solution against
at least .net **framework** and .net **10**, and all passed.
- [ ] I fixed **all** failing and flaky tests in the CI pipelines and
**all** CodeQL warnings.
- [ ] I have addressed **all** PR feedback received.
2026-10-21 05:37:29 +02:00

653 lines
39 KiB
C#

/* ========================================================================
* Copyright (c) 2005-2026 The OPC Foundation, Inc. All rights reserved.
*
* OPC Foundation MIT License 1.00
*
* Permission is hereby granted, free of charge, to any person
* obtaining a copy of this software and associated documentation
* files (the "Software"), to deal in the Software without
* restriction, including without limitation the rights to use,
* copy, modify, merge, publish, distribute, sublicense, and/or sell
* copies of the Software, and to permit persons to whom the
* Software is furnished to do so, subject to the following
* conditions:
*
* The above copyright notice and this permission notice shall be
* included in all copies or substantial portions of the Software.
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
* OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
* NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
* HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
* WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
* FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
* OTHER DEALINGS IN THE SOFTWARE.
*
* The complete license agreement can be found here:
* http://opcfoundation.org/License/MIT/1.00/
* ======================================================================*/
using System;
using System.Collections.Generic;
using System.IO;
using System.Threading;
using System.Threading.Tasks;
using Moq;
using NUnit.Framework;
using Opc.Ua.Server;
using Opc.Ua.Server.FileSystem;
using Opc.Ua.Vision.MachineVision;
using Opc.Ua.Vision.Server;
namespace Opc.Ua.Vision.Tests
{
public sealed partial class MachineVisionRuntimeTests
{
[Test]
public async Task RecipeTransferCommitsContentThenPreparationEnablesReadyAsync()
{
await using var fixture = new VisionServerFixture();
await fixture.StartAsync(MachineVisionParts.Control).ConfigureAwait(false);
MachineVisionSystemHandle system = await fixture.CreateBuildContext()
.AddMachineVisionSystemAsync("Inspection", Provider().Object).ConfigureAwait(false);
await CallAsync(fixture, system.ControlState, system.ControlState.SelectModeAutomatic!, [])
.ConfigureAwait(false);
RecipeManagementState recipes = system.State.RecipeManagement!;
ArrayOf<Variant> added = await CallAsync(fixture, recipes, recipes.AddRecipe!,
[
Variant.FromStructure(new RecipeIdExternalDataType { Id = "recipe" }),
Variant.FromStructure(new ProductIdDataType { Id = "product" })
]).ConfigureAwait(false);
Assert.That(added[0].TryGetStructure<RecipeIdInternalDataType>(out RecipeIdInternalDataType? id), Is.True);
ArrayOf<Variant> transfer = await CallAsync(fixture, recipes.RecipeTransfer!,
recipes.RecipeTransfer!.GenerateFileForWrite!,
[Variant.FromStructure(new RecipeTransferOptions { InternalId = id! })]).ConfigureAwait(false);
Assert.That(transfer[0].TryGetValue(out NodeId fileId), Is.True);
Assert.That(fileId.NamespaceIndex, Is.EqualTo(system.State.NodeId.NamespaceIndex));
Assert.That(transfer[1].TryGetValue(out uint fileHandle), Is.True);
FileState file = fixture.Manager.FindPredefinedNode<FileState>(fileId)!;
await CallAsync(fixture, file, file.Write!,
[Variant.From(fileHandle), Variant.From(ByteString.From(new byte[] { 1, 2, 3 }))])
.ConfigureAwait(false);
await CallAsync(fixture, recipes.RecipeTransfer, recipes.RecipeTransfer.CloseAndCommit!,
[Variant.From(fileHandle)]).ConfigureAwait(false);
MachineVisionDocument? document = await system.Store.ReadAsync(MachineVisionDocumentKind.Recipe, id!.Id!)
.ConfigureAwait(false);
Assert.That(document!.Content.Span.ToArray(), Is.EqualTo(new byte[] { 1, 2, 3 }));
Assert.That(fixture.Manager.FindPredefinedNode<NodeState>(fileId), Is.Null);
await CallAsync(fixture, recipes, recipes.PrepareRecipe!,
[
Variant.FromStructure(new RecipeIdExternalDataType()),
Variant.FromStructure(id)
]).ConfigureAwait(false);
Assert.That(system.AutomaticState.CurrentState!.Value.Text, Is.EqualTo("Ready"));
await CallAsync(fixture, recipes, recipes.UnprepareRecipe!,
[
Variant.FromStructure(new RecipeIdExternalDataType()),
Variant.FromStructure(id)
]).ConfigureAwait(false);
Assert.That(system.AutomaticState.CurrentState.Value.Text, Is.EqualTo("Initialized"));
}
[Test]
public async Task ClosingAnUncommittedRecipeTransferDiscardsItsBytesAsync()
{
await using var fixture = new VisionServerFixture();
await fixture.StartAsync(MachineVisionParts.Control).ConfigureAwait(false);
MachineVisionSystemHandle system = await fixture.CreateBuildContext()
.AddMachineVisionSystemAsync("Inspection", Provider().Object).ConfigureAwait(false);
RecipeManagementState recipes = system.State.RecipeManagement!;
ArrayOf<Variant> added = await CallAsync(fixture, recipes, recipes.AddRecipe!,
[
Variant.FromStructure(new RecipeIdExternalDataType { Id = "recipe" }),
Variant.FromStructure(new ProductIdDataType())
]).ConfigureAwait(false);
added[0].TryGetStructure<RecipeIdInternalDataType>(out RecipeIdInternalDataType? id);
ArrayOf<Variant> transfer = await CallAsync(fixture, recipes.RecipeTransfer!,
recipes.RecipeTransfer!.GenerateFileForWrite!,
[Variant.FromStructure(new RecipeTransferOptions { InternalId = id! })]).ConfigureAwait(false);
transfer[0].TryGetValue(out NodeId fileId);
transfer[1].TryGetValue(out uint fileHandle);
FileState file = fixture.Manager.FindPredefinedNode<FileState>(fileId)!;
await CallAsync(fixture, file, file.Write!,
[Variant.From(fileHandle), Variant.From(ByteString.From(new byte[] { 1, 2 }))]).ConfigureAwait(false);
await CallAsync(fixture, file, file.Close!, [Variant.From(fileHandle)]).ConfigureAwait(false);
MachineVisionDocument? document = await system.Store.ReadAsync(MachineVisionDocumentKind.Recipe, id!.Id!)
.ConfigureAwait(false);
Assert.That(document!.Content.IsNull, Is.True);
Assert.That(fixture.Manager.FindPredefinedNode<NodeState>(fileId), Is.Null);
}
[Test]
public async Task ResultVariablesAndEventsPreserveOptionalMetadataPresenceAsync()
{
await using var fixture = new VisionServerFixture();
await fixture.StartAsync(MachineVisionParts.Control).ConfigureAwait(false);
MachineVisionSystemHandle system = await fixture.CreateBuildContext()
.AddMachineVisionSystemAsync("Inspection", Provider().Object).ConfigureAwait(false);
ResultReadyEventState? notification = null;
system.State.EventReported += (_, _, value) =>
{
if (value is ResultReadyEventState ready)
{
notification = ready;
}
};
await system.PublishResultAsync(new ResultDataType
{
ResultId = new ResultIdDataType { Id = "transferable" },
ResultState = 1,
ResultContent = [Variant.From(42)],
EncodingMask = (uint)ResultDataTypeFields.ResultContent
}, ByteString.From(new byte[] { 1, 2, 3 })).ConfigureAwait(false);
Assert.That(notification, Is.Not.Null);
Assert.That(notification!.ProcessingTimes, Is.Null, "Missing processing times are not fabricated.");
Assert.That(notification.ResultContent!.Value[0].TryGetValue(out int content), Is.True);
Assert.That(content, Is.EqualTo(42));
var children = new List<BaseInstanceState>();
system.State.ResultManagement!.Results!.GetChildren(fixture.Manager.SystemContext, children);
Assert.That(children, Has.Count.EqualTo(1));
var result = (ResultVariableState)children[0];
Assert.That(result.HasTransferableDataOnFile, Is.Not.Null);
Assert.That(result.HasTransferableDataOnFile!.Value, Is.True);
await system.PublishResultAsync(new ResultDataType
{
ResultId = new ResultIdDataType { Id = "transferable" },
ResultState = 2
}).ConfigureAwait(false);
Assert.That(result.HasTransferableDataOnFile.Value, Is.False);
Assert.That(result.ResultContent!.StatusCode, Is.EqualTo(StatusCodes.BadWaitingForInitialData));
Assert.That(result.ResultContent.WrappedValue.IsNull, Is.True);
}
[Test]
public async Task ConfigurationPersistenceFailureDoesNotApplyHardwareStateAsync()
{
await using var fixture = new VisionServerFixture();
await fixture.StartAsync(MachineVisionParts.Control).ConfigureAwait(false);
var backing = new InMemoryMachineVisionStore();
await backing.WriteAsync(new MachineVisionDocument(MachineVisionDocumentKind.Configuration, "configuration",
Variant.FromStructure(new ConfigurationDataType
{
InternalId = new ConfigurationIdDataType { Id = "configuration" }
}), ByteString.From(new byte[] { 1 }))).ConfigureAwait(false);
var store = new Mock<IMachineVisionStore>(MockBehavior.Strict);
store.Setup(instance => instance.ReadAsync(It.IsAny<MachineVisionDocumentKind>(),
It.IsAny<string>(), It.IsAny<CancellationToken>()))
.Returns((MachineVisionDocumentKind kind, string id, CancellationToken ct) =>
backing.ReadAsync(kind, id, ct));
store.Setup(instance => instance.ListAsync(
It.IsAny<MachineVisionDocumentKind>(), It.IsAny<CancellationToken>()))
.Returns((MachineVisionDocumentKind kind, CancellationToken ct) => backing.ListAsync(kind, ct));
store.Setup(instance => instance.WriteAsync(
It.IsAny<MachineVisionDocument>(), It.IsAny<CancellationToken>()))
.Throws(new IOException("Persistent state is unavailable."));
Mock<IMachineVisionOperationsProvider> provider = Provider();
MachineVisionSystemHandle system = await fixture.CreateBuildContext().AddMachineVisionSystemAsync(
"Inspection", provider.Object,
new MachineVisionSystemOptions { Features = MachineVisionFeatures.Configurations }, store.Object)
.ConfigureAwait(false);
ConfigurationManagementState management = system.State.ConfigurationManagement!;
ServiceResult result = await management.ActivateConfiguration!.CallAsync(
SessionContext(fixture, new NodeId(1u)), management.NodeId,
[Variant.FromStructure(new ConfigurationIdDataType { Id = "configuration" })], [], [])
.ConfigureAwait(false);
Assert.That(result.StatusCode, Is.EqualTo(StatusCodes.BadInternalError));
provider.Verify(instance => instance.ActivateConfigurationAsync(
It.IsAny<MachineVisionDocument>(), It.IsAny<CancellationToken>()), Times.Never);
Assert.That(await backing.ReadAsync(MachineVisionDocumentKind.Runtime, "ActiveConfiguration")
.ConfigureAwait(false), Is.Null);
}
[TestCase(false)]
[TestCase(true)]
public async Task RefusedConfigurationRestoresPreviousDurableIntentAsync(bool previouslyActive)
{
await using var fixture = new VisionServerFixture();
await fixture.StartAsync(MachineVisionParts.Control).ConfigureAwait(false);
var store = new InMemoryMachineVisionStore();
foreach (string id in new[] { "previous", "refused" })
{
await store.WriteAsync(new MachineVisionDocument(MachineVisionDocumentKind.Configuration, id,
Variant.FromStructure(new ConfigurationDataType
{
InternalId = new ConfigurationIdDataType { Id = id }
}), ByteString.From(new byte[] { 1 }))).ConfigureAwait(false);
}
Mock<IMachineVisionOperationsProvider> provider = Provider();
provider.Setup(instance => instance.ActivateConfigurationAsync(
It.Is<MachineVisionDocument>(document => document.Id == "refused"), It.IsAny<CancellationToken>()))
.Returns(new ValueTask<MachineVisionOperationResult>(
new MachineVisionOperationResult(ServiceResult.Good, -9)));
MachineVisionSystemHandle system = await fixture.CreateBuildContext().AddMachineVisionSystemAsync(
"Inspection", provider.Object,
new MachineVisionSystemOptions { Features = MachineVisionFeatures.Configurations }, store)
.ConfigureAwait(false);
ConfigurationManagementState management = system.State.ConfigurationManagement!;
if (previouslyActive)
{
await CallAsync(fixture, management, management.ActivateConfiguration!,
[Variant.FromStructure(new ConfigurationIdDataType { Id = "previous" })]).ConfigureAwait(false);
}
ArrayOf<Variant> output = await CallAsync(fixture, management, management.ActivateConfiguration!,
[Variant.FromStructure(new ConfigurationIdDataType { Id = "refused" })]).ConfigureAwait(false);
Assert.That(output[0].TryGetValue(out int error), Is.True);
Assert.That(error, Is.EqualTo(-9));
MachineVisionDocument? intent = await store.ReadAsync(
MachineVisionDocumentKind.Runtime, "ActiveConfiguration")
.ConfigureAwait(false);
if (previouslyActive)
{
Assert.That(intent, Is.Not.Null);
Assert.That(intent!.Metadata.TryGetValue(out string activeId), Is.True);
Assert.That(activeId, Is.EqualTo("previous"));
Assert.That(management.ActiveConfiguration!.Value.InternalId.Id, Is.EqualTo("previous"));
}
else
{
Assert.That(intent, Is.Null);
}
}
[Test]
public async Task SharedFileBindingIsolatesReadersAndCommitsOneWriterAsync()
{
await using var fixture = new VisionServerFixture();
await fixture.StartAsync().ConfigureAwait(false);
var file = new FileState(null);
file.Create(fixture.Manager.SystemContext, new NodeId("file", fixture.Manager.NamespaceIndex),
new QualifiedName("File", fixture.Manager.NamespaceIndex), new LocalizedText("File"), true);
ByteString committed = default;
await using IFileTypeBinding binding = new FileTypeBinder().Bind(
file, fixture.Manager.SystemContext, ByteString.From(new byte[] { 10, 20, 30 }),
(content, _) =>
{
committed = content;
return default;
},
commitOnClose: true);
var owner = new NodeId(1u);
var other = new NodeId(2u);
uint first = await binding.OpenAsync(owner, 1).ConfigureAwait(false);
uint second = await binding.OpenAsync(other, 1).ConfigureAwait(false);
SessionSystemContext ownerContext = SessionContext(fixture, owner);
SessionSystemContext otherContext = SessionContext(fixture, other);
ArrayOf<Variant> a = await InvokeAsync(ownerContext, file, file.Read!,
[Variant.From(first), Variant.From(1)]).ConfigureAwait(false);
ArrayOf<Variant> b = await InvokeAsync(otherContext, file, file.Read!,
[Variant.From(second), Variant.From(1)]).ConfigureAwait(false);
a[0].TryGetValue(out ByteString aBytes);
b[0].TryGetValue(out ByteString bBytes);
Assert.That(aBytes.Span.ToArray(), Is.EqualTo("\n"u8.ToArray()));
Assert.That(bBytes.Span.ToArray(), Is.EqualTo("\n"u8.ToArray()));
await Assert.ThrowsAsync<ServiceResultException>(async () =>
await binding.CloseAsync(other, first).ConfigureAwait(false));
await Assert.ThrowsAsync<ServiceResultException>(async () =>
await binding.OpenAsync(owner, 6).ConfigureAwait(false));
await binding.CloseAsync(owner, first).ConfigureAwait(false);
await binding.CloseAsync(other, second).ConfigureAwait(false);
uint writer = await binding.OpenAsync(owner, 6).ConfigureAwait(false);
await InvokeAsync(ownerContext, file, file.Write!,
[Variant.From(writer), Variant.From(ByteString.From("*"u8.ToArray()))]).ConfigureAwait(false);
await binding.CloseAsync(owner, writer).ConfigureAwait(false);
Assert.That(committed.Span.ToArray(), Is.EqualTo("*"u8.ToArray()));
Assert.That(file.OpenCount!.Value, Is.Zero);
Assert.That(file.Size!.Value, Is.EqualTo(1));
}
/// <summary>
/// Session closure releases only its handles and never commits an abandoned writer.
/// </summary>
[TestCase(false)]
[TestCase(true)]
public async Task SharedFileBindingReleasesClosedSessionHandlesAsync(bool writer)
{
await using var fixture = new VisionServerFixture();
await fixture.StartAsync().ConfigureAwait(false);
var sessions = new Mock<ISessionManager>();
ServerSystemContext context = CreateFileBindingContext(fixture, sessions.Object);
var file = new FileState(null);
file.Create(context, new NodeId("session-file", fixture.Manager.NamespaceIndex),
new QualifiedName("SessionFile", fixture.Manager.NamespaceIndex),
new LocalizedText("SessionFile"), true);
bool committed = false;
await using IFileTypeBinding binding = new FileTypeBinder().Bind(
file, context, ByteString.From(new byte[] { 10, 20 }),
(_, _) =>
{
committed = true;
return default;
},
commitOnClose: true);
NodeId owner = new(1u);
NodeId other = new(2u);
uint first = await binding.OpenAsync(owner, writer ? (byte)6 : (byte)1).ConfigureAwait(false);
uint second = 0;
if (writer)
{
await InvokeAsync(SessionContext(fixture, owner), file, file.Write!,
[Variant.From(first), Variant.From(ByteString.From(new byte[] { 99 }))]).ConfigureAwait(false);
}
else
{
second = await binding.OpenAsync(other, 1).ConfigureAwait(false);
}
var closedSession = new Mock<Opc.Ua.Server.ISession>();
closedSession.SetupGet(session => session.Id).Returns(owner);
sessions.Raise(manager => manager.SessionClosing += null!,
closedSession.Object, SessionEventReason.Closing);
Assert.That(file.OpenCount!.Value, Is.EqualTo(writer ? 0 : 1));
Assert.That(committed, Is.False, "Session cleanup must discard an uncommitted write.");
ServiceResult stale = await file.Read!.CallAsync(SessionContext(fixture, owner), file.NodeId,
[Variant.From(first), Variant.From(1)], [], []).ConfigureAwait(false);
Assert.That(stale.StatusCode, Is.EqualTo(StatusCodes.BadInvalidArgument));
if (writer)
{
second = await binding.OpenAsync(other, 1).ConfigureAwait(false);
}
ArrayOf<Variant> response = await InvokeAsync(SessionContext(fixture, other), file, file.Read!,
[Variant.From(second), Variant.From(2)]).ConfigureAwait(false);
Assert.That(response[0].TryGetValue(out ByteString bytes), Is.True);
Assert.That(bytes.ToArray(), Is.EqualTo(new byte[] { 10, 20 }));
await binding.CloseAsync(other, second).ConfigureAwait(false);
await binding.DisposeAsync().ConfigureAwait(false);
sessions.VerifyRemove(manager => manager.SessionClosing -= It.IsAny<SessionEventHandler>(), Times.Once);
}
/// <summary>
/// Removing a recipe releases its direct file binding rather than retaining every retired payload and stream.
/// </summary>
[Test]
public async Task RemovingRecipesReleasesTheirPublishedFileBindingsAsync()
{
await using var fixture = new VisionServerFixture();
await fixture.StartAsync(MachineVisionParts.Control).ConfigureAwait(false);
var store = new InMemoryMachineVisionStore(1, 4);
MachineVisionSystemHandle system = await fixture.CreateBuildContext().AddMachineVisionSystemAsync(
"Removal", Provider().Object, store: store).ConfigureAwait(false);
RecipeManagementState recipes = system.State.RecipeManagement!;
for (int iteration = 0; iteration < 3; iteration++)
{
var externalId = new RecipeIdExternalDataType { Id = "replaceable" };
ArrayOf<Variant> added = await CallAsync(fixture, recipes, recipes.AddRecipe!,
[Variant.FromStructure(externalId), Variant.FromStructure(new ProductIdDataType())])
.ConfigureAwait(false);
Assert.That(added[1].TryGetValue(out NodeId recipeId), Is.True);
RecipeState recipe = fixture.Manager.FindPredefinedNode<RecipeState>(recipeId)!;
FileState file = recipe.Handle!;
ArrayOf<Variant> opened = await CallAsync(fixture, file, file.Open!, [Variant.From((byte)6)])
.ConfigureAwait(false);
Assert.That(opened[0].TryGetValue(out uint writer), Is.True);
await CallAsync(fixture, file, file.Write!,
[Variant.From(writer), Variant.From(ByteString.From(new byte[] { 1, 2 }))]).ConfigureAwait(false);
await CallAsync(fixture, file, file.Close!, [Variant.From(writer)]).ConfigureAwait(false);
opened = await CallAsync(fixture, file, file.Open!, [Variant.From((byte)1)]).ConfigureAwait(false);
Assert.That(opened[0].TryGetValue(out uint reader), Is.True);
PropertyState<ushort> openCount = file.OpenCount!;
MethodState read = file.Read!;
Assert.That(openCount.Value, Is.EqualTo(1));
await CallAsync(fixture, recipes, recipes.RemoveRecipe!, [Variant.FromStructure(externalId)])
.ConfigureAwait(false);
Assert.That(await store.ListAsync(MachineVisionDocumentKind.Recipe).ConfigureAwait(false), Is.Empty);
Assert.That(fixture.Manager.FindPredefinedNode<NodeState>(recipeId), Is.Null);
Assert.That(openCount.Value, Is.Zero, "Retired recipe files must release their open streams.");
ServiceResult retired = await read.CallAsync(
SessionContext(fixture, new NodeId(1u)), file.NodeId,
[Variant.From(reader), Variant.From(1)], [], []).ConfigureAwait(false);
Assert.That(retired.StatusCode, Is.EqualTo(StatusCodes.BadInvalidState));
}
}
/// <summary>
/// Removing a recipe cancels a commit waiting for the runtime gate without resurrecting the stored recipe.
/// </summary>
[Test]
public async Task RemovingRecipeCancelsPendingFileCommitWithoutDeadlockingAsync()
{
await using var fixture = new VisionServerFixture();
await fixture.StartAsync(MachineVisionParts.Control).ConfigureAwait(false);
using var deadline = new CancellationTokenSource(TimeSpan.FromSeconds(15));
CancellationToken ct = deadline.Token;
var backing = new InMemoryMachineVisionStore();
var deleting = new TaskCompletionSource<bool>(TaskCreationOptions.RunContinuationsAsynchronously);
var releaseDelete = new TaskCompletionSource<bool>(TaskCreationOptions.RunContinuationsAsynchronously);
var store = new Mock<IMachineVisionStore>(MockBehavior.Strict);
store.Setup(instance => instance.ReadAsync(It.IsAny<MachineVisionDocumentKind>(),
It.IsAny<string>(), It.IsAny<CancellationToken>()))
.Returns((MachineVisionDocumentKind kind, string id, CancellationToken token) =>
backing.ReadAsync(kind, id, token));
store.Setup(instance => instance.ListAsync(
It.IsAny<MachineVisionDocumentKind>(), It.IsAny<CancellationToken>()))
.Returns((MachineVisionDocumentKind kind, CancellationToken token) => backing.ListAsync(kind, token));
store.Setup(instance => instance.WriteAsync(
It.IsAny<MachineVisionDocument>(), It.IsAny<CancellationToken>()))
.Returns((MachineVisionDocument document, CancellationToken token) =>
backing.WriteAsync(document, token));
store.Setup(instance => instance.DeleteAsync(It.IsAny<MachineVisionDocumentKind>(),
It.IsAny<string>(), It.IsAny<CancellationToken>()))
.Returns(async (MachineVisionDocumentKind kind, string id, CancellationToken token) =>
{
deleting.TrySetResult(true);
await releaseDelete.Task.WaitAsync(token).ConfigureAwait(false);
return await backing.DeleteAsync(kind, id, token).ConfigureAwait(false);
});
MachineVisionSystemHandle system = await fixture.CreateBuildContext().AddMachineVisionSystemAsync(
"Removal", Provider().Object, store: store.Object, cancellationToken: ct).ConfigureAwait(false);
RecipeManagementState recipes = system.State.RecipeManagement!;
var external = new RecipeIdExternalDataType { Id = "pending" };
ArrayOf<Variant> added = await CallAsync(fixture, recipes, recipes.AddRecipe!,
[Variant.FromStructure(external), Variant.FromStructure(new ProductIdDataType())])
.ConfigureAwait(false);
Assert.That(added[1].TryGetValue(out NodeId recipeId), Is.True);
FileState file = fixture.Manager.FindPredefinedNode<RecipeState>(recipeId)!.Handle!;
PropertyState<ushort> openCount = file.OpenCount!;
ArrayOf<Variant> opened = await CallAsync(fixture, file, file.Open!, [Variant.From((byte)6)])
.ConfigureAwait(false);
Assert.That(opened[0].TryGetValue(out uint writer), Is.True);
await CallAsync(fixture, file, file.Write!,
[Variant.From(writer), Variant.From(ByteString.From(new byte[] { 9 }))]).ConfigureAwait(false);
Task<ServiceResult> removal = recipes.RemoveRecipe!.CallAsync(
SessionContext(fixture, new NodeId(1u)), recipes.NodeId,
[Variant.FromStructure(external)], [], [], ct).AsTask();
try
{
await deleting.Task.WaitAsync(ct).ConfigureAwait(false);
Task<ServiceResult> commit = file.Close!.CallAsync(
SessionContext(fixture, new NodeId(1u)), file.NodeId, [Variant.From(writer)], [], [], ct).AsTask();
Assert.That(commit.IsCompleted, Is.False, "The commit must be queued behind recipe removal.");
releaseDelete.TrySetResult(true);
ServiceResult[] results = await Task.WhenAll(removal, commit).WaitAsync(ct).ConfigureAwait(false);
Assert.That(results[0].StatusCode, Is.EqualTo(StatusCodes.Good));
Assert.That(results[1].StatusCode, Is.EqualTo(StatusCodes.BadInvalidState));
Assert.That(openCount.Value, Is.Zero);
Assert.That(await backing.ListAsync(MachineVisionDocumentKind.Recipe, ct).ConfigureAwait(false),
Is.Empty);
store.Verify(instance => instance.WriteAsync(
It.IsAny<MachineVisionDocument>(), It.IsAny<CancellationToken>()), Times.Once);
}
finally
{
releaseDelete.TrySetResult(true);
await removal.WaitAsync(ct).ConfigureAwait(false);
}
}
[Test]
public async Task FileCommitCanRefreshItsPublishedContentWithoutDeadlockingAsync()
{
await using var fixture = new VisionServerFixture();
await fixture.StartAsync().ConfigureAwait(false);
var file = new FileState(null);
file.Create(fixture.Manager.SystemContext, new NodeId("file", fixture.Manager.NamespaceIndex),
new QualifiedName("File", fixture.Manager.NamespaceIndex), new LocalizedText("File"), true);
IFileTypeBinding? current = null;
await using IFileTypeBinding binding = new FileTypeBinder().Bind(
file, fixture.Manager.SystemContext, ByteString.From(new byte[] { 1 }),
async (content, ct) => await current!.UpdateContentAsync(content, ct).ConfigureAwait(false));
current = binding;
NodeId owner = new(1u);
uint writer = await binding.OpenAsync(owner, 6).ConfigureAwait(false);
await InvokeAsync(SessionContext(fixture, owner), file, file.Write!,
[Variant.From(writer), Variant.From(ByteString.From(new byte[] { 4, 2 }))]).ConfigureAwait(false);
using var deadline = new CancellationTokenSource(TimeSpan.FromSeconds(3));
await binding.CommitAsync(owner, writer, deadline.Token).ConfigureAwait(false);
Assert.That(file.OpenCount!.Value, Is.Zero);
Assert.That(file.Size!.Value, Is.EqualTo(2));
uint reader = await binding.OpenAsync(owner, 1).ConfigureAwait(false);
ArrayOf<Variant> read = await InvokeAsync(SessionContext(fixture, owner), file, file.Read!,
[Variant.From(reader), Variant.From(2)]).ConfigureAwait(false);
Assert.That(read[0].TryGetValue(out ByteString bytes), Is.True);
Assert.That(bytes.Span.ToArray(), Is.EqualTo(new byte[] { 4, 2 }));
}
[TestCase(-1)]
[TestCase(0)]
public async Task BoundFileRejectsNonPositiveReadLengthsAsync(int length)
{
await using var fixture = new VisionServerFixture();
await fixture.StartAsync().ConfigureAwait(false);
var file = new FileState(null);
file.Create(fixture.Manager.SystemContext, new NodeId("file", fixture.Manager.NamespaceIndex),
new QualifiedName("File", fixture.Manager.NamespaceIndex), new LocalizedText("File"), true);
await using IFileTypeBinding binding = new FileTypeBinder().Bind(
file, fixture.Manager.SystemContext, ByteString.From(new byte[] { 1, 2, 3 }));
NodeId owner = new(1u);
uint reader = await binding.OpenAsync(owner, 1).ConfigureAwait(false);
ServiceResult result = await file.Read!.CallAsync(
SessionContext(fixture, owner), file.NodeId, [Variant.From(reader), Variant.From(length)], [], [])
.ConfigureAwait(false);
Assert.That(result.StatusCode, Is.EqualTo(StatusCodes.BadInvalidArgument));
}
[Test]
public async Task BoundFileClampsPositionsBeyondEndOfFileAsync()
{
await using var fixture = new VisionServerFixture();
await fixture.StartAsync().ConfigureAwait(false);
var file = new FileState(null);
file.Create(fixture.Manager.SystemContext, new NodeId("file", fixture.Manager.NamespaceIndex),
new QualifiedName("File", fixture.Manager.NamespaceIndex), new LocalizedText("File"), true);
await using IFileTypeBinding binding = new FileTypeBinder().Bind(
file, fixture.Manager.SystemContext, ByteString.From(new byte[] { 1, 2, 3 }));
NodeId owner = new(1u);
uint reader = await binding.OpenAsync(owner, 1).ConfigureAwait(false);
SessionSystemContext context = SessionContext(fixture, owner);
await InvokeAsync(context, file, file.SetPosition!, [Variant.From(reader), Variant.From(ulong.MaxValue)])
.ConfigureAwait(false);
ArrayOf<Variant> position = await InvokeAsync(context, file, file.GetPosition!, [Variant.From(reader)])
.ConfigureAwait(false);
Assert.That(position[0].TryGetValue(out ulong value), Is.True);
Assert.That(value, Is.EqualTo(3));
ArrayOf<Variant> end = await InvokeAsync(context, file, file.Read!, [Variant.From(reader), Variant.From(1)])
.ConfigureAwait(false);
Assert.That(end[0].TryGetValue(out ByteString bytes), Is.True);
Assert.That(bytes.IsEmpty, Is.True);
}
[Test]
public async Task FileSystemStoreSurvivesRestartAndRecoversAnInterruptedReplacementAsync()
{
await using var fixture = new VisionServerFixture();
await fixture.StartAsync(MachineVisionParts.All).ConfigureAwait(false);
// Leave room for legacy pending-file names on .NET Framework.
string directory = Path.Combine(AppContext.BaseDirectory, "v-" + Guid.NewGuid().ToString("N")[..8]);
var provider = new PhysicalFileSystemProvider(directory);
try
{
await using (var first = new FileSystemMachineVisionStore(
provider, fixture.Manager.SystemContext.AsMessageContext()))
{
await first.WriteAsync(new MachineVisionDocument(
MachineVisionDocumentKind.Result, "../logical-id",
Variant.FromStructure(new ResultIdDataType { Id = "persisted" }),
ByteString.From(new byte[] { 3, 2, 1 }))).ConfigureAwait(false);
}
string stored = Directory.GetFiles(directory, "*.bin", SearchOption.AllDirectories)[0];
string pending = stored + ".pending";
string oldPending = stored + ".pending-" + Guid.NewGuid().ToString("N");
File.WriteAllBytes(pending, [0x42]);
File.WriteAllBytes(oldPending, [0x43]);
File.Move(stored, stored + ".previous");
await using var second = new FileSystemMachineVisionStore(
provider, fixture.Manager.SystemContext.AsMessageContext());
MachineVisionDocument? recovered = await second.ReadAsync(
MachineVisionDocumentKind.Result, "../logical-id")
.ConfigureAwait(false);
Assert.That(recovered, Is.Not.Null);
Assert.That(recovered!.Metadata.TryGetStructure<ResultIdDataType>(out ResultIdDataType? id), Is.True);
Assert.That(id!.Id, Is.EqualTo("persisted"));
Assert.That(recovered.Content.Span.ToArray(), Is.EqualTo(new byte[] { 3, 2, 1 }));
Assert.That(File.Exists(stored), Is.True);
Assert.That(File.Exists(stored + ".previous"), Is.False);
Assert.That(File.Exists(pending), Is.False, "An interrupted pending write must not accumulate.");
Assert.That(File.Exists(oldPending), Is.False,
"Recover files created by the earlier unique-name format.");
Assert.That((await second.ListAsync(MachineVisionDocumentKind.Result).ConfigureAwait(false)).Count,
Is.EqualTo(1));
string interruptedFirstWrite = Path.Combine(Path.GetDirectoryName(stored)!,
new string('0', 64) + ".bin.pending");
File.WriteAllBytes(interruptedFirstWrite, [0x44]);
Assert.That((await second.ListAsync(MachineVisionDocumentKind.Result).ConfigureAwait(false)).Count,
Is.EqualTo(1), "An uncommitted first write is not a document.");
Assert.That(File.Exists(interruptedFirstWrite), Is.False);
Assert.That(await second.DeleteAsync(MachineVisionDocumentKind.Result, "../logical-id")
.ConfigureAwait(false), Is.True);
Assert.That(await second.ReadAsync(MachineVisionDocumentKind.Result, "../logical-id")
.ConfigureAwait(false), Is.Null);
}
finally
{
Directory.Delete(directory, true);
}
}
/// <summary>
/// Preserves the real model context while making session-close notifications controllable.
/// </summary>
private static ServerSystemContext CreateFileBindingContext(
VisionServerFixture fixture, ISessionManager sessions)
{
IServerInternal original = fixture.Server.CurrentInstance;
var server = new Mock<IServerInternal>();
server.SetupGet(value => value.Telemetry).Returns(original.Telemetry);
server.SetupGet(value => value.NamespaceUris).Returns(original.NamespaceUris);
server.SetupGet(value => value.ServerUris).Returns(original.ServerUris);
server.SetupGet(value => value.TypeTree).Returns(original.TypeTree);
server.SetupGet(value => value.Factory).Returns(original.Factory);
server.SetupGet(value => value.MessageContext).Returns(original.MessageContext);
server.SetupGet(value => value.SessionManager).Returns(sessions);
return new ServerSystemContext(server.Object) { NodeIdFactory = fixture.Manager.NodeIdFactory };
}
private static SessionSystemContext SessionContext(VisionServerFixture fixture, NodeId session)
{
return new SessionSystemContext(fixture.Manager.SystemContext.Telemetry)
{
SessionId = session,
NamespaceUris = fixture.Manager.SystemContext.NamespaceUris,
EncodeableFactory = fixture.Manager.SystemContext.EncodeableFactory,
TypeTable = fixture.Manager.SystemContext.TypeTable
};
}
private static async ValueTask<ArrayOf<Variant>> InvokeAsync(
ISystemContext context, NodeState parent, MethodState method, ArrayOf<Variant> arguments)
{
var errors = new List<ServiceResult>();
var output = new List<Variant>();
ServiceResult result = await method.CallAsync(context, parent.NodeId, arguments, errors, output)
.ConfigureAwait(false);
Assert.That(ServiceResult.IsGood(result), Is.True, result.ToString());
foreach (ServiceResult error in errors)
{
Assert.That(ServiceResult.IsGood(error), Is.True, error.ToString());
}
return output.ToArrayOf();
}
}
}