Fix workflow_start_time should be start_time

Author: JCZuurmond

src/databricks/labs/ucx/contexts/workflow_task.py

@@ -116,5 +116,5 @@ def workflow_run_recorder(self):
             workflow_id=int(self.named_parameters["job_id"]),
             workflow_run_id=int(self.named_parameters["parent_run_id"]),
             workflow_run_attempt=int(self.named_parameters.get("attempt", 0)),
-            workflow_start_time=self.named_parameters["workflow_start_time"],
+            workflow_start_time=self.named_parameters["start_time"],
         )

tests/integration/progress/test_workflow_runs.py

@@ -9,7 +9,7 @@ def test_workflow_run_recorder_records_workflow_run(installation_ctx) -> None:
         "workflow": "test",
         "job_id": "123",
         "parent_run_id": "456",
-        "workflow_start_time": start_time.isoformat(),
+        "start_time": start_time.isoformat(),
     }
     ctx = installation_ctx.replace(named_parameters=named_parameters)
     ctx.progress_tracking_installation.run()

tests/integration/progress/test_workflows.py

@@ -16,6 +16,9 @@ def test_running_real_migration_progress_job(installation_ctx: MockInstallationC
     installation_ctx.deployed_workflows.run_workflow("assessment")
     assert installation_ctx.deployed_workflows.validate_step("assessment")
 
+    # After the assessment, a user (maybe) installs the progress tracking
+    installation_ctx.progress_tracking_installation.run()
+
     # Run the migration-progress workflow until completion.
     installation_ctx.deployed_workflows.run_workflow("migration-progress-experimental")
     assert installation_ctx.deployed_workflows.validate_step("migration-progress-experimental")