hazelcast / hazelcast-nodejs-client

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

Fix out of date serializers API-1115 #1197

Closed srknzl closed 2 years ago

srknzl commented 2 years ago

I added java array serializer, fixed linkedlist serializer and arraylist serializer. Deleted write methods of some serializers since they are not used

codecov[bot] commented 2 years ago

Codecov Report

Merging #1197 (c74e3b2) into master (940dce9) will decrease coverage by 0.33%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1197      +/-   ##
==========================================
- Coverage   92.52%   92.19%   -0.34%     
==========================================
  Files         440      440              
  Lines       14556    14553       -3     
  Branches     1096     1095       -1     
==========================================
- Hits        13468    13417      -51     
- Misses        801      847      +46     
- Partials      287      289       +2     
Impacted Files Coverage Δ
src/serialization/DefaultSerializers.ts 97.93% <100.00%> (+1.97%) :arrow_up:
src/serialization/SerializationService.ts 94.93% <100.00%> (+0.03%) :arrow_up:
src/codec/ClientAuthenticationCustomCodec.ts 42.22% <0.00%> (-33.34%) :arrow_down:
src/network/ConnectionManager.ts 75.82% <0.00%> (-7.99%) :arrow_down:
src/config/ConfigBuilder.ts 90.26% <0.00%> (-1.06%) :arrow_down:
src/invocation/ClusterService.ts 94.01% <0.00%> (-0.86%) :arrow_down:
src/util/Util.ts 87.89% <0.00%> (-0.64%) :arrow_down:
src/core/HazelcastError.ts 75.20% <0.00%> (ø)
src/protocol/ErrorFactory.ts 63.90% <0.00%> (ø)
src/invocation/InvocationService.ts 95.57% <0.00%> (+1.32%) :arrow_up:

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 940dce9...c74e3b2. Read the comment docs.