fix: include cloudflare_domain in integration tests (#80)

Author: tamas-jozsa

.gitignore

@@ -1,6 +1,10 @@
 bin/
 .terraformrc-tf-migrate
 
+# Binaries (should be in bin/ directory only)
+/tf-migrate
+/e2e-runner
+
 # Local provider dev overrides
 .terraformrc-tf-migrate
 

Makefile

@@ -41,4 +41,5 @@ lint-testdata:
 clean:
 	@echo "Cleaning build artifacts..."
 	@rm -rf bin/
+	@rm -f tf-migrate e2e-runner
 	@echo "Clean complete"

integration/v4_to_v5/testdata/account_member/expected/account_member.tf

@@ -12,6 +12,11 @@ variable "cloudflare_zone_id" {
   type        = string
 }
 
+variable "cloudflare_domain" {
+  description = "Cloudflare domain for testing"
+  type        = string
+}
+
 # Test Case 1: Basic account member
 resource "cloudflare_account_member" "basic_am" {
   account_id = var.cloudflare_account_id
@@ -28,4 +33,4 @@ resource "cloudflare_account_member" "full_am" {
     "e58cefd75d7adae0b761796c28815e5c",
     "a4154d230e664f8b3e6e5c95a8cc812f"
   ]
-}
+}

integration/v4_to_v5/testdata/account_member/input/account_member.tf

@@ -12,6 +12,11 @@ variable "cloudflare_zone_id" {
   type        = string
 }
 
+variable "cloudflare_domain" {
+  description = "Cloudflare domain for testing"
+  type        = string
+}
+
 # Test Case 1: Basic account member
 resource "cloudflare_account_member" "basic_am" {
     account_id = var.cloudflare_account_id
@@ -28,4 +33,4 @@ resource "cloudflare_account_member" "full_am" {
         "e58cefd75d7adae0b761796c28815e5c",
         "a4154d230e664f8b3e6e5c95a8cc812f"
     ]
-}
+}

integration/v4_to_v5/testdata/api_token/expected/api_token.tf

@@ -12,6 +12,11 @@ variable "cloudflare_zone_id" {
   type        = string
 }
 
+variable "cloudflare_domain" {
+  description = "Cloudflare domain for testing"
+  type        = string
+}
+
 # Test Case 1: Basic API token with single policy
 resource "cloudflare_api_token" "basic_token" {
   name = "${local.name_prefix} Basic API Token"
@@ -268,4 +273,4 @@ resource "cloudflare_api_token" "variable_resources_token" {
       id = "c8fed203ed3043cba015a93ad1616f1f"
     }]
   }]
-}
+}

integration/v4_to_v5/testdata/api_token/input/api_token.tf

@@ -12,6 +12,11 @@ variable "cloudflare_zone_id" {
   type        = string
 }
 
+variable "cloudflare_domain" {
+  description = "Cloudflare domain for testing"
+  type        = string
+}
+
 # Test Case 1: Basic API token with single policy
 resource "cloudflare_api_token" "basic_token" {
   name = "${local.name_prefix} Basic API Token"
@@ -268,4 +273,4 @@ resource "cloudflare_api_token" "variable_resources_token" {
       "com.cloudflare.api.account.zone.${var.cloudflare_zone_id}" = "*"
     }
   }
-}
+}

integration/v4_to_v5/testdata/argo/expected/argo.tf

@@ -8,6 +8,11 @@ variable "cloudflare_zone_id" {
   type        = string
 }
 
+variable "cloudflare_domain" {
+  description = "Cloudflare domain for testing"
+  type        = string
+}
+
 resource "cloudflare_argo_smart_routing" "both_with_lifecycle" {
   zone_id = var.cloudflare_zone_id
   value   = "on"

integration/v4_to_v5/testdata/argo/input/argo.tf

@@ -8,6 +8,11 @@ variable "cloudflare_zone_id" {
   type        = string
 }
 
+variable "cloudflare_domain" {
+  description = "Cloudflare domain for testing"
+  type        = string
+}
+
 resource "cloudflare_argo" "both_with_lifecycle" {
   zone_id        = var.cloudflare_zone_id
   smart_routing  = "on"

integration/v4_to_v5/testdata/bot_management/expected/bot_management.tf

@@ -8,6 +8,11 @@ variable "cloudflare_zone_id" {
   type        = string
 }
 
+variable "cloudflare_domain" {
+  description = "Cloudflare domain for testing"
+  type        = string
+}
+
 # Comprehensive bot management test with available fields
 # Note: bot_management is a singleton per zone - only one config per zone
 resource "cloudflare_bot_management" "test" {

integration/v4_to_v5/testdata/bot_management/input/bot_management.tf

@@ -8,6 +8,11 @@ variable "cloudflare_zone_id" {
   type        = string
 }
 
+variable "cloudflare_domain" {
+  description = "Cloudflare domain for testing"
+  type        = string
+}
+
 # Comprehensive bot management test with available fields
 # Note: bot_management is a singleton per zone - only one config per zone
 resource "cloudflare_bot_management" "test" {