add functional test

databrickslabs · Jun 7, 2024 · 394be93 · 394be93
1 parent b6f27b0
commit 394be93
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/databricks/labs/ucx/source_code/linters/imports.py b/src/databricks/labs/ucx/source_code/linters/imports.py
@@ -116,7 +116,7 @@ def _convert_dbutils_notebook_run_to_advice(cls, node: NodeNG) -> Advisory:
         if None in paths:
             return Advisory.from_node(
                 'dbutils-notebook-run-dynamic',
-                "Path for 'dbutils.notebook.run' is too complex and requires adjusting the notebook path(s)",
+                "Path for 'dbutils.notebook.run' cannot be computed and requires adjusting the notebook path(s)",
                 node=node,
             )
         return Advisory.from_node(

diff --git a/tests/unit/source_code/samples/functional/missing-resolved-notebook.py b/tests/unit/source_code/samples/functional/missing-resolved-notebook.py
@@ -0,0 +1,7 @@
+# Databricks notebook source
+a = "./leaf1.py"
+# ucx[dbutils-notebook-run-literal:+1:0:+1:23] Call to 'dbutils.notebook.run' will be migrated automatically
+dbutils.notebook.run(a)
+b = some_function()
+# ucx[dbutils-notebook-run-dynamic:+1:0:+1:23] Path for 'dbutils.notebook.run' cannot be computed and requires adjusting the notebook path(s)
+dbutils.notebook.run(b)