[datadog_synthetics_test] Add missing disable_cors option (#1605)

* [datadog_synthetics_test] Add missing disable_cors option

* Fix indent

* make docs

* Add recorded cassettes

Co-authored-by: Jordan Pittier <[email protected]>

Author: nkzou

datadog/resource_datadog_synthetics_test_.go

@@ -583,6 +583,11 @@ func syntheticsTestOptionsList() *schema.Schema {
 					Type:        schema.TypeBool,
 					Optional:    true,
 				},
+				"disable_cors": {
+					Description: "Disable Cross-Origin Resource Sharing for browser tests.",
+					Type:        schema.TypeBool,
+					Optional:    true,
+				},
 				"initial_navigation_timeout": {
 					Description: "Timeout before declaring the initial step as failed (in seconds) for browser tests.",
 					Type:        schema.TypeInt,
@@ -1934,6 +1939,10 @@ func buildSyntheticsBrowserTestStruct(d *schema.ResourceData) *datadogV1.Synthet
 			options.SetDisableCsp(disableCsp.(bool))
 		}
 
+		if disableCors, ok := d.GetOk("options_list.0.disable_cors"); ok {
+			options.SetDisableCors(disableCors.(bool))
+		}
+
 		if initialNavigationTimeout, ok := d.GetOk("options_list.0.initial_navigation_timeout"); ok {
 			options.SetInitialNavigationTimeout(int64(initialNavigationTimeout.(int)))
 		}
@@ -2433,6 +2442,9 @@ func updateSyntheticsBrowserTestLocalState(d *schema.ResourceData, syntheticsTes
 	if actualOptions.HasDisableCsp() {
 		localOptionsList["disable_csp"] = actualOptions.GetDisableCsp()
 	}
+	if actualOptions.HasDisableCors() {
+		localOptionsList["disable_cors"] = actualOptions.GetDisableCors()
+	}
 	if actualOptions.HasInitialNavigationTimeout() {
 		localOptionsList["initial_navigation_timeout"] = actualOptions.GetInitialNavigationTimeout()
 	}

datadog/tests/cassettes/TestAccDatadogSyntheticsBrowserTest_Basic.freeze

@@ -1 +1 @@
-2022-09-06T21:35:16.447324+02:00
+2022-10-19T09:52:38.20238-04:00

datadog/tests/cassettes/TestAccDatadogSyntheticsBrowserTest_Basic.yaml

@@ -1 +1 @@
-2022-09-06T21:36:32.913866+02:00
+2022-10-19T09:52:06.698853-04:00

datadog/tests/cassettes/TestAccDatadogSyntheticsBrowserTest_Updated.freeze

@@ -1 +1 @@
-2022-09-06T21:36:51.410685+02:00
+2022-10-19T09:52:06.698853-04:00

datadog/tests/cassettes/TestAccDatadogSyntheticsBrowserTest_Updated.yaml

@@ -1 +1 @@
-2022-09-06T21:36:03.671229+02:00
+2022-10-19T09:52:56.69711-04:00

datadog/tests/cassettes/TestAccDatadogSyntheticsBrowserTest_Updated_RumSettings.freeze

@@ -2229,6 +2229,8 @@ func createSyntheticsBrowserTestStep(ctx context.Context, accProvider func() (*s
 				"datadog_synthetics_test.bar", "options_list.0.ignore_server_certificate_error", "true"),
 			resource.TestCheckResourceAttr(
 				"datadog_synthetics_test.bar", "options_list.0.disable_csp", "true"),
+			resource.TestCheckResourceAttr(
+				"datadog_synthetics_test.bar", "options_list.0.disable_cors", "true"),
 			resource.TestCheckResourceAttr(
 				"datadog_synthetics_test.bar", "options_list.0.initial_navigation_timeout", "150"),
 			resource.TestCheckResourceAttr(
@@ -2342,6 +2344,7 @@ resource "datadog_synthetics_test" "bar" {
 
 		ignore_server_certificate_error = true
 		disable_csp = true
+		disable_cors = true
 		initial_navigation_timeout = 150
 	}
 

datadog/tests/cassettes/TestAccDatadogSyntheticsBrowserTest_Updated_RumSettings.yaml

@@ -618,6 +618,7 @@ Optional:
 - `allow_insecure` (Boolean) Allows loading insecure content for an HTTP test.
 - `check_certificate_revocation` (Boolean) For SSL test, whether or not the test should fail on revoked certificate in stapled OCSP.
 - `ci` (Block List, Max: 1) CI/CD options for a Synthetic test. (see [below for nested schema](#nestedblock--options_list--ci))
+- `disable_cors` (Boolean) Disable Cross-Origin Resource Sharing for browser tests.
 - `disable_csp` (Boolean) Disable Content Security Policy for browser tests.
 - `follow_redirects` (Boolean) Determines whether or not the API HTTP test should follow redirects.
 - `ignore_server_certificate_error` (Boolean) Ignore server certificate error.