mirror of
https://github.com/OPCFoundation/UA-.NETStandard.git
synced 2026-10-21 00:56:28 +08:00
1bb4bdca1d
# Description Implement the released [OPC 40100-1](https://reference.opcfoundation.org/specs/OPC-40100-1) and [OPC 40100-2](https://reference.opcfoundation.org/specs/OPC-40100-2) specifications with generated models, provider-backed runtimes, typed clients, MCP workflows, and DI/Machinery integration. - **Part 1 stays in `Opc.Ua.Vision`** and its client/server packages alongside the distinct Perception model. Provide control/acquisition state machines, jobs and synchronized steps, recipe/product/configuration/result management, file transfers, diagnostics, events, and conditions. - **Part 2 is in `Opc.Ua.MachineVision`, `.Client`, and `.Server`.** Provide asset categories, identification, image-handling aspects, health/maintenance data, topology, and transactional updates. The base Vision assemblies do not reference these optional packages; the Part 2 wire model does not depend on Part 1. - Preserve the release-pinned model bytes, wire namespaces and NodeIds. Perception types are in `Opc.Ua.Vision.Perception`; the AMCM generated namespace remains `Opc.Ua.Vision.MachineVision.AMCM`, and asset client/server extension namespaces remain `Opc.Ua.Vision.Client` and `Opc.Ua.Vision.Server`. - Use `AddOpcUaMachineVision` and `AddOpcUaMachineVisionAMCM` for model loading and factory registration. Each has node-collection, encodeable-factory-builder and node-state-factory-builder overloads. These short entry points delegate to the generated implementations; existing namespace-derived generated names remain available for compatibility. - Support multiple independent named systems/assets under one namespace-owning manager. Reuse injectable providers, file binding, bounded memory stores, optional transactional `IFileSystemProvider` persistence, state machines, telemetry and conformance infrastructure. - Keep one `docs/Vision.md` guide, titled `Machine Vision (OPC 40100-1 and OPC 40100-2)`, and one `Opc.Ua.Mcp.Vision` package. Include deterministic server/client samples. - Expose 17 Machine Vision MCP tools, including capabilities/status, bounded relationships and observation, complete small-file uploads/downloads, management, explicit control, conditions, and chunked transfers. Catalogue counts: 39 Vision-specific, 43 with connection tools, 81 Vision+Robotics, 155 Full. Every Machine Vision wire parameter has a schema description. **Compatibility and limits:** `AddVision()` remains Perception-only unless released parts are selected. Standalone Part 2 uses `AddVision().AddMachineVision()`; Machinery composition uses `AddVisionToMachinery()`. Existing tool defaults and command signatures are preserved. Observation owns and cleans up its subscription: 5 seconds/50 notifications by default, capped at 60 seconds/100 notifications. Complete-file tools are capped at 1 MiB; larger files use chunked transfers. Domain Error values remain distinct from UA service failures. **Versioning:** the seven Vision/MachineVision/MCP packages and their Debug variants follow the repository root version instead of being forced to preview. Vision.OpenUsd and its descendants remain preview-only. Development builds still inherit the root prerelease version. No release version, tag, release branch, publication policy, or feed promotion is changed. **Review actions:** all fourteen review threads have action/evidence replies and are resolved. Changes include the unified guide, Perception namespace, separate Part 2 packages and final naming, multi-system ownership explanation, interrupted-write cleanup, observer failure isolation, complete MCP parameter descriptions, and explicit package inventory. The preview-layout migration topic and the requested ReleaseProcess caveat were removed. The test Publish collector now retries recognized transient failures within one cancellation budget and propagates other exceptions. Commit69b1f5db9applies the requested short loader names and renames standalone hosting to `AddMachineVision`, with callers, tests and documentation updated. The intervening master merge is preserved. ### Verification **Latest API naming changes, verified locally:** | Scope | Result | | --- | --- | | Full Vision suite, net10 | 572 passed, 1 existing skip, 0 failed | | Full Vision suite, net48 | 515 passed, 0 skipped, 0 failed | | Final loader/hosting cases after cross-TFM null-guard adjustment | 9 passed, 0 failed, 0 skipped | | Machine Vision MCP and profile cases | 198 passed, 0 failed, 0 skipped | | Main and MCP AOT test-project compilation | Both built with 0 warnings/errors | | MCP AOT companion, managed execution | 5 passed, 0 failed, 0 skipped; not claimed as native execution | `ModelLoaderNamesRegisterTypedFactories`, `ModelLoaderNamesRejectNullArguments`, and `GeneratedModelsLoadTogetherAndCreateTypedInstances` cover all six loader overloads, fluent return identities, typed instances, namespace coexistence and null guards. Existing standalone Part 2 tests verify idempotent registration, one-owner enforcement and null handling through `AddMachineVision`. No source-generator implementation, CLR namespace, wire identity or model payload was changed for these naming updates. The current69b1f5db9revision is running [CI](https://github.com/OPCFoundation/UA-.NETStandard/actions/runs/38165826937), [package validation](https://github.com/OPCFoundation/UA-.NETStandard/actions/runs/38165823359), [MCP usability](https://github.com/OPCFoundation/UA-.NETStandard/actions/runs/38165826874), [CodeQL](https://github.com/OPCFoundation/UA-.NETStandard/actions/runs/38165826906), and [Images CI](https://github.com/OPCFoundation/UA-.NETStandard/actions/runs/38165826884). These runs are not claimed as completed results. **Earlier completed verification:** all seven default-TFM package archives were inspected after the package rename: exact IDs, 27 clean DLLs, correct dependencies/readmes/licenses, and unchanged pinned model hashes. [CI onc75ec4fa6](https://github.com/OPCFoundation/UA-.NETStandard/actions/runs/38142519760) passed, including the formerly failing Linux batch and required summary. Its unchanged coverage gate passed at **88.56% lines, 78.96% branches, 84.28% patch** against the 75% patch floor. The later bounded-Publish-helper regressions reproduced 13 failures before the fix and passed afterward; the full mainline triggering fixture passed 67 tests on each of net10/net48. These are historical results, not current full-matrix results. <details> <summary>Executed API naming validation commands and outcomes</summary> Run from the worktree root using the established artifact trees. Dependency builds were performed before the no-build full net10 run; the focused loader/hosting selection was rebuilt and rerun after the final null-guard adjustment. ```powershell $files = 'C:\Users\mschier\.copilot\session-state\deb2ba10-b0e9-4a44-8783-8331bc850814\files' $env:DOTNET_CLI_USE_MSBUILD_SERVER = '0' $env:TEMP = 'C:\Temp\mv-deb2\followup' $env:TMP = $env:TEMP # Passed: 572; Failed: 0; Skipped: 1. dotnet test tests\Opc.Ua.Vision.Tests\Opc.Ua.Vision.Tests.csproj -c Release -f net10.0 --no-build --no-restore --artifacts-path artifacts\vision-followup -p:CustomTestTarget=net10.0 -p:UseSharedCompilation=false -m:1 -nr:false --logger 'console;verbosity=minimal' --logger trx --results-directory "$files\vision-api-names-net10" -- 'NUnit.WorkDirectory=C:\Temp\mv-deb2\followup' # Passed: 9; Failed: 0; Skipped: 0, on the final source. dotnet test tests\Opc.Ua.Vision.Tests\Opc.Ua.Vision.Tests.csproj -c Release -f net10.0 --no-restore --artifacts-path artifacts\vision-followup -p:CustomTestTarget=net10.0 -p:UseSharedCompilation=false -m:1 -nr:false --filter 'FullyQualifiedName~MachineVisionModelTests|FullyQualifiedName~StandalonePartTwoRegistrationUsesOneOwnerAndOneModelProvider|FullyQualifiedName~StandalonePartTwoRequiresRegistrationAndRejectsCompetingOwnership' --logger 'console;verbosity=minimal' --logger trx --results-directory "$files\vision-api-names-final-model" -- 'NUnit.WorkDirectory=C:\Temp\mv-deb2\followup' # Passed: 198; Failed: 0; Skipped: 0. dotnet test tests\Opc.Ua.Tools.Tests\Opc.Ua.Tools.Tests.csproj -c Release -f net10.0 --no-restore --artifacts-path artifacts\vision-followup -p:CustomTestTarget=net10.0 -p:UseSharedCompilation=false -m:1 -nr:false --filter 'FullyQualifiedName~VisionMachineToolsTests|FullyQualifiedName~McpHostBuilderTests|FullyQualifiedName~McpToolProfileCompositionTests' --logger 'console;verbosity=minimal' --logger trx --results-directory "$files\vision-api-names-final-mcp" -- 'NUnit.WorkDirectory=C:\Temp\mv-deb2\followup' $env:TEMP = 'C:\Temp\mv-deb2\followup48' $env:TMP = $env:TEMP # Passed: 515; Failed: 0; Skipped: 0. dotnet test tests\Opc.Ua.Vision.Tests\Opc.Ua.Vision.Tests.csproj -c Debug -f net48 --no-restore --artifacts-path "$files\solution-net48-artifacts" -p:CustomTestTarget=net48 -p:UseSharedCompilation=false -m:1 -nr:false --logger 'console;verbosity=minimal' --logger trx --results-directory "$files\vision-api-names-net48-final" -- 'NUnit.WorkDirectory=C:\Temp\mv-deb2\followup48' ``` Opt-in long-running/stress/durable tiers have not been rerun locally; the unqualified **all tests** checklist item remains open. Original implementation matrix counts are not presented as current results. </details> ## Related Issues - Fixes #4732. The requester reviewed and approved the implementation plan and subsequent packaging/API decisions in the development session. The tracking issue records the design; maintainer architectural discussion/sign-off there is still pending. This PR remains a draft. ## 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. - [x] 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. - [x] I have addressed **all** PR feedback received.
474 lines
20 KiB
C#
474 lines
20 KiB
C#
/* ========================================================================
|
|
* Copyright (c) 2005-2025 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/
|
|
* ======================================================================*/
|
|
|
|
// CA2000: test code; many disposables are ownership-transferred to test fixtures or short-lived,
|
|
// making CA2000 noisy without a real leak risk. Disabled file-level for the suite.
|
|
#pragma warning disable CA2000
|
|
using System;
|
|
using System.Threading.Tasks;
|
|
using NUnit.Framework;
|
|
using Opc.Ua.Tests;
|
|
|
|
namespace Opc.Ua.Core.Tests.Stack.State
|
|
{
|
|
/// <summary>
|
|
/// Tests for the ConditionState class.
|
|
/// </summary>
|
|
[TestFixture]
|
|
[Category("ConditionState")]
|
|
[SetCulture("en-us")]
|
|
[SetUICulture("en-us")]
|
|
[Parallelizable]
|
|
public class ConditionStateTests
|
|
{
|
|
private ISystemContext m_context;
|
|
private ITelemetryContext m_telemetry;
|
|
|
|
[OneTimeSetUp]
|
|
protected void OneTimeSetUp()
|
|
{
|
|
m_telemetry = NUnitTelemetryContext.Create();
|
|
var messageContext = ServiceMessageContext.Create(m_telemetry);
|
|
// Add OPC UA namespace
|
|
messageContext.NamespaceUris.GetIndexOrAppend(Namespaces.OpcUa);
|
|
m_context = new SystemContext(m_telemetry)
|
|
{
|
|
NamespaceUris = messageContext.NamespaceUris
|
|
};
|
|
}
|
|
|
|
[OneTimeTearDown]
|
|
protected void OneTimeTearDown()
|
|
{
|
|
(m_context as IDisposable)?.Dispose();
|
|
}
|
|
|
|
[TestCase(true, true, true, true)]
|
|
[TestCase(false, true, true, false)]
|
|
[TestCase(false, false, true, true)]
|
|
[TestCase(true, true, false, false)]
|
|
public async Task AdditionalRetentionPreservesStandardAndDisabledRulesAsync(
|
|
bool additionalRetain, bool acknowledged, bool enabled, bool expectedRetain)
|
|
{
|
|
var condition = new AcknowledgeableConditionState(null);
|
|
condition.Create(m_context, new NodeId(1u), QualifiedName.From("Condition"), default, true);
|
|
condition.SetEnableState(m_context, !enabled);
|
|
condition.SetAcknowledgedState(m_context, acknowledged);
|
|
int evaluated = 0;
|
|
condition.RetainStateProvider = current =>
|
|
{
|
|
Assert.That(current, Is.SameAs(condition));
|
|
evaluated++;
|
|
return additionalRetain;
|
|
};
|
|
MethodState method = enabled ? condition.Enable! : condition.Disable!;
|
|
ServiceResult result = await method.CallAsync(m_context, condition.NodeId, [], [], [])
|
|
.ConfigureAwait(false);
|
|
Assert.That(result.StatusCode, Is.EqualTo(StatusCodes.Good));
|
|
Assert.That(condition.Retain!.Value, Is.EqualTo(expectedRetain));
|
|
Assert.That(evaluated, enabled ? Is.GreaterThan(0) : Is.Zero);
|
|
}
|
|
|
|
[Test]
|
|
public void BranchPreservesAdditionalRetentionProvider()
|
|
{
|
|
var condition = new AcknowledgeableConditionState(null);
|
|
condition.Create(m_context, new NodeId(1u), QualifiedName.From("Condition"), default, true);
|
|
condition.SetEnableState(m_context, true);
|
|
condition.RetainStateProvider = current => !current.BranchId!.Value.IsNull;
|
|
ConditionState? branch = condition.CreateBranch(m_context, new NodeId(2u));
|
|
Assert.That(branch, Is.Not.Null);
|
|
Assert.That(branch!.RetainStateProvider, Is.SameAs(condition.RetainStateProvider));
|
|
Assert.That(branch.RetainStateProvider!(branch), Is.True);
|
|
Assert.That(condition.RetainStateProvider(condition), Is.False);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test that SetEnableState updates the timestamp and clears change masks.
|
|
/// </summary>
|
|
[Test]
|
|
public void SetEnableStateUpdatesTimestampAndClearsChangeMasks()
|
|
{
|
|
var condition = new ConditionState(null);
|
|
condition.Create(m_context, new NodeId(1), QualifiedName.From("Condition"), default, true);
|
|
|
|
// Set initial state
|
|
DateTimeUtc beforeTime = DateTimeUtc.Now;
|
|
condition.SetEnableState(m_context, true);
|
|
DateTimeUtc afterTime = DateTimeUtc.Now;
|
|
|
|
// Verify timestamp is updated
|
|
Assert.That(condition.EnabledState!.Timestamp, Is.GreaterThanOrEqualTo(beforeTime));
|
|
Assert.That(condition.EnabledState.Timestamp, Is.LessThanOrEqualTo(afterTime));
|
|
|
|
// Verify change masks are cleared (all should be None)
|
|
Assert.That(condition.ChangeMasks, Is.EqualTo(NodeStateChangeMasks.None));
|
|
Assert.That(condition.EnabledState.ChangeMasks, Is.EqualTo(NodeStateChangeMasks.None));
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test that SetSeverity updates the timestamp and clears change masks.
|
|
/// </summary>
|
|
[Test]
|
|
public void SetSeverityUpdatesTimestampAndClearsChangeMasks()
|
|
{
|
|
var condition = new ConditionState(null);
|
|
condition.Create(m_context, new NodeId(1), QualifiedName.From("Condition"), default, true);
|
|
|
|
DateTimeUtc beforeTime = DateTimeUtc.Now;
|
|
condition.SetSeverity(m_context, EventSeverity.High);
|
|
DateTimeUtc afterTime = DateTimeUtc.Now;
|
|
|
|
// Verify timestamps are updated
|
|
Assert.That(condition.Severity!.Timestamp, Is.GreaterThanOrEqualTo(beforeTime));
|
|
Assert.That(condition.Severity.Timestamp, Is.LessThanOrEqualTo(afterTime));
|
|
Assert.That(condition.LastSeverity!.Timestamp, Is.GreaterThanOrEqualTo(beforeTime));
|
|
Assert.That(condition.LastSeverity.Timestamp, Is.LessThanOrEqualTo(afterTime));
|
|
|
|
// Verify change masks are cleared
|
|
Assert.That(condition.ChangeMasks, Is.EqualTo(NodeStateChangeMasks.None));
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test that SetActiveState updates the timestamp and clears change masks.
|
|
/// </summary>
|
|
[Test]
|
|
public void SetActiveStateUpdatesTimestampAndClearsChangeMasks()
|
|
{
|
|
var alarm = new AlarmConditionState(m_telemetry, null!);
|
|
alarm.Create(m_context, new NodeId(1), QualifiedName.From("Alarm"), default, true);
|
|
|
|
DateTimeUtc beforeTime = DateTimeUtc.Now;
|
|
alarm.SetActiveState(m_context, true);
|
|
DateTimeUtc afterTime = DateTimeUtc.Now;
|
|
|
|
// Verify timestamp is updated
|
|
Assert.That(alarm.ActiveState!.Timestamp, Is.GreaterThanOrEqualTo(beforeTime));
|
|
Assert.That(alarm.ActiveState.Timestamp, Is.LessThanOrEqualTo(afterTime));
|
|
|
|
// Verify change masks are cleared
|
|
Assert.That(alarm.ChangeMasks, Is.EqualTo(NodeStateChangeMasks.None));
|
|
Assert.That(alarm.ActiveState.ChangeMasks, Is.EqualTo(NodeStateChangeMasks.None));
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test that SetSuppressedState updates the timestamp and clears change masks.
|
|
/// </summary>
|
|
[Test]
|
|
public void SetSuppressedStateUpdatesTimestampAndClearsChangeMasks()
|
|
{
|
|
var alarm = new AlarmConditionState(m_telemetry, null!);
|
|
alarm.Create(m_context, new NodeId(1), QualifiedName.From("Alarm"), default, true);
|
|
alarm.SuppressedState = new TwoStateVariableState(alarm);
|
|
alarm.SuppressedState.Create(m_context, default, QualifiedName.From(BrowseNames.SuppressedState), default, false);
|
|
|
|
DateTimeUtc beforeTime = DateTimeUtc.Now;
|
|
alarm.SetSuppressedState(m_context, true);
|
|
DateTimeUtc afterTime = DateTimeUtc.Now;
|
|
|
|
// Verify timestamp is updated
|
|
Assert.That(alarm.SuppressedState.Timestamp, Is.GreaterThanOrEqualTo(beforeTime));
|
|
Assert.That(alarm.SuppressedState.Timestamp, Is.LessThanOrEqualTo(afterTime));
|
|
|
|
// Verify change masks are cleared
|
|
Assert.That(alarm.ChangeMasks, Is.EqualTo(NodeStateChangeMasks.None));
|
|
Assert.That(alarm.SuppressedState.ChangeMasks, Is.EqualTo(NodeStateChangeMasks.None));
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test that SetAcknowledgedState updates the timestamp and clears change masks.
|
|
/// </summary>
|
|
[Test]
|
|
public void SetAcknowledgedStateUpdatesTimestampAndClearsChangeMasks()
|
|
{
|
|
var condition = new AcknowledgeableConditionState(null);
|
|
condition.Create(m_context, new NodeId(1), QualifiedName.From("AckCondition"), default, true);
|
|
|
|
DateTimeUtc beforeTime = DateTimeUtc.Now;
|
|
condition.SetAcknowledgedState(m_context, true);
|
|
DateTimeUtc afterTime = DateTimeUtc.Now;
|
|
|
|
// Verify timestamp is updated
|
|
Assert.That(condition.AckedState!.Timestamp, Is.GreaterThanOrEqualTo(beforeTime));
|
|
Assert.That(condition.AckedState.Timestamp, Is.LessThanOrEqualTo(afterTime));
|
|
|
|
// Verify change masks are cleared
|
|
Assert.That(condition.ChangeMasks, Is.EqualTo(NodeStateChangeMasks.None));
|
|
Assert.That(condition.AckedState.ChangeMasks, Is.EqualTo(NodeStateChangeMasks.None));
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test that SetConfirmedState updates the timestamp and clears change masks.
|
|
/// </summary>
|
|
[Test]
|
|
public void SetConfirmedStateUpdatesTimestampAndClearsChangeMasks()
|
|
{
|
|
var condition = new AcknowledgeableConditionState(null);
|
|
condition.Create(m_context, new NodeId(1), QualifiedName.From("AckCondition"), default, true);
|
|
condition.ConfirmedState = new TwoStateVariableState(condition);
|
|
condition.ConfirmedState.Create(m_context, default, QualifiedName.From(BrowseNames.ConfirmedState), default, false);
|
|
|
|
DateTimeUtc beforeTime = DateTimeUtc.Now;
|
|
condition.SetConfirmedState(m_context, true);
|
|
DateTimeUtc afterTime = DateTimeUtc.Now;
|
|
|
|
// Verify timestamp is updated
|
|
Assert.That(condition.ConfirmedState.Timestamp, Is.GreaterThanOrEqualTo(beforeTime));
|
|
Assert.That(condition.ConfirmedState.Timestamp, Is.LessThanOrEqualTo(afterTime));
|
|
|
|
// Verify change masks are cleared
|
|
Assert.That(condition.ChangeMasks, Is.EqualTo(NodeStateChangeMasks.None));
|
|
Assert.That(condition.ConfirmedState.ChangeMasks, Is.EqualTo(NodeStateChangeMasks.None));
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test that SetShelvingState updates the timestamp and clears change masks.
|
|
/// </summary>
|
|
[Test]
|
|
public void SetShelvingStateUpdatesTimestampAndClearsChangeMasks()
|
|
{
|
|
var alarm = new AlarmConditionState(m_telemetry, null!);
|
|
alarm.Create(m_context, new NodeId(1), QualifiedName.From("Alarm"), default, true);
|
|
alarm.ShelvingState = new ShelvedStateMachineState(alarm);
|
|
alarm.ShelvingState.Create(m_context, default, QualifiedName.From(BrowseNames.ShelvingState), default, false);
|
|
alarm.ShelvingState.UnshelveTime = PropertyState<double>.With<VariantBuilder>(alarm.ShelvingState);
|
|
|
|
DateTimeUtc beforeTime = DateTimeUtc.Now;
|
|
alarm.SetShelvingState(m_context, true, false, 1000);
|
|
DateTimeUtc afterTime = DateTimeUtc.Now;
|
|
|
|
// Verify timestamp is updated on UnshelveTime
|
|
Assert.That(alarm.ShelvingState.UnshelveTime.Timestamp, Is.GreaterThanOrEqualTo(beforeTime));
|
|
Assert.That(alarm.ShelvingState.UnshelveTime.Timestamp, Is.LessThanOrEqualTo(afterTime));
|
|
|
|
// Verify change masks are cleared
|
|
Assert.That(alarm.ChangeMasks, Is.EqualTo(NodeStateChangeMasks.None));
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test that subscribed clients are notified when SetActiveState is called.
|
|
/// This simulates the issue scenario.
|
|
/// </summary>
|
|
[Test]
|
|
public void SetActiveStateNotifiesSubscribers()
|
|
{
|
|
var alarm = new AlarmConditionState(m_telemetry, null!);
|
|
alarm.Create(m_context, new NodeId(1), QualifiedName.From("Alarm"), default, true);
|
|
|
|
// Initially inactive
|
|
alarm.SetActiveState(m_context, false);
|
|
|
|
// Now activate the alarm - timestamp should be greater than or equal to the initial timestamp
|
|
// since both could be set to DateTimeUtc.Now which has limited precision
|
|
DateTimeUtc beforeActivation = DateTimeUtc.Now;
|
|
alarm.SetActiveState(m_context, true);
|
|
DateTimeUtc afterActivation = DateTimeUtc.Now;
|
|
|
|
// Verify that the timestamp is within the expected range
|
|
Assert.That(alarm.ActiveState!.Timestamp, Is.GreaterThanOrEqualTo(beforeActivation));
|
|
Assert.That(alarm.ActiveState.Timestamp, Is.LessThanOrEqualTo(afterActivation));
|
|
|
|
// Verify that change masks were cleared (indicating subscribers were notified)
|
|
Assert.That(alarm.ActiveState.ChangeMasks, Is.EqualTo(NodeStateChangeMasks.None));
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test that UpdateStateAfterEnable calls EvaluateRetainStateOnEnable
|
|
/// and that the default implementation sets Retain based on GetRetainState.
|
|
/// </summary>
|
|
[Test]
|
|
public void UpdateStateAfterEnableCallsEvaluateRetainStateOnEnable()
|
|
{
|
|
// Arrange
|
|
var condition = new ConditionState(null);
|
|
condition.Create(m_context, default, QualifiedName.From("TestCondition"), default, true);
|
|
|
|
// Initially disabled
|
|
condition.SetEnableState(m_context, false);
|
|
Assert.That(condition.EnabledState!.Id!.Value, Is.False);
|
|
Assert.That(condition.Retain!.Value, Is.False);
|
|
|
|
// Act - Enable the condition
|
|
condition.SetEnableState(m_context, true);
|
|
|
|
// Assert - Enabled state should be true
|
|
Assert.That(condition.EnabledState.Id.Value, Is.True);
|
|
|
|
// The default implementation should call UpdateRetainState which uses GetRetainState
|
|
// For base ConditionState with no branches, GetRetainState returns false when enabled
|
|
Assert.That(condition.Retain.Value, Is.False);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test that UpdateStateAfterDisable sets Retain to false as per specification.
|
|
/// </summary>
|
|
[Test]
|
|
public void UpdateStateAfterDisableSetsRetainToFalse()
|
|
{
|
|
// Arrange
|
|
var condition = new TestConditionStateWithRetain(null!);
|
|
condition.Create(m_context, default, QualifiedName.From("TestCondition"), default, true);
|
|
|
|
// Enable the condition and set retain to true
|
|
condition.SetEnableState(m_context, true);
|
|
condition.ForceRetain(true);
|
|
Assert.That(condition.Retain!.Value, Is.True);
|
|
|
|
// Act - Disable the condition
|
|
condition.SetEnableState(m_context, false);
|
|
|
|
// Assert - Retain should be false per specification
|
|
Assert.That(condition.EnabledState!.Id!.Value, Is.False);
|
|
Assert.That(condition.Retain.Value, Is.False);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test that a derived class can override EvaluateRetainStateOnEnable
|
|
/// to provide custom logic for determining the Retain value.
|
|
/// </summary>
|
|
[Test]
|
|
public void DerivedClassCanOverrideEvaluateRetainStateOnEnable()
|
|
{
|
|
// Arrange
|
|
var condition = new TestConditionStateWithCustomRetain(null!);
|
|
condition.Create(m_context, default, QualifiedName.From("TestCondition"), default, true);
|
|
|
|
// Initially disabled
|
|
condition.SetEnableState(m_context, false);
|
|
Assert.That(condition.Retain!.Value, Is.False);
|
|
|
|
// Act - Enable the condition
|
|
condition.SetEnableState(m_context, true);
|
|
|
|
// Assert - The custom implementation should have been called
|
|
Assert.That(condition.EnabledState!.Id!.Value, Is.True);
|
|
Assert.That(condition.EvaluateCalled, Is.True);
|
|
Assert.That(condition.Retain.Value, Is.True); // Custom logic always sets to true
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test that the branch collection can be safely created, enumerated and
|
|
/// cleared from multiple threads concurrently without throwing.
|
|
/// Reproduces the race that previously crashed the process when the
|
|
/// branch dictionary was enumerated while being modified.
|
|
/// </summary>
|
|
[Test]
|
|
[CancelAfter(10_000)]
|
|
public void ConcurrentBranchAccessIsThreadSafe()
|
|
{
|
|
var condition = new ConditionState(null);
|
|
condition.Create(m_context, new NodeId(1), QualifiedName.From("Condition"), default, true);
|
|
// Ensure each created branch receives a unique EventId.
|
|
condition.AutoReportStateChanges = true;
|
|
condition.SetEnableState(m_context, true);
|
|
|
|
const int iterations = 2000;
|
|
using var cts = new System.Threading.CancellationTokenSource();
|
|
|
|
// Writer: continuously creates and clears branches.
|
|
var writer = System.Threading.Tasks.Task.Run(() =>
|
|
{
|
|
for (int i = 0; i < iterations; i++)
|
|
{
|
|
condition.CreateBranch(m_context, new NodeId((uint)(i + 2)));
|
|
if (i % 10 == 0)
|
|
{
|
|
condition.ClearBranches();
|
|
}
|
|
}
|
|
});
|
|
|
|
// Reader: continuously enumerates and counts branches.
|
|
var reader = System.Threading.Tasks.Task.Run(() =>
|
|
{
|
|
while (!cts.Token.IsCancellationRequested)
|
|
{
|
|
foreach (System.Collections.Generic.KeyValuePair<string, ConditionState> branch
|
|
in condition.GetBranches())
|
|
{
|
|
Assert.That(branch.Value, Is.Not.Null);
|
|
}
|
|
|
|
_ = condition.GetBranchCount();
|
|
}
|
|
});
|
|
|
|
Assert.That(() =>
|
|
{
|
|
try
|
|
{
|
|
writer.GetAwaiter().GetResult();
|
|
}
|
|
finally
|
|
{
|
|
cts.Cancel();
|
|
}
|
|
|
|
reader.GetAwaiter().GetResult();
|
|
}, Throws.Nothing);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test condition that exposes method to force Retain value for testing.
|
|
/// </summary>
|
|
private sealed class TestConditionStateWithRetain : ConditionState
|
|
{
|
|
public TestConditionStateWithRetain(NodeState parent)
|
|
: base(parent)
|
|
{
|
|
}
|
|
|
|
public void ForceRetain(bool value)
|
|
{
|
|
Retain!.Value = value;
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test condition that overrides EvaluateRetainStateOnEnable.
|
|
/// </summary>
|
|
private sealed class TestConditionStateWithCustomRetain : ConditionState
|
|
{
|
|
public bool EvaluateCalled { get; private set; }
|
|
|
|
public TestConditionStateWithCustomRetain(NodeState parent)
|
|
: base(parent)
|
|
{
|
|
}
|
|
|
|
protected override void EvaluateRetainStateOnEnable(ISystemContext context)
|
|
{
|
|
EvaluateCalled = true;
|
|
// Custom logic: always set Retain to true when enabled
|
|
Retain!.Value = true;
|
|
}
|
|
}
|
|
}
|
|
}
|