Skip to content

Add disabled tasks support, tests, and doc#5486

Open
ricardoc-db wants to merge 5 commits intodatabricks:mainfrom
ricardoc-db:JOBS-27406-disabled-tasks-support-tests-and-doc
Open

Add disabled tasks support, tests, and doc#5486
ricardoc-db wants to merge 5 commits intodatabricks:mainfrom
ricardoc-db:JOBS-27406-disabled-tasks-support-tests-and-doc

Conversation

@ricardoc-db
Copy link

@ricardoc-db ricardoc-db commented Mar 17, 2026

Changes

Add disabled tasks support, tests, and doc

Tests

  • make test run locally
  • relevant change in docs/ folder
  • covered with integration tests in internal/acceptance
  • using Go SDK
  • using TF Plugin Framework
  • has entry in NEXT_CHANGELOG.md file

@ricardoc-db ricardoc-db requested review from a team as code owners March 17, 2026 10:26
@ricardoc-db ricardoc-db requested review from tanmay-db and removed request for a team March 17, 2026 10:26
jobs/job_test.go Outdated
})
}

func TestAccJobDisabledTask(t *testing.T) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I ran the integration test locally and it failed with

expected task 'b' to have Disabled=true

From the logs, I don't see disabled as true in the API response, is this expected?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed the integration test from this PR as the field is not yet working in every single environment.
Integration test was moved to another PR: #5506

Ricardo Costa added 2 commits March 23, 2026 07:48
…o SDK structs

Replaces HTTPFixture-based test with MockWorkspaceClientFunc pattern using
jobs.Task SDK structs. Removes unnecessary Disabled field from JobTaskSettings
since the schema already picks it up via the SDK. Adds state assertions for
the disabled attribute on both tasks.

Co-authored-by: Isaac
Integration test moved to JOBS-27406-disabled-tasks-integration-test branch.

Co-authored-by: Isaac
@github-actions
Copy link

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/terraform

Inputs:

  • PR number: 5486
  • Commit SHA: 2f15f5a43bebb9e7ba694859e15a5917d8b7223c

Checks will be approved automatically on success.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants