node-opcua-address-space 2.163.0 → 2.164.2

package/src/base_node_impl.ts

@@ -167,7 +167,7 @@ export class BaseNodeImpl extends EventEmitter implements BaseNode {
 
     public get addressSpace(): IAddressSpace {
         const _private = BaseNode_getPrivate(this);
-        // istanbul ignore next
+        // c8 ignore next
         if (!_private) {
             throw new Error("Internal error , cannot extract private data from " + this.browseName.toString());
         }
@@ -176,7 +176,7 @@ export class BaseNodeImpl extends EventEmitter implements BaseNode {
 
     protected get addressSpacePrivate(): AddressSpacePrivate {
         const _private = BaseNode_getPrivate(this);
-        // istanbul ignore next
+        // c8 ignore next
         if (!_private) {
             throw new Error("Internal error , cannot extract private data from " + this.browseName.toString());
         }
@@ -445,7 +445,7 @@ export class BaseNodeImpl extends EventEmitter implements BaseNode {
         if (_cache._ref.has(hash)) {
             return _cache._ref.get(hash)!;
         }
-        // istanbul ignore next
+        // c8 ignore next
         if (doDebug && !this.addressSpace.findReferenceType(referenceTypeNode.nodeId)) {
             throw new Error("expecting valid reference name " + referenceType);
         }
@@ -711,7 +711,7 @@ export class BaseNodeImpl extends EventEmitter implements BaseNode {
         if (this.parentNodeId) {
             const parent = this.addressSpace.findNode(this.parentNodeId) as BaseNode;
 
-            // istanbul ignore else
+            // c8 ignore next
             if (parent) {
                 return parent.fullName() + "." + this.browseName.toString() + "";
             } else {
@@ -796,7 +796,7 @@ export class BaseNodeImpl extends EventEmitter implements BaseNode {
 
             const obj = resolveReferenceNode(this.addressSpace, reference);
 
-            // istanbul ignore next
+            // c8 ignore next
             if (!obj) {
                 throw new Error(" cannot find node with id " + reference.nodeId.toString());
             }
@@ -850,7 +850,7 @@ export class BaseNodeImpl extends EventEmitter implements BaseNode {
         // get all possible references
         let references = this.allReferences();
 
-        /* istanbul ignore next */
+        /* c8 ignore next */
         if (do_debug) {
             debugLog("all references :", this.nodeId.toString(), this.browseName.toString());
             dumpReferences(addressSpace, _private._referenceIdx.values());
@@ -870,7 +870,7 @@ export class BaseNodeImpl extends EventEmitter implements BaseNode {
         }
         const referenceDescriptions = _constructReferenceDescription(addressSpace, references, browseDescription.resultMask);
 
-        /* istanbul ignore next */
+        /* c8 ignore next */
         if (do_debug) {
             dumpReferenceDescriptions(this.addressSpace, referenceDescriptions);
         }
@@ -1004,14 +1004,14 @@ export class BaseNodeImpl extends EventEmitter implements BaseNode {
 
         const name = lowerFirstLetter(childNode.browseName.name!.toString());
         if (Object.prototype.hasOwnProperty.call(reservedNames, name)) {
-            // istanbul ignore next
+            // c8 ignore next
             if (doDebug) {
                 // tslint:disable-next-line:no-console
                 debugLog(chalk.bgWhite.red("Ignoring reserved keyword                                     " + name));
             }
             return;
         }
-        /* istanbul ignore next */
+        /* c8 ignore next */
         if (!Object.prototype.hasOwnProperty.call(this, name)) {
             return;
         }
@@ -1170,7 +1170,7 @@ export class BaseNodeImpl extends EventEmitter implements BaseNode {
         return !this.addressSpacePrivate;
     }
 
-    // istanbul ignore next
+    // c8 ignore next
     public dumpXML(xmlWriter: XmlWriter): void {
         console.error(" This ", (NodeClass as any)[this.nodeClass]);
         assert(false, "BaseNode#dumpXML NOT IMPLEMENTED !");
@@ -1189,7 +1189,7 @@ export class BaseNodeImpl extends EventEmitter implements BaseNode {
             // filter out non  Hierarchical References
             const referenceType = resolveReferenceType(addressSpace, reference);
 
-            // istanbul ignore next
+            // c8 ignore next
             if (!referenceType) {
                 console.error(chalk.red(" ERROR"), " cannot find reference ", reference.referenceType, reference.toString());
             }
@@ -1262,7 +1262,7 @@ export class BaseNodeImpl extends EventEmitter implements BaseNode {
         let result: UAReferenceType;
         if (typeof referenceType === "string") {
             result = this.addressSpace.findReferenceType(referenceType)!;
-            /* istanbul ignore next */
+            /* c8 ignore next */
             if (!result) {
                 errorLog("referenceType ", referenceType, " cannot be found");
                 throw new Error("Cannot coerce reference with name " + referenceType);
@@ -1497,7 +1497,7 @@ function _setup_parent_item(this: BaseNode, referencesMap: Map<string, UAReferen
         references = this.findReferencesEx("Aggregates", BrowseDirection.Inverse);
 
         if (references.length >= 1) {
-            // istanbul ignore next
+            // c8 ignore next
             if (references.length > 1) {
                 if (displayWarning) {
                     const options = { addressSpace };
@@ -1519,7 +1519,7 @@ function _setup_parent_item(this: BaseNode, referencesMap: Map<string, UAReferen
 
 function toObject(addressSpace: IAddressSpace, reference: UAReference): BaseNode {
     const obj = resolveReferenceNode(addressSpace, reference);
-    // istanbul ignore next
+    // c8 ignore next
     if (doDebug && !obj) {
         debugLog(
             chalk.red(" Warning :  object with nodeId ") +
@@ -1603,7 +1603,7 @@ function _propagate_ref(this: BaseNode, addressSpace: MinimalistAddressSpace, re
     // filter out non  Hierarchical References
     const referenceType = ReferenceImpl.resolveReferenceType(addressSpace, reference);
 
-    // istanbul ignore next
+    // c8 ignore next
     if (!referenceType) {
         errorLog(chalk.red(" ERROR"), " cannot find reference ", reference.referenceType, reference.toString());
     }
@@ -1622,7 +1622,7 @@ function _propagate_ref(this: BaseNode, addressSpace: MinimalistAddressSpace, re
     if (related_node) {
         // verify that reference doesn't point to object it this (see mantis 3099)
         if (sameNodeId(reference.nodeId, this.nodeId)) {
-            // istanbul ignore next
+            // c8 ignore next
             if (displayWarningReferencePointingToItSelf) {
                 // this could happen with method
                 warningLog("  Warning: a Reference is pointing to source ", this.nodeId.toString(), this.browseName.toString(), ". Is this intentional ?");
@@ -1687,7 +1687,7 @@ function _filter_by_referenceType(
         assert(referenceTypeId instanceof NodeId);
         const referenceType = this.addressSpace.findNode(referenceTypeId) as UAReferenceType;
         dumpIf(!referenceType, referenceTypeId);
-        // istanbul ignore next
+        // c8 ignore next
         if (!referenceType || referenceType.nodeClass !== NodeClass.ReferenceType) {
             throw new Error("Cannot find reference type");
         }
@@ -1698,7 +1698,7 @@ function _filter_by_referenceType(
 
         references = references.filter((reference) => {
             const ref = resolveReferenceType(this.addressSpace, reference)!;
-            // istanbul ignore next
+            // c8 ignore next
             if (!ref) {
                 throw new Error("Cannot find reference type " + reference.toString());
             }
@@ -1775,13 +1775,13 @@ function _filter_by_userFilter(this: BaseNode, references: UAReference[], contex
     const addressSpace = this.addressSpace;
     return references.filter((reference: UAReference) => {
         const obj = resolveReferenceNode(addressSpace, reference) as BaseNode;
-        // istanbul ignore next
+        // c8 ignore next
         if (!obj) {
             return false;
         }
 
         const _private = BaseNode_getPrivate(obj);
-        // istanbul ignore next
+        // c8 ignore next
         if (!_private._browseFilter) {
             throw Error("Internal error : cannot find browseFilter");
         }

package/src/base_node_private.ts

@@ -506,7 +506,7 @@ function _clone_children_on_template(
             : null;
     if (!nodeToCloneTypeDefinition) return;
 
-    // istanbul ignore next
+    // c8 ignore next
     doTrace &&
         traceLog(
             extraInfo?.pad(),
@@ -533,7 +533,7 @@ function _clone_children_on_template(
 
     let typeDefinitionNode: UAVariableType | UAObjectType | null = nodeToCloneTypeDefinition;
     while (typeDefinitionNode) {
-        // istanbul ignore next
+        // c8 ignore next
         doTrace &&
             traceLog(
                 extraInfo?.pad(),
@@ -564,7 +564,7 @@ function _clone_children_on_template(
                 const hasAlready = newParentChild.getChildByName(grandChild.browseName) !== null;
                 if (!hasAlready) {
                     if (optionalFilter && node && !optionalFilter.shouldKeep(node)) {
-                        // istanbul ignore next
+                        // c8 ignore next
                         doTrace &&
                             traceLog(
                                 extraInfo.pad(),
@@ -635,7 +635,7 @@ function _clone_collection_new(
         // ensure node is of the correct type,
         // it may happen that the xml nodeset2 file was malformed
 
-        // istanbul ignore next
+        // c8 ignore next
         if (typeof (node as any).clone !== "function") {
             // tslint:disable-next-line:no-console
             warningLog(
@@ -649,14 +649,14 @@ function _clone_collection_new(
             continue;
         }
         if (node && !node.modellingRule) {
-            // istanbul ignore next
+            // c8 ignore next
             doTrace && traceLog(extraInfo.pad(), "skipping  with no modelling rule", fullPath2(node));
             // those node stays in the Type
             continue; // skip this node
         }
 
         if (optionalFilter && node && !optionalFilter.shouldKeep(node)) {
-            // istanbul ignore next
+            // c8 ignore next
             doTrace && traceLog(extraInfo.pad(), "skipping optional that doesn't appear in the filter", fullPath2(node));
             continue; // skip this node
         }
@@ -687,7 +687,7 @@ function _clone_collection_new(
 
         const alreadyCloned = extraInfo.getCloned({ originalParent: nodeToClone, clonedParent: newParent, originalNode: node });
         if (alreadyCloned) {
-            // istanbul ignore next
+            // c8 ignore next
             doTrace &&
                 traceLog(
                     extraInfo.pad(),
@@ -710,11 +710,11 @@ function _clone_collection_new(
                     nodeId: newParent.nodeId
                 });
             } else {
-                // istanbul ignore next
+                // c8 ignore next
                 doTrace && traceLog(extraInfo.pad(), "reference to node  ", fullPath2(alreadyCloned), " already exists !");
             }
         } else {
-            // istanbul ignore next
+            // c8 ignore next
             doTrace &&
                 traceLog(
                     extraInfo.pad(),
@@ -729,7 +729,7 @@ function _clone_collection_new(
             const clonedNode = node.clone(options, optionalFilter, extraInfo);
             extraInfo.level -= 1;
 
-            // istanbul ignore next
+            // c8 ignore next
             doTrace &&
                 traceLog(
                     extraInfo.pad(),
@@ -794,7 +794,7 @@ function _extractInterfaces2(typeDefinitionNode: UAObjectType | UAVariableType,
 
     const baseInterfaces: UAInterface[] = [];
     for (const iface of interfaces) {
-        // istanbul ignore next
+        // c8 ignore next
         doTrace &&
             traceLog(
                 extraInfo.pad(),
@@ -811,7 +811,7 @@ function _extractInterfaces2(typeDefinitionNode: UAObjectType | UAVariableType,
     }
     interfaces.push(...baseInterfaces);
     if (typeDefinitionNode.subtypeOfObj) {
-        // istanbul ignore next
+        // c8 ignore next
         doTrace &&
             traceLog(
                 extraInfo.pad(),
@@ -825,7 +825,7 @@ function _extractInterfaces2(typeDefinitionNode: UAObjectType | UAVariableType,
     }
     const deduplicatedInterfaces = [...new Set(interfaces)];
 
-    // istanbul ignore next
+    // c8 ignore next
     doTrace &&
         deduplicatedInterfaces.length &&
         traceLog(
@@ -845,7 +845,7 @@ function _cloneInterface(
     extraInfo: CloneExtraInfo,
     browseNameMap: Set<string>
 ): void {
-    // istanbul ignore next
+    // c8 ignore next
     doTrace &&
         traceLog(
             extraInfo?.pad(),
@@ -864,14 +864,14 @@ function _cloneInterface(
     }
     const interfaces = _extractInterfaces2(typeDefinitionNode, extraInfo);
     if (interfaces.length === 0) {
-        // istanbul ignore next
+        // c8 ignore next
         doTrace &&
             false &&
             traceLog(extraInfo.pad(), chalk.yellow("No interface for ", node.browseName.toString(), node.nodeId.toString()));
         return;
     }
 
-    // istanbul ignore next
+    // c8 ignore next
     doTrace && traceLog(extraInfo?.pad(), chalk.green("-------------------- interfaces are  ", interfaces.length));
 
     const localFilter = optionalFilter.filterFor(node);
@@ -879,7 +879,7 @@ function _cloneInterface(
     for (const iface of interfaces) {
         const aggregates = iface.findReferencesEx("Aggregates", BrowseDirection.Forward);
         if (aggregates.length === 0) continue;
-        // istanbul ignore next
+        // c8 ignore next
         doTrace &&
             traceLog(
                 extraInfo.pad(),
@@ -1043,7 +1043,7 @@ export function _clone<T extends UAObject | UAVariable | UAMethod>(
             });
 
             while (typeDefinitionNode) {
-                // istanbul ignore next
+                // c8 ignore next
                 doTrace &&
                     traceLog(
                         extraInfo?.pad(),
@@ -1227,14 +1227,14 @@ export function BaseNode_add_backward_reference(this: BaseNodeImpl, reference: U
     const _private = BaseNode_getPrivate(this);
     const h = (<ReferenceImpl>reference).hash;
     assert(typeof h === "string");
-    // istanbul ignore next
+    // c8 ignore next
     if (_private._referenceIdx.has(h)) {
         //  the reference exists already in the forward references
         //  this append for instance when the XML NotSetFile has redundant <UAReference>
         //  in this case there is nothing to do
         return;
     }
-    // istanbul ignore next
+    // c8 ignore next
     if (_private._back_referenceIdx.has(h)) {
         const opts = { addressSpace: this.addressSpace };
         warningLog(" Warning !", this.browseName.toString());

package/src/data_access/ua_multistate_value_discrete_impl.ts

@@ -216,7 +216,7 @@ export class UAMultiStateValueDiscreteImpl<T, DT extends DataType>
         // MultiStateValueDiscrete Variables can have any numeric Data Type;
         // this includes signed and unsigned integers from 8 to 64 Bit length.
 
-        // istanbul ignore next
+        // c8 ignore next
         validateIsNumericDataType(this.dataType.value);
 
         // find the enum value type

package/src/data_access/ua_two_state_discrete_impl.ts

@@ -38,10 +38,11 @@ export class UATwoStateDiscreteImpl extends UAVariableImpl implements UATwoState
         const handler = this.handle_semantic_changed.bind(this);
 
         const falseState = this.getPropertyByName("FalseState");
-        /* istanbul ignore else */
         if (falseState) {
             falseState.on("value_changed", handler);
-        } else {
+        }
+        /* c8 ignore next */
+        else {
             console.warn(
                 "warning: UATwoStateDiscrete -> a FalseState property is mandatory ",
                 this.browseName.toString(),
@@ -49,10 +50,11 @@ export class UATwoStateDiscreteImpl extends UAVariableImpl implements UATwoState
             );
         }
         const trueState = this.getPropertyByName("TrueState");
-        /* istanbul ignore else */
         if (trueState) {
             trueState.on("value_changed", handler);
-        } else {
+        }
+        /* c8 ignore next */
+        else {
             console.warn(
                 "waring: UATwoStateDiscrete -> a TrueState property is mandatory",
                 this.browseName.toString(),

package/src/extension_object_array_node.ts

@@ -83,7 +83,7 @@ export function createExtObjArrayNode<T extends ExtensionObject>(parentFolder: U
     const namespace = parentFolder.namespace;
 
     const complexVariableType = addressSpace.findVariableType(options.complexVariableType);
-    // istanbul ignore next
+    // c8 ignore next
     if (!complexVariableType) {
         throw new Error("cannot find complex variable type");
     }
@@ -100,7 +100,7 @@ export function createExtObjArrayNode<T extends ExtensionObject>(parentFolder: U
 
     const dataType = addressSpace.findDataType(variableType.dataType);
 
-    // istanbul ignore next
+    // c8 ignore next
     if (!dataType) {
         errorLog(variableType.toString());
         throw new Error("cannot find Data Type");
@@ -149,19 +149,19 @@ export function bindExtObjArrayNode<T extends ExtensionObject>(
     const addressSpace = uaArrayVariableNode.addressSpace;
 
     const variableType = addressSpace.findVariableType(variableTypeNodeId);
-    // istanbul ignore next
+    // c8 ignore next
     if (!variableType || variableType.nodeId.isEmpty()) {
         throw new Error("Cannot find VariableType " + variableTypeNodeId.toString());
     }
 
     const structure = addressSpace.findDataType("Structure");
-    // istanbul ignore next
+    // c8 ignore next
     if (!structure) {
         throw new Error("Structure Type not found: please check your nodeset file");
     }
 
     let dataType = addressSpace.findDataType(variableType.dataType);
-    // istanbul ignore next
+    // c8 ignore next
     if (!dataType) {
         throw new Error("Cannot find DataType " + variableType.dataType.toString());
     }
@@ -272,7 +272,7 @@ export function removeElement<T extends ExtensionObject>(
     assert(element, "removeElement: element must exist");
     const _array = uaArrayVariableNode.$$extensionObjectArray;
 
-    // istanbul ignore next
+    // c8 ignore next
     if (_array.length === 0) {
         throw new Error(" cannot remove an element from an empty array ");
     }
@@ -295,7 +295,7 @@ export function removeElement<T extends ExtensionObject>(
         throw new Error("Unsupported anymore!!! please use a functor instead");
     }
 
-    // istanbul ignore next
+    // c8 ignore next
     if (elementIndex < 0) {
         throw new Error("removeElement: cannot find element matching " + element.toString());
     }

package/src/historical_access/address_space_historical_data_node.ts

@@ -8,7 +8,7 @@ import chalk from "chalk";
 import { assert } from "node-opcua-assert";
 import { AccessLevelFlag, NodeClass, QualifiedNameLike } from "node-opcua-data-model";
 import { DataValue } from "node-opcua-data-value";
-import { isMinDate } from "node-opcua-date-time";
+import { isMinDate, minDate } from "node-opcua-date-time";
 import { UAHistoricalDataConfiguration } from "node-opcua-nodeset-ua";
 import { NumericRange } from "node-opcua-numeric-range";
 import {
@@ -221,6 +221,10 @@ function _update_startOfOnlineArchive(this: UAVariableImpl, newDate: Date): void
     if (!this.$historicalDataConfiguration) {
         throw new Error("this variable has no HistoricalDataConfiguration");
     }
+    newDate = newDate || minDate;
+    if (!newDate) {
+        throw new Error("newDate must be provided");
+    }
 
     // The StartOfArchive Variable specifies the date before which there is no data
     // in the archive either online or offline.
@@ -237,7 +241,8 @@ function _update_startOfOnlineArchive(this: UAVariableImpl, newDate: Date): void
         (startOfArchiveDataValue.statusCode.isNotGood() ||
             !startOfArchiveDataValue.value ||
             !startOfArchiveDataValue.value.value ||
-            startOfArchiveDataValue.value.value.getTime() >= newDate.getTime())
+            !(startOfArchiveDataValue.value.value instanceof Date) ||
+            (startOfArchiveDataValue.value.value as Date).getTime() >= newDate.getTime())
     ) {
         this._update_startOfArchive(newDate);
     }
@@ -429,12 +434,12 @@ function _historyReadRaw(
 
     const session = context.session;
 
-    // istanbul ignore next
+    // c8 ignore next
     if (!session) {
         throw new Error("Internal Error: context.session not defined");
     }
 
-    // istanbul ignore next
+    // c8 ignore next
     if (!session.continuationPointManager) {
         throw new Error("Internal Error: context.session.continuationPointManager not defined");
     }

package/src/namespace_impl.ts

@@ -131,7 +131,7 @@ function _makeHashKey(nodeId: NodeId): string | number {
         case NodeIdType.NUMERIC:
             return nodeId.value as number;
         default:
-            // istanbul ignore next
+            // c8 ignore next
             if (nodeId.identifierType !== NodeIdType.BYTESTRING) {
                 throw new Error("invalid nodeIdType");
             }
@@ -238,15 +238,15 @@ export class NamespaceImpl implements NamespacePrivate {
     private defaultRolePermissions?: RolePermissionType[];
 
     constructor(options: NamespaceConstructorOptions) {
-        // istanbul ignore next
+        // c8 ignore next
         if (!(typeof options.namespaceUri === "string")) {
             throw new Error("NamespaceImpl constructor: namespaceUri must exists and be a string : got " + options.namespaceUri);
         }
-        // istanbul ignore next
+        // c8 ignore next
         if (typeof options.index !== "number") {
             throw new Error("NamespaceImpl constructor: index must be a number");
         }
-        // istanbul ignore next
+        // c8 ignore next
         if (!options.addressSpace) {
             throw new Error("NamespaceImpl constructor: Must specify a valid address space");
         }
@@ -689,7 +689,7 @@ export class NamespaceImpl implements NamespacePrivate {
         if (nodeOrNodeId instanceof NodeId) {
             nodeId = nodeOrNodeId;
             node = this.findNode(nodeId);
-            // istanbul ignore next
+            // c8 ignore next
             if (!node) {
                 throw new Error(" deleteNode : cannot find node with nodeId" + nodeId.toString());
             }
@@ -697,7 +697,7 @@ export class NamespaceImpl implements NamespacePrivate {
             node = nodeOrNodeId;
             nodeId = node.nodeId;
         }
-        // istanbul ignore next
+        // c8 ignore next
         if (nodeId.namespace !== this.index) {
             throw new Error("this node doesn't belong to this namespace");
         }
@@ -705,7 +705,7 @@ export class NamespaceImpl implements NamespacePrivate {
         const addressSpace = this.addressSpace;
 
         addressSpace.modelChangeTransaction(() => {
-            /* istanbul ignore next */
+            /* c8 ignore next */
             if (!node) {
                 throw new Error("this node doesn't belong to this namespace");
             }
@@ -1473,7 +1473,7 @@ export class NamespaceImpl implements NamespacePrivate {
 
         const fromStateNode = _component.getComponentByName(fromState);
 
-        // istanbul ignore next
+        // c8 ignore next
         if (!fromStateNode) {
             throw new Error("Cannot find state with name " + fromState);
         }
@@ -1481,7 +1481,7 @@ export class NamespaceImpl implements NamespacePrivate {
 
         const toStateNode = _component.getComponentByName(toState);
 
-        // istanbul ignore next
+        // c8 ignore next
         if (!toStateNode) {
             throw new Error("Cannot find state with name " + toState);
         }
@@ -1639,7 +1639,7 @@ export class NamespaceImpl implements NamespacePrivate {
     public _register(node: BaseNode): void {
         assert(node instanceof BaseNodeImpl, "Expecting a instance of BaseNode in _register");
         assert(node.nodeId instanceof NodeId, "Expecting a NodeId");
-        // istanbul ignore next
+        // c8 ignore next
         if (node.nodeId.namespace !== this.index) {
             throw new Error(
                 "node must belong to this namespace : " +
@@ -1656,7 +1656,7 @@ export class NamespaceImpl implements NamespacePrivate {
 
         const hashKey = _makeHashKey(node.nodeId);
 
-        // istanbul ignore next
+        // c8 ignore next
         if (this._nodeid_index.has(hashKey)) {
             const existingNode = this.findNode(node.nodeId)!;
             throw new Error(
@@ -1784,7 +1784,7 @@ export class NamespaceImpl implements NamespacePrivate {
         this._register(node);
 
         // object shall now be registered
-        // istanbul ignore next
+        // c8 ignore next
         if (doDebug) {
             assert(this.findNode(node.nodeId) !== null && typeof this.findNode(node.nodeId) === "object");
         }
@@ -1795,7 +1795,7 @@ export class NamespaceImpl implements NamespacePrivate {
         assert(node instanceof BaseNodeImpl);
 
         const hashKey = _makeHashKey(node.nodeId);
-        // istanbul ignore next
+        // c8 ignore next
         if (!this._nodeid_index.has(hashKey)) {
             throw new Error("deleteNode : nodeId " + node.nodeId.displayText() + " is not registered " + node.nodeId.toString());
         }
@@ -1958,14 +1958,14 @@ export class NamespaceImpl implements NamespacePrivate {
 
         options.historizing = !!options.historizing;
 
-        // istanbul ignore next
+        // c8 ignore next
         if (Object.prototype.hasOwnProperty.call(options, "hasTypeDefinition")) {
             throw new Error("hasTypeDefinition option is invalid. Do you mean typeDefinition instead ?");
         }
         // ------------------------------------------ TypeDefinition
         let typeDefinition = options.typeDefinition || baseDataVariableTypeId;
         if (typeDefinition instanceof BaseNodeImpl) {
-            // istanbul ignore next
+            // c8 ignore next
             if (typeDefinition.nodeClass !== NodeClass.VariableType) {
                 const message = `invalid typeDefinition expecting a VariableType got ${NodeClass[typeDefinition.nodeClass]}`;
                 errorLog(message);
@@ -1993,7 +1993,7 @@ export class NamespaceImpl implements NamespacePrivate {
             return typeof options.value?.get === "function" || typeof options.value?.timestamped_get === "function";
         };
 
-        // istanbul ignore next
+        // c8 ignore next
         if (options.minimumSamplingInterval === undefined && hasGetter(options)) {
             // a getter has been specified and no options.minimumSamplingInterval has been specified
             warningLog(
@@ -2006,7 +2006,7 @@ export class NamespaceImpl implements NamespacePrivate {
 
         options.minimumSamplingInterval = options.minimumSamplingInterval !== undefined ? +options.minimumSamplingInterval : 0;
 
-        // istanbul ignore next
+        // c8 ignore next
         if (options.minimumSamplingInterval === 0 && hasGetter(options)) {
             warningLog(
                 "[NODE-OPCUA-W31",
@@ -2239,7 +2239,7 @@ function _create_node_version_if_needed(node: BaseNode, options: { nodeVersion?:
     assert(options);
     if (typeof options.nodeVersion == "string") {
         assert(node.nodeClass === NodeClass.Variable || node.nodeClass === NodeClass.Object);
-        // istanbul ignore next
+        // c8 ignore next
         if (node.getNodeVersion()) {
             return; // already exists
         }

package/src/nodeid_manager.ts

@@ -59,7 +59,7 @@ function _findParentNodeId(addressSpace: AddressSpacePartial, options: Construct
     }
     for (const ref of options.references) {
         (ref as any)._referenceType = addressSpace.findReferenceType(ref.referenceType);
-        /* istanbul ignore next */
+        /* c8 ignore next */
         if (!getReferenceType(ref)) {
             throw new Error("Cannot find referenceType " + JSON.stringify(ref));
         }