Merge pull request #28 from masterpointio/feat/testing

feat: native Terraform test framework

Author: oycyc

tests/.gitignore

@@ -1,13 +1,13 @@
 variables {
-  vpc_id = "vpc-12345678"
-  subnet_ids = ["subnet-12345678", "subnet-87654321"]
-  stage = "test"
-  namespace = "mp"
-  name = "ssm-agent"
-  region = "us-east-1"
-  availability_zones = ["us-east-1a"]
+  vpc_id              = "vpc-12345678"
+  subnet_ids          = ["subnet-12345678", "subnet-87654321"]
+  stage               = "test"
+  namespace           = "mp"
+  name                = "ssm-agent"
+  region              = "us-east-1"
+  availability_zones  = ["us-east-1a"]
   nat_gateway_enabled = true
-  ipv6_enabled = true
+  ipv6_enabled        = true
 }
 
 ### TESTING INSTANCE and ARCHITECTURE COMPATIBILITY ###
@@ -20,7 +20,7 @@ run "valid_x86_64_instance" {
 
   variables {
     instance_type = "t3.micro"
-    architecture = "x86_64"
+    architecture  = "x86_64"
   }
 
   assert {
@@ -35,7 +35,7 @@ run "valid_arm64_instance" {
 
   variables {
     instance_type = "t4g.micro"
-    architecture = "arm64"
+    architecture  = "arm64"
   }
 
   assert {
@@ -50,7 +50,7 @@ run "invalid_x86_64_instance" {
 
   variables {
     instance_type = "t4g.micro"
-    architecture = "x86_64"
+    architecture  = "x86_64"
   }
 
   expect_failures = [
@@ -64,7 +64,7 @@ run "invalid_arm64_instance" {
 
   variables {
     instance_type = "t3.micro"
-    architecture = "arm64"
+    architecture  = "arm64"
   }
 
   expect_failures = [
@@ -79,7 +79,7 @@ run "graphics_instance_arm_incompatiblity_edge_case" {
 
   variables {
     instance_type = "g3s.xlarge"
-    architecture = "arm64"
+    architecture  = "arm64"
   }
 
   expect_failures = [
@@ -94,7 +94,7 @@ run "graphics_instance_x86_compatibility_edge_case" {
 
   variables {
     instance_type = "g4dn.xlarge"
-    architecture = "x86_64"
+    architecture  = "x86_64"
   }
 
   assert {

tests/Gopkg.lock

@@ -0,0 +1,46 @@
+variables {
+  vpc_id              = "vpc-12345678"
+  subnet_ids          = ["subnet-12345678", "subnet-87654321"]
+  stage               = "test"
+  namespace           = "mp"
+  name                = "ssm-agent"
+  region              = "us-east-1"
+  availability_zones  = ["us-east-1a"]
+  nat_gateway_enabled = true
+  ipv6_enabled        = true
+}
+
+run "verify_session_logging" {
+  command = plan
+
+  variables {
+    session_logging_enabled            = true
+    session_logging_bucket_name        = ""
+    session_logging_encryption_enabled = true
+    cloudwatch_retention_in_days       = 365
+  }
+
+  assert {
+    condition     = aws_cloudwatch_log_group.session_logging[0].retention_in_days == 365
+    error_message = "CloudWatch log retention days not set correctly when variables passed in."
+  }
+
+  assert {
+    condition     = length(aws_iam_role_policy.session_logging) > 0
+    error_message = "Session logging IAM policy not created when variables passed in."
+  }
+}
+
+run "verify_session_logging_bucket_logic" {
+  command = plan
+
+  variables {
+    session_logging_enabled     = true
+    session_logging_bucket_name = "" # Empty name should trigger bucket creation
+  }
+
+  assert {
+    condition     = local.logs_bucket_enabled == true
+    error_message = "Logs bucket should be enabled when session logging is enabled and no bucket name is provided"
+  }
+}