openweave / openweave-core

openWeave is a home area network application protocol stack designed to enable asynchronous, symmetric, device-to-device, device-to-mobile and device-to-cloud communications for control path and data path messaging.
Apache License 2.0
232 stars 106 forks source link

Fix TLVreader context tag err #599

Closed didishe90 closed 4 years ago

didishe90 commented 4 years ago

Should use AnonymousTag to read out shared session endnode.

codecov-commenter commented 4 years ago

Codecov Report

Merging #599 into master will increase coverage by 15.74%. The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #599       +/-   ##
===========================================
+ Coverage   54.16%   69.91%   +15.74%     
===========================================
  Files         329      483      +154     
  Lines       57161    86731    +29570     
===========================================
+ Hits        30962    60635    +29673     
+ Misses      26199    26096      -103     
Impacted Files Coverage Δ
src/lib/core/WeaveFabricState.cpp 62.56% <0.00%> (+24.23%) :arrow_up:
src/test-apps/schema/nest/test/trait/TestETrait.h 55.55% <0.00%> (-44.45%) :arrow_down:
src/lib/profiles/time/WeaveTime.h 60.00% <0.00%> (-40.00%) :arrow_down:
src/lib/support/logging/WeaveLogging.cpp 56.86% <0.00%> (-19.33%) :arrow_down:
...i/security-support/native/WeaveSecuritySupport.cpp 57.89% <0.00%> (-15.44%) :arrow_down:
src/lib/profiles/security/WeaveSig.cpp 87.25% <0.00%> (-5.55%) :arrow_down:
src/lib/profiles/security/WeaveCASEEngine.cpp 92.80% <0.00%> (-0.53%) :arrow_down:
src/lib/core/WeaveTLV.h 100.00% <0.00%> (ø)
src/test-apps/ToolCommon.h 100.00% <0.00%> (ø)
src/lib/core/WeaveTLVTypes.h 100.00% <0.00%> (ø)
... and 265 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 2468803...38b5050. Read the comment docs.