diff --git a/pkg/synthetics/monitor_locations_integration_test.go b/pkg/synthetics/monitor_locations_integration_test.go index ab56e4243..56737c812 100644 --- a/pkg/synthetics/monitor_locations_integration_test.go +++ b/pkg/synthetics/monitor_locations_integration_test.go @@ -12,7 +12,7 @@ import ( ) func TestIntegrationGetMonitorLocations(t *testing.T) { - t.Parallel() + t.Skipf("Synthetics REST API is deprecated") tc := mock.NewIntegrationTestConfig(t) diff --git a/pkg/synthetics/monitors_integration_test.go b/pkg/synthetics/monitors_integration_test.go index f2789663a..fdb266217 100644 --- a/pkg/synthetics/monitors_integration_test.go +++ b/pkg/synthetics/monitors_integration_test.go @@ -27,7 +27,7 @@ var ( ) func TestIntegrationMonitors(t *testing.T) { - t.Parallel() + t.Skipf("Synthetics REST API is deprecated") tc := mock.NewIntegrationTestConfig(t) diff --git a/pkg/synthetics/secure_credentials_integration_test.go b/pkg/synthetics/secure_credentials_integration_test.go index 1459126a2..928315640 100644 --- a/pkg/synthetics/secure_credentials_integration_test.go +++ b/pkg/synthetics/secure_credentials_integration_test.go @@ -24,7 +24,7 @@ var ( ) func TestIntegrationSecureCredentials(t *testing.T) { - t.Parallel() + t.Skipf("Synthetics REST API is deprecated") tc := mock.NewIntegrationTestConfig(t) synthetics := New(tc) diff --git a/pkg/synthetics/synthetics_api_integration_test.go b/pkg/synthetics/synthetics_api_integration_test.go index 3656c6312..55aeed367 100644 --- a/pkg/synthetics/synthetics_api_integration_test.go +++ b/pkg/synthetics/synthetics_api_integration_test.go @@ -19,8 +19,6 @@ import ( var tv bool = true func TestSyntheticsSecureCredential_Basic(t *testing.T) { - t.Parallel() - testAccountID, err := mock.GetTestAccountID() if err != nil { t.Skipf("%s", err) @@ -41,8 +39,6 @@ func TestSyntheticsSecureCredential_Basic(t *testing.T) { } func TestSyntheticsSecureCredential_Error(t *testing.T) { - t.Parallel() - testAccountID, err := mock.GetTestAccountID() if err != nil { t.Skipf("%s", err) @@ -56,7 +52,6 @@ func TestSyntheticsSecureCredential_Error(t *testing.T) { } func TestSyntheticsSimpleBrowserMonitor_Basic(t *testing.T) { - t.Parallel() testAccountID, err := mock.GetTestAccountID() if err != nil { @@ -164,7 +159,6 @@ func TestSyntheticsSimpleBrowserMonitor_Basic(t *testing.T) { } func TestSyntheticsSimpleMonitor_Basic(t *testing.T) { - t.Parallel() testAccountID, err := mock.GetTestAccountID() if err != nil { t.Skipf("%s", err) @@ -255,7 +249,6 @@ func TestSyntheticsSimpleMonitor_Basic(t *testing.T) { } func TestSyntheticsScriptApiMonitor_Basic(t *testing.T) { - t.Parallel() testAccountID, err := mock.GetTestAccountID() if err != nil { @@ -361,7 +354,6 @@ func TestSyntheticsScriptApiMonitor_Basic(t *testing.T) { } func TestSyntheticsScriptApiMonitorLegacy_Basic(t *testing.T) { - t.Parallel() testAccountID, err := mock.GetTestAccountID() if err != nil { @@ -457,7 +449,6 @@ func TestSyntheticsScriptApiMonitorLegacy_Basic(t *testing.T) { } func TestSyntheticsScriptBrowserMonitor_Basic(t *testing.T) { - t.Parallel() testAccountID, err := mock.GetTestAccountID() if err != nil { t.Skipf("%s", err) @@ -539,7 +530,6 @@ func TestSyntheticsScriptBrowserMonitor_Basic(t *testing.T) { } func TestSyntheticsScriptBrowserMonitor_InvalidRuntimeValues(t *testing.T) { - t.Parallel() testAccountID, err := mock.GetTestAccountID() if err != nil { t.Skipf("%s", err) @@ -570,7 +560,6 @@ func TestSyntheticsScriptBrowserMonitor_InvalidRuntimeValues(t *testing.T) { } func TestSyntheticsScriptBrowserMonitor_DeviceEmulation(t *testing.T) { - t.Parallel() testAccountID, err := mock.GetTestAccountID() if err != nil { t.Skipf("%s", err) @@ -641,7 +630,6 @@ func TestSyntheticsScriptBrowserMonitor_DeviceEmulation(t *testing.T) { } func TestSyntheticsScriptBrowserMonitor_LegacyRuntime(t *testing.T) { - t.Parallel() testAccountID, err := mock.GetTestAccountID() if err != nil { t.Skipf("%s", err) @@ -702,7 +690,6 @@ func TestSyntheticsScriptBrowserMonitor_LegacyRuntime(t *testing.T) { } func TestSyntheticsPrivateLocation_Basic(t *testing.T) { - t.Parallel() testAccountID, err := mock.GetTestAccountID() if err != nil { @@ -727,7 +714,6 @@ func TestSyntheticsPrivateLocation_Basic(t *testing.T) { } func TestSyntheticsBrokenLinksMonitor_Basic(t *testing.T) { - t.Parallel() testAccountID, err := mock.GetTestAccountID() if err != nil { t.Skipf("%s", err) @@ -782,7 +768,6 @@ func TestSyntheticsBrokenLinksMonitor_Basic(t *testing.T) { } func TestSyntheticsCertCheckMonitor_Basic(t *testing.T) { - t.Parallel() testAccountID, err := mock.GetTestAccountID() if err != nil { t.Skipf("%s", err) @@ -840,7 +825,6 @@ func TestSyntheticsCertCheckMonitor_Basic(t *testing.T) { } func TestSyntheticsStepMonitor_Basic(t *testing.T) { - t.Parallel() testAccountID, err := mock.GetTestAccountID() if err != nil { t.Skipf("%s", err) @@ -908,6 +892,8 @@ func TestSyntheticsStepMonitor_Basic(t *testing.T) { }, } + time.Sleep(10 * time.Second) + updatedMonitor, err := a.SyntheticsUpdateStepMonitor(createdMonitor.Monitor.GUID, monitorUpdateInput) require.NoError(t, err) require.NotNil(t, updatedMonitor.Monitor) @@ -915,6 +901,8 @@ func TestSyntheticsStepMonitor_Basic(t *testing.T) { require.Equal(t, monitorNameUpdate, updatedMonitor.Monitor.Name) require.Equal(t, 3, len(updatedMonitor.Monitor.Steps)) + time.Sleep(10 * time.Second) + deletedMonitor, err := a.SyntheticsDeleteMonitor(createdMonitor.Monitor.GUID) require.NoError(t, err) require.NotNil(t, deletedMonitor) @@ -922,7 +910,6 @@ func TestSyntheticsStepMonitor_Basic(t *testing.T) { } func TestSyntheticsStepMonitor_GetSteps(t *testing.T) { - t.Parallel() testAccountID, err := mock.GetTestAccountID() if err != nil { t.Skipf("%s", err) @@ -979,7 +966,6 @@ func TestSyntheticsStepMonitor_GetSteps(t *testing.T) { } func TestSyntheticsStepMonitor_GetScript(t *testing.T) { - t.Parallel() testAccountID, err := mock.GetTestAccountID() if err != nil { t.Skipf("%s", err) @@ -1029,7 +1015,6 @@ func newIntegrationTestClient(t *testing.T) Synthetics { // syntheticsStartAutomatedTest mutation to create a batch with these three monitors. The expected // behaviour of this test is to return a valid batchId and throw no error. func TestSyntheticsStartAutomatedTest_Basic(t *testing.T) { - t.Parallel() t.Skipf( `Temporarily skipping tests associated with the Synthetics Automated Tests feature, ` + `given the API is currently unstable and endpoint access is not configured to all accounts at the moment. @@ -1176,7 +1161,6 @@ func TestSyntheticsStartAutomatedTest_Basic(t *testing.T) { // TestSyntheticsStartAutomatedTest_Error performs a test on the syntheticsStartAutomatedTest mutation by specifying // an invalid GUID in the input field of a monitor to obtain an error, in alignment with expected behaviour. func TestSyntheticsStartAutomatedTest_Error(t *testing.T) { - t.Parallel() t.Skipf( `Temporarily skipping tests associated with the Synthetics Automated Tests feature, ` + `given the API is currently unstable and endpoint access is not configured to all accounts at the moment. @@ -1198,7 +1182,6 @@ func TestSyntheticsStartAutomatedTest_Error(t *testing.T) { // TestSyntheticsAutomatedTestResults_TwoMonitorsTest performs a test by creating two scripted browser monitors, // creating a batch with those monitors, querying the batch and evaluating the status accordingly. func TestSyntheticsAutomatedTestResults_TwoMonitorsTest(t *testing.T) { - t.Parallel() t.Skipf( `Temporarily skipping tests associated with the Synthetics Automated Tests feature, ` + `given the API is currently unstable and endpoint access is not configured to all accounts at the moment. @@ -1285,7 +1268,6 @@ func TestSyntheticsAutomatedTestResults_TwoMonitorsTest(t *testing.T) { // creating a batch with the monitor, querying the batch and evaluating the status accordingly. Since the scripted // browser monitor is bound to fail, this tests inspects the consolidated status and the status of the monitor. func TestSyntheticsAutomatedTestResults_OneMonitorTest(t *testing.T) { - t.Parallel() t.Skipf( `Temporarily skipping tests associated with the Synthetics Automated Tests feature, ` + `given the API is currently unstable and endpoint access is not configured to all accounts at the moment. @@ -1351,7 +1333,6 @@ func TestSyntheticsAutomatedTestResults_OneMonitorTest(t *testing.T) { // TestSyntheticsAutomatedTestResults_ErrorTest performs a test on the automatedTestResults query by // specifying an invalid batchId, which is expected to throw an error. func TestSyntheticsAutomatedTestResults_ErrorTest(t *testing.T) { - t.Parallel() t.Skipf( `Temporarily skipping tests associated with the Synthetics Automated Tests feature, ` + `given the API is currently unstable and endpoint access is not configured to all accounts at the moment.