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

Add aws_iam_group #1477

Merged
merged 1 commit into from
May 13, 2022
Merged

Add aws_iam_group #1477

merged 1 commit into from
May 13, 2022

Conversation

sundowndev-snyk
Copy link
Contributor

@sundowndev-snyk sundowndev-snyk commented Apr 29, 2022

Description

This PR adds support for the aws_iam_group resource.

Docs PR snyk/driftctl-docs#239

@sundowndev-snyk sundowndev-snyk added the kind/enhancement New feature or improvement label Apr 29, 2022
@sundowndev-snyk sundowndev-snyk force-pushed the feat/add_aws_iam_group branch 3 times, most recently from d03b6f3 to c55e8e9 Compare May 5, 2022 15:47
@sundowndev-snyk sundowndev-snyk marked this pull request as ready for review May 5, 2022 15:50
@sundowndev-snyk sundowndev-snyk requested a review from a team as a code owner May 5, 2022 15:50
@sundowndev-snyk sundowndev-snyk requested review from almog27 and removed request for a team May 5, 2022 15:50
Copy link
Contributor

@eliecharra eliecharra left a comment

Choose a reason for hiding this comment

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

There is some issues with the acceptance test, the diff said that you renamed pkg/resource/aws/testdata/acc/aws_iam_group_policy/.terraform.lock.hcl to pkg/resource/aws/testdata/acc/aws_iam_group/.terraform.lock.hcl. We should keep the lock for those two tests.

@codecov-commenter
Copy link

Codecov Report

Merging #1477 (6ad889c) into main (c5ed5b9) will increase coverage by 6.99%.
The diff coverage is 96.29%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1477      +/-   ##
==========================================
+ Coverage   75.07%   82.06%   +6.99%     
==========================================
  Files         512      438      -74     
  Lines       19195    15907    -3288     
==========================================
- Hits        14410    13054    -1356     
+ Misses       4476     2537    -1939     
- Partials      309      316       +7     
Impacted Files Coverage Δ
pkg/remote/aws/init.go 0.00% <0.00%> (-95.99%) ⬇️
pkg/resource/resource_types.go 100.00% <ø> (ø)
pkg/remote/aws/iam_group_enumerator.go 100.00% <100.00%> (ø)
pkg/remote/aws/iam_group_policy_enumerator.go 100.00% <100.00%> (ø)
pkg/terraform/plugin_client.go 0.00% <0.00%> (-100.00%) ⬇️
pkg/remote/aws/client/s3_client_factory.go 0.00% <0.00%> (-100.00%) ⬇️
pkg/resource/azurerm/azurerm_container_registry.go 12.50% <0.00%> (-87.50%) ⬇️
pkg/resource/google/google_compute_address.go 14.28% <0.00%> (-85.72%) ⬇️
...g/resource/google/google_compute_global_address.go 14.28% <0.00%> (-85.72%) ⬇️
... and 175 more

@eliecharra eliecharra merged commit bc58a4e into main May 13, 2022
@eliecharra eliecharra deleted the feat/add_aws_iam_group branch May 13, 2022 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement New feature or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants