Mqtt311 compliance client behavior2

[bretambrose]

• Update 311 client to set dup flag when resubmitting qos 1 publishes on session resumption
• Additional tests are client behavior related to dup flag and resubmitting subscribe/unsubscribe which the 311 spec forgot to forbid

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[codecov-commenter]

Codecov Report

Patch coverage: 98.11% and project coverage change: +0.18% 🎉

Comparison is base (6ee615f) 82.23% compared to head (a44784f) 82.42%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #326      +/-   ##
==========================================
+ Coverage   82.23%   82.42%   +0.18%     
==========================================
  Files          20       20              
  Lines        8709     8729      +20     
==========================================
+ Hits         7162     7195      +33     
+ Misses       1547     1534      -13     

Files Changed	Coverage Δ
source/v5/mqtt5_utils.c	81.28% <ø> (+1.19%)	⬆️
source/client.c	70.81% <96.15%> (+1.11%)	⬆️
source/mqtt.c	96.00% <100.00%> (+1.40%)	⬆️
source/packets.c	85.19% <100.00%> (+0.33%)	⬆️
source/v5/mqtt5_options_storage.c	90.59% <100.00%> (ø)

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.