fix bug - nodeset2 export of namespace index #689

[cristipogacean]

small fix for issue #689.
this is a relatively exotic scenario, pretty difficult to replicate, therefore no tests are defined.

[codecov]

Codecov Report

Merging #1510 (3a622c7) into master (9b89bd0) will increase coverage by 0.32%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1510      +/-   ##
==========================================
+ Coverage   51.88%   52.20%   +0.32%     
==========================================
  Files         305      307       +2     
  Lines       58080    59488    +1408     
==========================================
+ Hits        30134    31055     +921     
- Misses      27946    28433     +487     

Impacted Files	Coverage Δ
Stack/Opc.Ua.Core/Schema/UANodeSetHelpers.cs	0.00% <0.00%> (ø)
...Certificates/X509Certificate/CertificateBuilder.cs	80.91% <0.00%> (-2.61%)	⬇️
...Gds.Client.Common/ServerPushConfigurationClient.cs	73.50% <0.00%> (-0.41%)	⬇️
Libraries/Opc.Ua.Server/Configuration/TrustList.cs	80.18% <0.00%> (-0.13%)	⬇️
...es/Opc.Ua.Client.ComplexTypes/ComplexTypeSystem.cs	0.00% <0.00%> (ø)
Libraries/Opc.Ua.Client/SessionAsync.cs	86.36% <0.00%> (ø)
Libraries/Opc.Ua.Client/SubscriptionAsync.cs	64.11% <0.00%> (ø)
Libraries/Opc.Ua.Client/Session.cs	61.20% <0.00%> (+0.04%)	⬆️
...raries/Opc.Ua.Server/Subscription/MonitoredItem.cs	44.09% <0.00%> (+0.30%)	⬆️
...tack/Opc.Ua.Core/Stack/Tcp/TcpTransportListener.cs	72.54% <0.00%> (+0.39%)	⬆️
... and 5 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9b89bd0...3a622c7. Read the comment docs.

[mregen]

Hi @cristipogacean , is this quite ready yet?
I only see a new commented out line, so no real change in the code?

[cristipogacean]

thanks @mregen, it was a test hick-up. it is fixed now.

[mregen]

some tests for UA Nodesethelpers would be nice...

[mrsuciu]

ok