Remove use of cancel by job id in integration test (#632)

## Changes JobId was recently made optional for the cancel API. This means `CancelAllRunsByJobId` will no longer exist once we bump the SDK. This PR removes usage of this method from our integration tests and examples to unblock SDK generation. ## Tests Existing tests - [x] `make test` passing - [x] `make fmt` applied - [x] relevant integration tests applied
databricks · Sep 27, 2023 · 02993df · 02993df
1 parent a77197b
commit 02993df
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 6 deletions.
diff --git a/internal/jobs_test.go b/internal/jobs_test.go
@@ -80,7 +80,9 @@ func TestAccJobsApiFullIntegration(t *testing.T) {
 	})
 	require.NoError(t, err)
 
-	err = w.Jobs.CancelAllRunsByJobId(ctx, createdJob.JobId)
+	err = w.Jobs.CancelAllRuns(ctx, jobs.CancelAllRuns{
+		JobId: createdJob.JobId,
+	})
 	require.NoError(t, err)
 
 	runNowResponse, err := w.Jobs.RunNow(ctx, jobs.RunNow{

diff --git a/service/iam/model.go b/service/iam/model.go
diff --git a/service/jobs/jobs_usage_test.go b/service/jobs/jobs_usage_test.go
diff --git a/service/ml/model_registry_usage_test.go b/service/ml/model_registry_usage_test.go
diff --git a/service/sql/model.go b/service/sql/model.go