node-opcua / node-opcua / 33619949964

92%

Build Type

push

github

Committed by erossignon

Commit Message

refactor(address-space): UAVariableImplT states that it satisfies UAVariableT

It overrides readValue, readValueAsync and writeValue to narrow them to the
value type, but never said which interface those overrides were narrowing
towards, so nothing checked them. That is where the arity-forwarding bug lived.

UAVariableImpl itself stays as it is: its type parameter is the event map, not
the value type, so the generic pair belongs on the T variant.

Coverage Stats

22350 of 26319 branches covered (84.92%)

2 of 2 new or added lines in 1 file covered. (100.0%)

340 existing lines in 16 files now uncovered.

182778 of 197973 relevant lines covered (92.32%)

442346.2 hits per line