fix: Change variable type and loop check

Author: bryantbiggs

.pre-commit-config.yaml

@@ -1,6 +1,6 @@
 repos:
   - repo: https://github.com/antonbabenko/pre-commit-terraform
-    rev: v1.99.0
+    rev: v1.99.4
     hooks:
       - id: terraform_fmt
       - id: terraform_wrapper_module_for_each

README.md

@@ -414,7 +414,7 @@ No modules.
 | <a name="input_ipam_pools"></a> [ipam\_pools](#input\_ipam\_pools) | The IPAM pools to use with the load balancer | `map(string)` | `{}` | no |
 | <a name="input_listeners"></a> [listeners](#input\_listeners) | Map of listener configurations to create | `any` | `{}` | no |
 | <a name="input_load_balancer_type"></a> [load\_balancer\_type](#input\_load\_balancer\_type) | The type of load balancer to create. Possible values are `application`, `gateway`, or `network`. The default value is `application` | `string` | `"application"` | no |
-| <a name="input_minimum_load_balancer_capacity"></a> [minimum\_load\_balancer\_capacity](#input\_minimum\_load\_balancer\_capacity) | Minimum capacity for a load balancer. Only valid for Load Balancers of type `application` or `network` | `number` | `null` | no |
+| <a name="input_minimum_load_balancer_capacity"></a> [minimum\_load\_balancer\_capacity](#input\_minimum\_load\_balancer\_capacity) | Minimum capacity for a load balancer. Only valid for Load Balancers of type `application` or `network` | `any` | `{}` | no |
 | <a name="input_name"></a> [name](#input\_name) | The name of the LB. This name must be unique within your AWS account, can have a maximum of 32 characters, must contain only alphanumeric characters or hyphens, and must not begin or end with a hyphen | `string` | `null` | no |
 | <a name="input_name_prefix"></a> [name\_prefix](#input\_name\_prefix) | Creates a unique name beginning with the specified prefix. Conflicts with `name` | `string` | `null` | no |
 | <a name="input_preserve_host_header"></a> [preserve\_host\_header](#input\_preserve\_host\_header) | Indicates whether the Application Load Balancer should preserve the Host header in the HTTP request and send it to the target without any change. Defaults to `false` | `bool` | `null` | no |

examples/complete-alb/main.tf

@@ -71,7 +71,9 @@ module "alb" {
     ipv4_ipam_pool_id = aws_vpc_ipam_pool.this.id
   }
 
-  minimum_load_balancer_capacity = 10
+  minimum_load_balancer_capacity = {
+    capacity_units = 10
+  }
 
   client_keep_alive = 7200
 

main.tf

@@ -40,10 +40,10 @@ resource "aws_lb" "this" {
   }
 
   dynamic "minimum_load_balancer_capacity" {
-    for_each = var.minimum_load_balancer_capacity != null ? [var.minimum_load_balancer_capacity] : []
+    for_each = length(var.minimum_load_balancer_capacity) > 0 ? [var.minimum_load_balancer_capacity] : []
 
     content {
-      capacity_units = minimum_load_balancer_capacity.value
+      capacity_units = minimum_load_balancer_capacity.value.capacity_units
     }
   }
 

variables.tf

@@ -136,8 +136,8 @@ variable "enforce_security_group_inbound_rules_on_private_link_traffic" {
 
 variable "minimum_load_balancer_capacity" {
   description = "Minimum capacity for a load balancer. Only valid for Load Balancers of type `application` or `network`"
-  type        = number
-  default     = null
+  type        = any
+  default     = {}
 }
 
 variable "name" {

wrappers/main.tf

@@ -30,7 +30,7 @@ module "wrapper" {
   ipam_pools                                                   = try(each.value.ipam_pools, var.defaults.ipam_pools, {})
   listeners                                                    = try(each.value.listeners, var.defaults.listeners, {})
   load_balancer_type                                           = try(each.value.load_balancer_type, var.defaults.load_balancer_type, "application")
-  minimum_load_balancer_capacity                               = try(each.value.minimum_load_balancer_capacity, var.defaults.minimum_load_balancer_capacity, null)
+  minimum_load_balancer_capacity                               = try(each.value.minimum_load_balancer_capacity, var.defaults.minimum_load_balancer_capacity, {})
   name                                                         = try(each.value.name, var.defaults.name, null)
   name_prefix                                                  = try(each.value.name_prefix, var.defaults.name_prefix, null)
   preserve_host_header                                         = try(each.value.preserve_host_header, var.defaults.preserve_host_header, null)