OpenMined / KotlinSyft

The official Syft worker for secure on-device machine learning
https://www.openmined.org
Apache License 2.0
85 stars 28 forks source link

Update unittest #281

Closed galalen closed 3 years ago

galalen commented 3 years ago

Description

Write unit tests for BatteryStatusRepository and update unit test for SyftWebSocket, fixes issue 123.

Affected Dependencies

None.

How has this been tested?

Run the test cases.

Checklist

codecov[bot] commented 3 years ago

Codecov Report

Merging #281 into dev will increase coverage by 0.59%. The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #281      +/-   ##
==========================================
+ Coverage   60.72%   61.31%   +0.59%     
==========================================
  Files          44       44              
  Lines        1487     1489       +2     
  Branches      195      195              
==========================================
+ Hits          903      913      +10     
+ Misses        492      484       -8     
  Partials       92       92              
Impacted Files Coverage Δ
...openmined/syft/networking/clients/SyftWebSocket.kt 79.16% <75.00%> (+0.90%) :arrow_up:
...ed/syft/monitor/battery/BatteryStatusRepository.kt 90.90% <0.00%> (+45.45%) :arrow_up:
...enmined/syft/monitor/battery/BatteryStatusModel.kt 100.00% <0.00%> (+75.00%) :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 5f0ec57...6d9d6d5. Read the comment docs.