Files
UA-.NETStandard/tests/Opc.Ua.Core.Tests/Stack/State/ConditionStateTests.cs
Marc Schier 1bb4bdca1d Add Machine Vision Parts 1 and 2 (#4733)
# 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. Commit 69b1f5db9 applies 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 current 69b1f5db9 revision 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 on
c75ec4fa6](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.
2026-10-12 07:01:27 +00:00

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;
}
}
}
}