Skip to content

Fix failing unit tests#282

Merged
asnare merged 6 commits into
mainfrom
fix/broken-tests
Jun 24, 2026
Merged

Fix failing unit tests#282
asnare merged 6 commits into
mainfrom
fix/broken-tests

Conversation

@asnare

@asnare asnare commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Changes

This PR works around a behaviour change introduced by Databricks SDK v0.102.0 that causes unit tests to fail if they are using the Config class. Specifically, during __init__() the Config() class now performs a HTTP request against the host, and retries for 5 minutes if the host does not exist.

Unit tests typically fill in a bogus host, which was fine previously because initialising Config was inert and didn't trigger network calls. Now that it does, some sort of workaround (such as monkey-patching) is required.

Incidental change:

  • Dependency-management for the project was out-of-date, and stopped working because the lockfile referred directly to an internal PyPi proxy that is no longer intended for use. As such the plumbing for locking dependencies was updated, and the lockfile regenerated.

Linked issues

Tests, same as: databrickslabs/blueprint#331
For Makefile, same as: databrickslabs/blueprint#392

Tests

  • updated unit tests

@asnare asnare self-assigned this Jun 12, 2026
@asnare asnare added the bug Something isn't working label Jun 12, 2026
@asnare asnare added this to UCX Jun 12, 2026
@asnare asnare temporarily deployed to account-admin June 12, 2026 16:09 — with GitHub Actions Inactive
@asnare asnare moved this to Ready for Review in UCX Jun 12, 2026
@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown

✅ 40/40 passed, 6 skipped, 14m20s total

Running from acceptance #253

Comment thread tests/unit/conftest.py Outdated
Comment thread tests/unit/conftest.py Outdated
@asnare asnare temporarily deployed to account-admin June 24, 2026 09:04 — with GitHub Actions Inactive
@asnare asnare requested a review from m-abulazm June 24, 2026 09:05
@asnare asnare merged commit ad3fb5b into main Jun 24, 2026
9 checks passed
@asnare asnare deleted the fix/broken-tests branch June 24, 2026 16:24
@github-project-automation github-project-automation Bot moved this from Ready for Review to Done in UCX Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working tech debt

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants