Partial revert "Let dashboard name adhere to naming convention" (#3795)

## Changes

Reverts back to dashboard names with special characters like spaces and
brackets

### Linked issues

Requires #3799
Partially reverts #3789
Reverts 048bc8f

### Functionality

- [x] modified migration progress dashboard

### Tests

- [x] manually tested

---------

Co-authored-by: pritishpai <[email protected]>

Author: JCZuurmond

src/databricks/labs/ucx/install.py

@@ -662,16 +662,7 @@ def _create_dashboard(self, folder: Path, *, parent_path: str) -> None:
                 catalog_to_replace="ucx_catalog",
             )
         )
-        metadata.display_name = f"{self._name('UCX ')} {folder.parent.stem.title()} ({folder.stem.title()})"
-        # TODO: Remove temporary fix for the dashboard name:
-        # resource names should only contain alphanumeric characters (a-z, A-Z, 0-9), hyphens (-), or underscores (_)
-        metadata.display_name = (
-            metadata.display_name.replace(" ", "_")
-            .replace("[", "-")
-            .replace("]", "-")
-            .replace("(", "")
-            .replace(")", "")
-        )
+        metadata.display_name = self._name(f"{folder.parent.stem.title()}_{folder.stem.title()}")
         reference = f"{folder.parent.stem}_{folder.stem}".lower()
         dashboard_id = self._install_state.dashboards.get(reference)
         if dashboard_id is not None:

src/databricks/labs/ucx/installer/mixins.py

@@ -19,7 +19,7 @@ def __init__(self, config: WorkspaceConfig, installation: Installation, ws: Work
 
     def _name(self, name: str) -> str:
         prefix = os.path.basename(self._installation.install_folder()).removeprefix('.')
-        return f"[{prefix.upper()}] {name}"
+        return f"{prefix.upper()}_{name}"
 
     @property
     def _my_username(self):

tests/unit/install/test_install.py

@@ -153,7 +153,7 @@ def test_install_cluster_override_jobs(ws, mock_installation) -> None:
 
     workflows_installation.create_jobs()
 
-    tasks = created_job_tasks(ws, '[MOCK] assessment')
+    tasks = created_job_tasks(ws, 'MOCK_assessment')
     assert tasks['assess_jobs'].existing_cluster_id == 'one'
     assert tasks['crawl_grants'].existing_cluster_id == 'two'
     wheels.upload_to_wsfs.assert_called_once()
@@ -175,7 +175,7 @@ def test_writeable_dbfs(ws, tmp_path, mock_installation) -> None:
 
     workflows_installation.create_jobs()
 
-    job = created_job(ws, '[MOCK] assessment')
+    job = created_job(ws, 'MOCK_assessment')
     job_clusters = {_.job_cluster_key: _ for _ in job['job_clusters']}
     assert 'main' in job_clusters
     assert 'tacl' in job_clusters