Skip to content

Commit 8d0e018

Browse files
authored
Unit test fix: test_cluster_remap_error (didn't fix it properly in #4030) (#4032)
## Changes One of the unit tests (`test_cluster_remap_error`) updated in #4030 wasn't done properly, and still wasn't capturing the logs in the right way. This PR fixes that. ### Linked issues Follows #4030. ### Tests - [x] manually tested (against `blueprint@cli-missing-debug-logging`) - [x] updated unit test
1 parent 3d89c45 commit 8d0e018

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/unit/test_cli.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -919,9 +919,9 @@ def test_cluster_remap(ws, caplog) -> None:
919919
def test_cluster_remap_error(ws, caplog) -> None:
920920
prompts = MockPrompts({"Please provide the cluster id's as comma separated value from the above list.*": "1"})
921921
ws.clusters.list.return_value = []
922-
cluster_remap(ws, prompts)
923922
with caplog.at_level(logging.INFO, logger="databricks.labs.ucx"):
924-
assert "No cluster information present in the workspace" in caplog.messages
923+
cluster_remap(ws, prompts)
924+
assert "No cluster information present in the workspace" in caplog.messages
925925

926926

927927
def test_revert_cluster_remap(caplog):

0 commit comments

Comments
 (0)