fix(tests,cli): implement IOpcUaAddressSpaceSink.EnsureVariable in test fakes; fix CLI CS1587

Resolves the 12 reported build errors (7 CS0535 sink fakes + 5 CLI CS1587).
Runtime.Tests green (74). NOTE: OpcUaServer.Tests still has pre-existing CS7036
errors from the in-progress Galaxy-tag workstream (Phase7Plan/Phase7CompositionResult
new required params) — separate, test-only, not addressed here.

src/Drivers/Cli/ZB.MOM.WW.OtOpcUa.Driver.AbLegacy.Cli/Commands/SubscribeCommand.cs

@@ -12,20 +12,20 @@ namespace ZB.MOM.WW.OtOpcUa.Driver.AbLegacy.Cli.Commands;
 [Command("subscribe", Description = "Watch a PCCC file address via polled subscription until Ctrl+C.")]
 public sealed class SubscribeCommand : AbLegacyCommandBase
 {
-    [CommandOption("address", 'a', Description = "PCCC file address — same format as `read`.", IsRequired = true)]
     /// <summary>Gets or sets the PCCC file address to subscribe to.</summary>
+    [CommandOption("address", 'a', Description = "PCCC file address — same format as `read`.", IsRequired = true)]
     public string Address { get; init; } = default!;
 
+    /// <summary>Gets or sets the data type of the address.</summary>
     [CommandOption("type", 't', Description =
         "Bit / Int / Long / Float / AnalogInt / String / TimerElement / CounterElement / " +
         "ControlElement (default Int).")]
-    /// <summary>Gets or sets the data type of the address.</summary>
     public AbLegacyDataType DataType { get; init; } = AbLegacyDataType.Int;
 
+    /// <summary>Gets or sets the polling interval in milliseconds.</summary>
     [CommandOption("interval-ms", 'i', Description =
         "Publishing interval in milliseconds (default 1000). PollGroupEngine floors " +
         "sub-250ms values.")]
-    /// <summary>Gets or sets the polling interval in milliseconds.</summary>
     public int IntervalMs { get; init; } = 1000;
 
     /// <inheritdoc />

src/Drivers/Cli/ZB.MOM.WW.OtOpcUa.Driver.S7.Cli/Commands/ProbeCommand.cs

@@ -13,14 +13,14 @@ namespace ZB.MOM.WW.OtOpcUa.Driver.S7.Cli.Commands;
 [Command("probe", Description = "Verify the S7 endpoint is reachable and a sample read succeeds.")]
 public sealed class ProbeCommand : S7CommandBase
 {
+    /// <summary>Gets or sets the S7 address to probe.</summary>
     [CommandOption("address", 'a', Description =
         "Probe address (default MW0 — merker word 0). DB1.DBW0 if your PLC project " +
         "reserves a fingerprint DB.")]
-    /// <summary>Gets or sets the S7 address to probe.</summary>
     public string Address { get; init; } = "MW0";
 
-    [CommandOption("type", Description = "Probe data type (default Int16).")]
     /// <summary>Gets or sets the data type of the probe address.</summary>
+    [CommandOption("type", Description = "Probe data type (default Int16).")]
     public S7DataType DataType { get; init; } = S7DataType.Int16;
 
     /// <inheritdoc />

tests/Server/ZB.MOM.WW.OtOpcUa.OpcUaServer.Tests/DeferredAddressSpaceSinkTests.cs

@@ -84,6 +84,9 @@ public sealed class DeferredAddressSpaceSinkTests
         public void EnsureFolder(string folderNodeId, string? parentNodeId, string displayName)
             => CallQueue.Enqueue($"EF:{folderNodeId}");
         /// <inheritdoc />
+        public void EnsureVariable(string variableNodeId, string? parentFolderNodeId, string displayName, string dataType)
+            => CallQueue.Enqueue($"EV:{variableNodeId}");
+        /// <inheritdoc />
         public void RebuildAddressSpace() => CallQueue.Enqueue("RB");
     }
 }

tests/Server/ZB.MOM.WW.OtOpcUa.OpcUaServer.Tests/Phase7ApplierHierarchyTests.cs

@@ -149,6 +149,12 @@ public sealed class Phase7ApplierHierarchyTests : IDisposable
         /// <param name="displayName">The display name of the folder.</param>
         public void EnsureFolder(string folderNodeId, string? parentNodeId, string displayName)
             => _calls.Enqueue((folderNodeId, parentNodeId, displayName));
+        /// <summary>Ensures a variable exists (stub implementation for testing).</summary>
+        /// <param name="variableNodeId">The node ID of the variable.</param>
+        /// <param name="parentFolderNodeId">The node ID of the parent folder, or null for root.</param>
+        /// <param name="displayName">The display name of the variable.</param>
+        /// <param name="dataType">The OPC UA built-in type name.</param>
+        public void EnsureVariable(string variableNodeId, string? parentFolderNodeId, string displayName, string dataType) { }
         /// <summary>Rebuilds the address space (stub implementation for testing).</summary>
         public void RebuildAddressSpace() { }
     }

tests/Server/ZB.MOM.WW.OtOpcUa.OpcUaServer.Tests/Phase7ApplierTests.cs

@@ -133,6 +133,8 @@ public sealed class Phase7ApplierTests
         public ConcurrentQueue<(string NodeId, bool Active, bool Acknowledged)> AlarmQueue { get; } = new();
         /// <summary>Gets the queue of folder creation calls.</summary>
         public ConcurrentQueue<(string NodeId, string? Parent, string DisplayName)> FolderQueue { get; } = new();
+        /// <summary>Gets the queue of variable creation calls.</summary>
+        public ConcurrentQueue<(string NodeId, string? Parent, string DisplayName, string DataType)> VariableQueue { get; } = new();
         /// <summary>Gets the number of rebuild calls made on this sink.</summary>
         public int RebuildCalls;
 
@@ -140,6 +142,8 @@ public sealed class Phase7ApplierTests
         public List<(string NodeId, bool Active, bool Acknowledged)> AlarmWrites => AlarmQueue.ToList();
         /// <summary>Gets the list of recorded folder creation calls.</summary>
         public List<(string NodeId, string? Parent, string DisplayName)> FolderCalls => FolderQueue.ToList();
+        /// <summary>Gets the list of recorded variable creation calls.</summary>
+        public List<(string NodeId, string? Parent, string DisplayName, string DataType)> VariableCalls => VariableQueue.ToList();
 
         /// <summary>Records a value write (no-op in this recording sink).</summary>
         /// <param name="nodeId">The node ID.</param>
@@ -160,6 +164,13 @@ public sealed class Phase7ApplierTests
         /// <param name="displayName">The display name for the folder.</param>
         public void EnsureFolder(string folderNodeId, string? parentNodeId, string displayName)
             => FolderQueue.Enqueue((folderNodeId, parentNodeId, displayName));
+        /// <summary>Records a variable creation call.</summary>
+        /// <param name="variableNodeId">The variable node ID.</param>
+        /// <param name="parentFolderNodeId">The parent folder node ID, if any.</param>
+        /// <param name="displayName">The display name for the variable.</param>
+        /// <param name="dataType">The OPC UA built-in type name.</param>
+        public void EnsureVariable(string variableNodeId, string? parentFolderNodeId, string displayName, string dataType)
+            => VariableQueue.Enqueue((variableNodeId, parentFolderNodeId, displayName, dataType));
         /// <summary>Records a rebuild address space call.</summary>
         public void RebuildAddressSpace() => Interlocked.Increment(ref RebuildCalls);
     }
@@ -192,6 +203,12 @@ public sealed class Phase7ApplierTests
         /// <param name="parentNodeId">The parent folder node ID, if any.</param>
         /// <param name="displayName">The display name for the folder.</param>
         public void EnsureFolder(string folderNodeId, string? parentNodeId, string displayName) { }
