Skip to content

Conversation

@noursaidi
Copy link
Owner

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented May 19, 2021

Codecov Report

Merging #16 (ada8538) into master (cd59227) will increase coverage by 1.47%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
+ Coverage   81.66%   83.13%   +1.47%     
==========================================
  Files          43       42       -1     
  Lines        5273     5942     +669     
==========================================
+ Hits         4306     4940     +634     
- Misses        967     1002      +35     
Flag Coverage Δ
ata 62.95% <16.66%> (?)
aux 73.59% <100.00%> (+4.99%) ⬆️
base 71.74% <16.66%> (+5.31%) ⬆️
dhcp 73.27% <16.66%> (+5.54%) ⬆️
many 73.15% <16.66%> (+5.27%) ⬆️
mud 77.39% <16.66%> (+4.44%) ⬆️
switch 71.89% <16.66%> (+5.15%) ⬆️
topo 72.01% <16.66%> (+5.17%) ⬆️
unit ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
daq/report.py 98.39% <100.00%> (+0.02%) ⬆️
daq/device_report_client.py 48.48% <0.00%> (-51.52%) ⬇️
daq/session_server.py 31.96% <0.00%> (-46.73%) ⬇️
daq/proto/session_server_pb2_grpc.py 57.89% <0.00%> (-21.06%) ⬇️
daq/acl_state_collector.py 73.97% <0.00%> (-9.59%) ⬇️
daq/configurator.py 88.80% <0.00%> (-0.80%) ⬇️
daq/base_gateway.py 91.90% <0.00%> (-0.26%) ⬇️
daq/proto/system_config_pb2.py 100.00% <0.00%> (ø)
daq/test_modules/base_module.py 100.00% <0.00%> (ø)
daq/test_modules/docker_module.py 100.00% <0.00%> (ø)
... and 11 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 cd59227...ada8538. Read the comment docs.

noursaidi added 4 commits May 19, 2021 23:57
Merge branch 'ntpdhcpanddns' of github.com:noursaidi/daq into ntpdhcpanddns
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants