Skip to content

Commit

Permalink
Add dummy id for jobs to include on MockInstallationContext.config (#…
Browse files Browse the repository at this point in the history
…3499)

## Changes
Add dummy id for jobs to include on `MockInstallationContext.config` to
avoid assessing all workflows.

### Linked issues

A more proper solution would be to resolve:
databrickslabs/blueprint#179

### Tests

- [x] manually tested
  • Loading branch information
JCZuurmond authored Jan 8, 2025
1 parent edf2713 commit f5c913c
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion tests/integration/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -1128,7 +1128,11 @@ def config(self) -> WorkspaceConfig:
renamed_group_prefix=self.renamed_group_prefix,
include_group_names=self.created_groups,
include_databases=self.created_databases,
include_job_ids=self.created_jobs,
# An empty list is not saved in the installation.save below.
# With a dummy id, we signal to skip all jobs.
# A temporary hack to speed up the WorkflowLinter, proper solution follows from issue:
# https://github.com/databrickslabs/blueprint/issues/179
include_job_ids=self.created_jobs or [1],
include_dashboard_ids=self.created_dashboards,
include_query_ids=self.created_queries,
include_object_permissions=self.created_object_permissions,
Expand Down

0 comments on commit f5c913c

Please sign in to comment.