Fix org policies factories

modules/folder/organization-policies.tf

@@ -17,14 +17,10 @@
 # tfdoc:file:description Folder-level organization policies.
 
 locals {
-  _factory_data_raw = (
+  _factory_data_raw = merge([
-    var.org_policies_data_path == null
+    for f in try(fileset(var.org_policies_data_path, "*.yaml"), []) :
-    ? tomap({})
+    yamldecode(file("${var.org_policies_data_path}/${f}"))
-    : merge([
+  ]...)
-      for f in fileset(var.org_policies_data_path, "*.yaml") :
-      yamldecode(file("${var.org_policies_data_path}/${f}"))
-    ]...)
-  )
 
   # simulate applying defaults to data coming from yaml files
   _factory_data = {

modules/organization/org-policy-custom-constraints.tf

@@ -15,14 +15,11 @@
  */
 
 locals {
-  _custom_constraints_factory_data_raw = (
+  _custom_constraints_factory_data_raw = merge([
-    var.org_policy_custom_constraints_data_path == null
+    for f in try(fileset(var.org_policy_custom_constraints_data_path, "*.yaml"), []) :
-    ? tomap({})
+    yamldecode(file("${var.org_policy_custom_constraints_data_path}/${f}"))
-    : merge([
+  ]...)
-      for f in fileset(var.org_policy_custom_constraints_data_path, "*.yaml") :
+
-      yamldecode(file("${var.org_policy_custom_constraints_data_path}/${f}"))
-    ]...)
-  )
 
   _custom_constraints_factory_data = {
     for k, v in local._custom_constraints_factory_data_raw :

modules/organization/organization-policies.tf

@@ -17,14 +17,10 @@
 # tfdoc:file:description Organization-level organization policies.
 
 locals {
-  _factory_data_raw = (
+  _factory_data_raw = merge([
-    var.org_policies_data_path == null
+    for f in try(fileset(var.org_policies_data_path, "*.yaml"), []) :
-    ? tomap({})
+    yamldecode(file("${var.org_policies_data_path}/${f}"))
-    : merge([
+  ]...)
-      for f in fileset(var.org_policies_data_path, "*.yaml") :
-      yamldecode(file("${var.org_policies_data_path}/${f}"))
-    ]...)
-  )
 
   # simulate applying defaults to data coming from yaml files
   _factory_data = {

modules/project/organization-policies.tf

@@ -17,14 +17,10 @@
 # tfdoc:file:description Project-level organization policies.
 
 locals {
-  _factory_data_raw = (
+  _factory_data_raw = merge([
-    var.org_policies_data_path == null
+    for f in try(fileset(var.org_policies_data_path, "*.yaml"), []) :
-    ? tomap({})
+    yamldecode(file("${var.org_policies_data_path}/${f}"))
-    : merge([
+  ]...)
-      for f in fileset(var.org_policies_data_path, "*.yaml") :
-      yamldecode(file("${var.org_policies_data_path}/${f}"))
-    ]...)
-  )
 
   # simulate applying defaults to data coming from yaml files
   _factory_data = {

tests/modules/organization/test_plan_org_policies_modules.py

@@ -38,8 +38,8 @@ def test_policy_implementation():
       '+# tfdoc:file:description Folder-level organization policies.\n',
       ' \n',
       ' locals {\n',
-      '   _factory_data_raw = (\n',
+      '   _factory_data_raw = merge([\n',
-      '@@ -69,8 +69,8 @@\n',
+      '@@ -65,8 +65,8 @@\n',
       '   org_policies = {\n',
       '     for k, v in local._org_policies :\n',
       '     k => merge(v, {\n',
@@ -64,8 +64,8 @@ def test_policy_implementation():
       '+# tfdoc:file:description Organization-level organization policies.\n',
       ' \n',
       ' locals {\n',
-      '   _factory_data_raw = (\n',
+      '   _factory_data_raw = merge([\n',
-      '@@ -69,8 +69,8 @@\n',
+      '@@ -65,8 +65,8 @@\n',
       '   org_policies = {\n',
       '     for k, v in local._org_policies :\n',
       '     k => merge(v, {\n',
@@ -76,7 +76,7 @@ def test_policy_implementation():
       ' \n',
       '       is_boolean_policy = v.allow == null && v.deny == null\n',
       '       has_values = (\n',
-      '@@ -143,4 +143,13 @@\n',
+      '@@ -139,4 +139,13 @@\n',
       '       }\n',
       '     }\n',
       '   }\n',