+        /// <summary>No-op variable creation call.</summary>
+        /// <param name="variableNodeId">The variable node ID.</param>
+        /// <param name="parentFolderNodeId">The parent folder node ID, if any.</param>
+        /// <param name="displayName">The display name for the variable.</param>
+        /// <param name="dataType">The OPC UA built-in type name.</param>
+        public void EnsureVariable(string variableNodeId, string? parentFolderNodeId, string displayName, string dataType) { }
         /// <summary>No-op rebuild address space call.</summary>
         public void RebuildAddressSpace() { }
     }

tests/Server/ZB.MOM.WW.OtOpcUa.Runtime.Tests/Observability/OtOpcUaTelemetryHookTests.cs

@@ -206,6 +206,12 @@ public sealed class OtOpcUaTelemetryHookTests : RuntimeActorTestBase
         /// <param name="parentNodeId">The parent folder node identifier.</param>
         /// <param name="displayName">The display name for the folder.</param>
         public void EnsureFolder(string folderNodeId, string? parentNodeId, string displayName) { }
+        /// <summary>Ensures variable exists (stub implementation).</summary>
+        /// <param name="variableNodeId">The variable node identifier.</param>
+        /// <param name="parentFolderNodeId">The parent folder node identifier.</param>
+        /// <param name="displayName">The display name for the variable.</param>
+        /// <param name="dataType">The OPC UA built-in type name.</param>
+        public void EnsureVariable(string variableNodeId, string? parentFolderNodeId, string displayName, string dataType) { }
         /// <summary>Rebuilds address space (recorded via span).</summary>
         public void RebuildAddressSpace() { /* recorded via span */ }
     }

tests/Server/ZB.MOM.WW.OtOpcUa.Runtime.Tests/OpcUa/OpcUaPublishActorRebuildTests.cs

@@ -161,6 +161,13 @@ public sealed class OpcUaPublishActorRebuildTests : RuntimeActorTestBase
         /// <param name="displayName">The display name of the folder.</param>
         public void EnsureFolder(string folderNodeId, string? parentNodeId, string displayName)
             => Calls.Enqueue($"EF:{folderNodeId}");
+        /// <summary>Records a variable ensure call.</summary>
+        /// <param name="variableNodeId">The variable node ID.</param>
+        /// <param name="parentFolderNodeId">The parent folder node ID, or null if this is a root variable.</param>
+        /// <param name="displayName">The display name of the variable.</param>
+        /// <param name="dataType">The OPC UA built-in type name.</param>
+        public void EnsureVariable(string variableNodeId, string? parentFolderNodeId, string displayName, string dataType)
+            => Calls.Enqueue($"EV:{variableNodeId}");
         /// <summary>Records a rebuild address space call.</summary>
         public void RebuildAddressSpace() => Interlocked.Increment(ref RebuildCalls);
     }

tests/Server/ZB.MOM.WW.OtOpcUa.Runtime.Tests/OpcUa/OpcUaPublishActorTests.cs

@@ -182,6 +182,13 @@ public sealed class OpcUaPublishActorTests : RuntimeActorTestBase
         /// <param name="displayName">The display name of the folder.</param>
         public void EnsureFolder(string folderNodeId, string? parentNodeId, string displayName) { }
 
+        /// <summary>Ensures a variable exists (no-op in test).</summary>
+        /// <param name="variableNodeId">The OPC UA variable node identifier.</param>
+        /// <param name="parentFolderNodeId">The parent folder node identifier, or null for root.</param>
+        /// <param name="displayName">The display name of the variable.</param>
+        /// <param name="dataType">The OPC UA built-in type name.</param>
+        public void EnsureVariable(string variableNodeId, string? parentFolderNodeId, string displayName, string dataType) { }
+
         /// <summary>Records a rebuild call.</summary>
         public void RebuildAddressSpace() => Interlocked.Increment(ref RebuildCalls);
     }