Merge pull request #580 from jafowler/do-not-return-custom-pages-without-urls

Author: jacobbednarz

internal/app/cf-terraforming/cmd/generate.go

@@ -412,10 +412,18 @@ func generateResources() func(cmd *cobra.Command, args []string) {
 				}
 			}
 
+			var newJsonStructData []interface{}
 			// remap ID to the "type" field
 			for i := 0; i < resourceCount; i++ {
 				jsonStructData[i].(map[string]interface{})["type"] = jsonStructData[i].(map[string]interface{})["id"]
+				// we only want repsonses that have 'url'
+				if jsonStructData[i].(map[string]interface{})["url"] != nil {
+					newJsonStructData = append(newJsonStructData, jsonStructData[i])
+				}
 			}
+			jsonStructData = newJsonStructData
+			resourceCount = len(jsonStructData)
+
 		case "cloudflare_custom_hostname_fallback_origin":
 			var jsonPayload []cloudflare.CustomHostnameFallbackOrigin
 			apiCall, err := api.CustomHostnameFallbackOrigin(context.Background(), zoneID)

testdata/cloudflare/cloudflare_custom_pages_account.yaml

@@ -27,6 +27,18 @@ interactions:
             ],
             "preview_target": "preview:target",
             "description": "example"
+          },
+          {
+            "id": "basic_challenge",
+            "created_on": "2014-01-01T05:20:00.12345Z",
+            "modified_on": "2014-01-01T05:20:00.12345Z",
+            "url": null,
+            "state": "customized",
+            "required_tokens": [
+              "::CAPTCHA_BOX::"
+            ],
+            "preview_target": "preview:target",
+            "description": "example"
           }
         ]
       }

testdata/cloudflare/cloudflare_custom_pages_zone.yaml

@@ -27,6 +27,18 @@ interactions:
             ],
             "preview_target": "preview:target",
             "description": "example"
+          },
+          {
+            "id": "basic_challenge",
+            "created_on": "2014-01-01T05:20:00.12345Z",
+            "modified_on": "2014-01-01T05:20:00.12345Z",
+            "url": null,
+            "state": "customized",
+            "required_tokens": [
+              "::CAPTCHA_BOX::"
+            ],
+            "preview_target": "preview:target",
+            "description": "example"
           }
         ]
       }