hazelcast / hazelcast-nodejs-client

Hazelcast Node.js Client
https://hazelcast.com/clients/node-js/
Apache License 2.0
150 stars 59 forks source link

Compact serialization codecov test results issues[API-1791] #1451

Closed harunalpak closed 1 year ago

harunalpak commented 1 year ago

Some unused methods removed, typo fixes.

And 2 test were written to check throwing error when field is undefined or field type not matched.

codecov[bot] commented 1 year ago

Codecov Report

Merging #1451 (3b3d0e3) into master (7c4f088) will increase coverage by 0.03%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1451      +/-   ##
==========================================
+ Coverage   93.38%   93.41%   +0.03%     
==========================================
  Files         466      466              
  Lines       16623    16622       -1     
  Branches     1351     1350       -1     
==========================================
+ Hits        15523    15528       +5     
+ Misses        799      797       -2     
+ Partials      301      297       -4     
Impacted Files Coverage Δ
src/serialization/compact/Schema.ts 78.57% <ø> (ø)
...rialization/generic_record/CompactGenericRecord.ts 97.26% <ø> (+1.31%) :arrow_up:
src/serialization/compact/DefaultCompactWriter.ts 100.00% <100.00%> (+2.19%) :arrow_up:
src/network/ConnectionManager.ts 83.99% <0.00%> (-1.17%) :arrow_down:
src/invocation/InvocationService.ts 94.87% <0.00%> (-0.37%) :arrow_down:
src/util/Util.ts 87.67% <0.00%> (+0.68%) :arrow_up:
src/proxy/NearCachedMapProxy.ts 94.44% <0.00%> (+1.38%) :arrow_up:
src/nearcache/NearCache.ts 93.79% <0.00%> (+1.55%) :arrow_up:
...rc/serialization/generic_record/FieldDescriptor.ts 100.00% <0.00%> (+16.66%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more