More tests (#2812)

## Changes
Add tests to reproduce #2810 

### Linked issues
Resolves #2810

### Functionality
None

### Tests
- [x] added unit tests
- [x] added integration tests

---------

Co-authored-by: Eric Vergnaud <[email protected]>

Author: ericvergnaud

tests/integration/assessment/test_workflows.py

@@ -11,6 +11,8 @@ def test_running_real_assessment_job(
     make_cluster_policy,
     make_cluster_policy_permissions,
     make_dashboard,
+    sql_backend,
+    inventory_schema,
 ) -> None:
     ws_group, _ = installation_ctx.make_ucx_group()
     # TODO: Move `make_cluster_policy` and `make_cluster_policy_permissions` to context like other `make_` methods
@@ -43,7 +45,9 @@ def test_running_real_assessment_job(
     after = installation_ctx.generic_permissions_support.load_as_dict("cluster-policies", cluster_policy.policy_id)
     assert after[ws_group.display_name] == PermissionLevel.CAN_USE
 
-    tables = set(table.name for table in installation_ctx.tables_crawler.snapshot())
     expected_tables = {managed_table.name, external_table.name, tmp_table.name, view.name, non_delta.name}
-    assert len(tables) == len(expected_tables)
-    assert set(tables) == expected_tables
+    assert set(table.name for table in installation_ctx.tables_crawler.snapshot()) == expected_tables
+
+    query = f"SELECT * FROM {inventory_schema}.workflow_problems"
+    for row in sql_backend.fetch(query):
+        assert row['path'] != 'UNKNOWN'

tests/unit/source_code/test_jobs.py

@@ -23,7 +23,7 @@
     DependencyGraph,
     DependencyResolver,
 )
-from databricks.labs.ucx.source_code.jobs import JobProblem, WorkflowLinter, WorkflowTaskContainer
+from databricks.labs.ucx.source_code.jobs import JobProblem, WorkflowLinter, WorkflowTaskContainer, LintingWalker
 from databricks.labs.ucx.source_code.notebooks.loaders import NotebookResolver, NotebookLoader
 from databricks.labs.ucx.source_code.used_table import UsedTablesCrawler
 
@@ -521,3 +521,15 @@ def test_xxx(graph):
     assert workflow_task_container.spark_conf == {"spark.databricks.cluster.profile": "singleNode"}
 
     ws.assert_not_called()
+
+
+def test_linting_walker_populates_paths(dependency_resolver, mock_path_lookup, migration_index):
+    path = mock_path_lookup.resolve(Path("functional/values_across_cells.py"))
+    root = Dependency(NotebookLoader(), path)
+    xgraph = DependencyGraph(root, None, dependency_resolver, mock_path_lookup, CurrentSessionState())
+    walker = LintingWalker(xgraph, set(), mock_path_lookup, "key", CurrentSessionState(), migration_index)
+    advices = 0
+    for advice in walker:
+        advices += 1
+        assert "UNKNOWN" not in advice.path.as_posix()
+    assert advices