Ran go fmt on test and updated bridgecrew comment as advised.

cloudposse · Apr 9, 2021 · 2afe01c · 2afe01c
1 parent 9afe9a8
commit 2afe01c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/main.tf b/main.tf
@@ -3,7 +3,7 @@ locals {
 }
 
 resource "aws_s3_bucket" "default" {
-  #bridgecrew:skip=BC_AWS_S3_13:Skipping `Enable S3 Bucket Logging` check until bridgecrew will support dynamic blocks (https://github.com/bridgecrewio/checkov/issues/776).
+  #bridgecrew:skip=BC_AWS_S3_13:Skipping `Enable S3 Bucket Logging` because we do not have good defaults
   #bridgecrew:skip=CKV_AWS_52:Skipping `Ensure S3 bucket has MFA delete enabled` due to issue in terraform (https://github.com/hashicorp/terraform-provider-aws/issues/629).
   #bridgecrew:skip=BC_AWS_S3_16:Skipping `Ensure S3 bucket versioning is enabled` because dynamic blocks are not supported by checkov
   count         = module.this.enabled ? 1 : 0

diff --git a/test/src/examples_complete_test.go b/test/src/examples_complete_test.go
@@ -104,7 +104,7 @@ func TestExamplesCompleteWithObjectLock(t *testing.T) {
 	attributes := []string{strconv.Itoa(rand.Intn(100000))}
 	rootFolder := "../../"
 	terraformFolderRelativeToRoot := "examples/complete"
-  varFiles := []string{"object-lock.us-west-1.tfvars"}
+	varFiles := []string{"object-lock.us-west-1.tfvars"}
 
 	tempTestFolder := test_structure.CopyTerraformFolderToTemp(t, rootFolder, terraformFolderRelativeToRoot)
 
@@ -129,7 +129,7 @@ func TestExamplesCompleteWithObjectLock(t *testing.T) {
 	userName := terraform.Output(t, terraformOptions, "user_name")
 
 	expectedUserName := "eg-test-s3-object-lock-test-" + attributes[0]
-  
+
 	// Verify we're getting back the outputs we expect
 	assert.Equal(t, expectedUserName, userName)