Merge pull request #1668 from snyk/acc-tests-aws-kms-reliability

chore: don't disable acc test KMS keys
snyk · Jun 5, 2023 · 714d779 · 714d779
2 parents 7f729dc + 7eec7e2
commit 714d779
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/pkg/resource/aws/testdata/acc/aws_kms_alias/terraform.tf b/pkg/resource/aws/testdata/acc/aws_kms_alias/terraform.tf
@@ -10,7 +10,6 @@ terraform {
 
 resource "aws_kms_key" "key" {
   deletion_window_in_days = 7
-  is_enabled              = false
 }
 
 resource "aws_kms_alias" "foo" {
@@ -19,6 +18,6 @@ resource "aws_kms_alias" "foo" {
 }
 
 resource "aws_kms_alias" "baz" {
-  name_prefix          = "alias/baz"
+  name_prefix   = "alias/baz"
   target_key_id = aws_kms_key.key.key_id
 }
diff --git a/pkg/resource/aws/testdata/acc/aws_kms_key/terraform.tf b/pkg/resource/aws/testdata/acc/aws_kms_key/terraform.tf
@@ -10,23 +10,20 @@ terraform {
 
 resource "aws_kms_key" "foo" {
   description              = "Foo"
-  deletion_window_in_days = 7
-  is_enabled              = false
+  deletion_window_in_days  = 7
   customer_master_key_spec = "RSA_4096"
 }
 
 resource "aws_kms_key" "bar" {
   description              = "Bar"
-  deletion_window_in_days = 7
-  is_enabled              = false
+  deletion_window_in_days  = 7
   customer_master_key_spec = "RSA_2048"
-  key_usage = "SIGN_VERIFY"
+  key_usage                = "SIGN_VERIFY"
 }
 
 resource "aws_kms_key" "baz" {
   description             = "Baz"
   deletion_window_in_days = 7
-  is_enabled              = false
   tags = {
     "Foo" = "true"
   }