Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactoring/aws/directconnect #222

Merged
merged 11 commits into from
Mar 10, 2019

Conversation

Aboisier
Copy link
Contributor

@Aboisier Aboisier commented Mar 5, 2019

Migrated DirectConnect. See #183 .

@codecov-io
Copy link

Codecov Report

Merging #222 into refactoring/resource-configs will increase coverage by 0.06%.
The diff coverage is 60%.

Impacted file tree graph

@@                       Coverage Diff                        @@
##           refactoring/resource-configs     #222      +/-   ##
================================================================
+ Coverage                         28.71%   28.78%   +0.06%     
================================================================
  Files                                97       98       +1     
  Lines                              5599     5611      +12     
================================================================
+ Hits                               1608     1615       +7     
- Misses                             3991     3996       +5
Impacted Files Coverage Δ
ScoutSuite/providers/aws/facade/facade.py 61.9% <100%> (+4.01%) ⬆️
ScoutSuite/providers/aws/configs/services.py 49.18% <50%> (ø) ⬆️
...e/providers/aws/resources/directconnect/service.py 52.94% <52.94%> (ø)
ScoutSuite/providers/aws/facade/directconnect.py 75% <75%> (ø)

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 c577278...0e7e5ce. Read the comment docs.

@codecov-io
Copy link

codecov-io commented Mar 5, 2019

Codecov Report

Merging #222 into refactoring/resource-configs will increase coverage by 0.07%.
The diff coverage is 59.25%.

Impacted file tree graph

@@                       Coverage Diff                       @@
##           refactoring/resource-configs    #222      +/-   ##
===============================================================
+ Coverage                         29.63%   29.7%   +0.07%     
===============================================================
  Files                               100     101       +1     
  Lines                              5312    5326      +14     
===============================================================
+ Hits                               1574    1582       +8     
- Misses                             3738    3744       +6
Impacted Files Coverage Δ
ScoutSuite/providers/aws/facade/facade.py 66.66% <100%> (+3.03%) ⬆️
ScoutSuite/providers/aws/configs/services.py 49.18% <50%> (ø) ⬆️
...e/providers/aws/resources/directconnect/service.py 52.94% <52.94%> (ø)
ScoutSuite/providers/aws/facade/directconnect.py 66.66% <66.66%> (ø)

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 64be4fe...911ed6c. Read the comment docs.

Copy link
Contributor

@misg misg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job! Although, we need to make directconnect facade async (see #224 for context and details), I left a couple of suggestions to do so.

Copy link
Contributor

@Remi05 Remi05 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! 👌

@Aboisier Aboisier merged commit de4d46b into refactoring/resource-configs Mar 10, 2019
@Aboisier Aboisier deleted the refactoring/aws/directconnect branch March 10, 2019 21:11
@Aboisier Aboisier mentioned this pull request Mar 13, 2019
53 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants