Test to validate client publish request count

[mregen]

fixes #1455
Test validates that client handles the BadTooManyPublishrequest properly and lowers number of active requests if necessary.

[lgtm-com]

This pull request introduces 1 alert when merging cedb6bf into 92ae1d3 - view on LGTM.com

new alerts:

• 1 for Useless assignment to local variable

[codecov]

Codecov Report

Merging #1516 (b5a2157) into master (92ae1d3) will increase coverage by 0.69%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1516      +/-   ##
==========================================
+ Coverage   52.17%   52.87%   +0.69%     
==========================================
  Files         307      307              
  Lines       58370    58489     +119     
==========================================
+ Hits        30455    30925     +470     
+ Misses      27915    27564     -351     

Impacted Files	Coverage Δ
Libraries/Opc.Ua.Client/Session.cs	61.56% <0.00%> (-1.09%)	⬇️
Libraries/Opc.Ua.Client/Subscription.cs	69.58% <0.00%> (-0.73%)	⬇️
Stack/Opc.Ua.Core/Stack/State/NodeState.cs	39.10% <0.00%> (+0.20%)	⬆️
...braries/Opc.Ua.Server/Subscription/Subscription.cs	56.64% <0.00%> (+0.22%)	⬆️
...raries/Opc.Ua.Server/Subscription/MonitoredItem.cs	44.39% <0.00%> (+0.30%)	⬆️
Stack/Opc.Ua.Core/Stack/Tcp/TcpMessageSocket.cs	73.72% <0.00%> (+0.34%)	⬆️
...tack/Opc.Ua.Core/Stack/Tcp/TcpTransportListener.cs	72.54% <0.00%> (+0.39%)	⬆️
...ack/Opc.Ua.Core/Stack/Tcp/ChannelAsyncOperation.cs	68.75% <0.00%> (+1.04%)	⬆️
Stack/Opc.Ua.Core/Stack/State/NodeBrowser.cs	78.04% <0.00%> (+1.21%)	⬆️
Stack/Opc.Ua.Core/Stack/Tcp/UaSCBinaryChannel.cs	69.36% <0.00%> (+1.70%)	⬆️
... and 8 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 92ae1d3...b5a2157. Read the comment docs.

[lgtm-com]

This pull request introduces 1 alert when merging 9995d2a into 5048189 - view on LGTM.com

new alerts:

• 1 for Useless assignment to local variable

[lgtm-com]

This pull request introduces 1 alert when merging bdfe340 into 5048189 - view on LGTM.com

new alerts:

• 1 for Useless assignment to local variable

[mregen]

The SequentialPublishing Test is still a bit flaky...

[mregen]

Reinvestigate the flaky test cases...

[lgtm-com]

This pull request introduces 1 alert when merging b5a2157 into 7d310d5 - view on LGTM.com

new alerts:

• 1 for Useless assignment to local variable