aquasecurity / kube-hunter

Hunt for security weaknesses in Kubernetes clusters
Apache License 2.0
4.71k stars 581 forks source link

Remove dynamic imports #335

Closed iyehuda closed 4 years ago

iyehuda commented 4 years ago

Description

Replace dynamic imports with explicit imports.

Depends on #323

Contribution Guidelines

Please Read through the Contribution Guidelines.

Fixed Issues

Fixes #290

"BEFORE" and "AFTER" output

To verify that the change works as desired, please include an output of terminal before and after the changes under headings "BEFORE" and "AFTER".

BEFORE

Any Terminal Output Before Changes.

AFTER

Any Terminal Output Before Changes.

Contribution checklist

Notes

Tests are not fully made, see #334

codecov-io commented 4 years ago

Codecov Report

Merging #335 into master will increase coverage by 0.04%. The diff coverage is 95.12%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #335      +/-   ##
==========================================
+ Coverage   60.16%   60.20%   +0.04%     
==========================================
  Files          41       41              
  Lines        1968     1970       +2     
==========================================
+ Hits         1184     1186       +2     
  Misses        784      784              
Impacted Files Coverage Δ
kube_hunter/core/events/types.py 82.45% <ø> (ø)
kube_hunter/conf/parser.py 95.23% <80.00%> (-4.77%) :arrow_down:
kube_hunter/modules/discovery/hosts.py 79.83% <95.23%> (+2.44%) :arrow_up:
kube_hunter/conf/__init__.py 100.00% <100.00%> (ø)
kube_hunter/conf/logging.py 100.00% <100.00%> (ø)
kube_hunter/modules/discovery/__init__.py 100.00% <100.00%> (ø)
kube_hunter/modules/hunting/__init__.py 100.00% <100.00%> (ø)

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 124a51d...0d7604b. Read the comment docs.